Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove oauth provider #15588

Open
wants to merge 7 commits into
base: devel
Choose a base branch
from
Open

Conversation

gravesm
Copy link
Member

@gravesm gravesm commented Oct 17, 2024

SUMMARY

This removes the oauth provider functionality from awx. The oauth2_provider app and all references to it have been removed. Migrations to delete the two tables that locally overwrote oauth2_provider tables are included. This change does not include migrations to delete the tables provided by the oauth2_provider app.

ISSUE TYPE
  • Breaking Change
COMPONENT NAME
  • API
  • Docs
AWX VERSION

ADDITIONAL INFORMATION

@adrisala
Copy link

Shouldn't we also modify references of OAuth usage inside the awx_collection directory? The default login method for the collections right now seems to be OAuth.
cc @tvo318
https://github.com/ansible/awx/blob/devel/awx_collection/README.md#running
https://github.com/ansible/awx/blob/devel/awx_collection/TESTING.md

@gravesm
Copy link
Member Author

gravesm commented Oct 21, 2024

Shouldn't we also modify references of OAuth usage inside the awx_collection directory? The default login method for the collections right now seems to be OAuth.

Docs will definitely need to be changed. #15554 should have fixed the collection to allow basic auth. I'm still working through test failures, though, so there may be further changes needed.

@tvo318
Copy link
Member

tvo318 commented Oct 21, 2024

@adrisala @gravesm Yes I agree, the docs in the awx_collection need to be updated as well. Thanks for working on this!

@github-actions github-actions bot added component:awx_collection issues related to the collection for controlling AWX component:cli labels Oct 24, 2024
@adrisala adrisala self-assigned this Nov 12, 2024
@adrisala adrisala force-pushed the remove-oauth-provider branch 2 times, most recently from 9ffd555 to 8d187f1 Compare November 14, 2024 09:24
This removes the oauth provider functionality from awx. The
oauth2_provider app and all references to it have been removed.
Migrations to delete the two tables that locally overwrote
oauth2_provider tables are included. This change does not include
migrations to delete the tables provided by the oauth2_provider app.

Also not included here are changes to awxkit, awx_collection or the ui.
The changes in ansible#15554 will cause a
few collection tests to fail, depending on what the test configuration
is. This changes the tests to look for a specific warning rather than
counting the number of warnings emitted.
This removes the application and token modules, and their associated
tests, from the collection. In addition, the export module and awxkit
have been updated to no longer export application resources.
Copy link

sonarcloud bot commented Nov 15, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:api component:awx_collection issues related to the collection for controlling AWX component:cli component:docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants