Infrastructure/ansible
5
0
Fork 0
Code Issues Pull requests 3 Projects Releases Packages Wiki Activity Actions

openqa/server: allow OAuth2 authentication, enable on lab #2546

Merged
ryanlerch merged 1 commit from openqa-oauth2-stg into main 2025-03-28 21:04:01 +00:00
Conversation 5 Commits 1 Files changed 5 +36 -3
adamwill commented 2025-03-28 20:45:47 +00:00
Contributor
Copy link

OpenID support in FAS is going away. openQA has OAuth2 support.
I've tested this config to work with manual edits on lab, now
ansiblizing it (for lab only to start with).

Signed-off-by: Adam Williamson awilliam@redhat.com

OpenID support in FAS is going away. openQA has OAuth2 support. I've tested this config to work with manual edits on lab, now ansiblizing it (for lab only to start with). Signed-off-by: Adam Williamson <awilliam@redhat.com>
zuul commented 2025-03-28 20:49:18 +00:00
First-time contributor
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/81a5df18e94341ef862bc533afc4cae4

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci https://fedora.softwarefactory-project.io/zuul/buildset/81a5df18e94341ef862bc533afc4cae4 - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/2a466fe1a0a04463a18c1d9e2504954f) : FAILURE in 3m 10s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/30577c64ed984f4a807fa9ac5741b818) : SUCCESS in 2m 32s
kevin commented 2025-03-28 21:02:43 +00:00
Contributor
Copy link

Looks fine to me.

Only thing odd I see is that openqa_oauth2_secret is defined in prod while it's still on OpenID, but that shouldn't matter since it won't use it until you switch it.

Looks fine to me. Only thing odd I see is that openqa_oauth2_secret is defined in prod while it's still on OpenID, but that shouldn't matter since it won't use it until you switch it.
adamwill commented 2025-03-28 21:03:40 +00:00
Author
Contributor
Copy link

Failures aren't related to the PR.

Failures aren't related to the PR.
adamwill commented 2025-03-28 21:04:04 +00:00
Author
Contributor
Copy link

Pull-Request has been merged by adamwill

Pull-Request has been merged by adamwill
adamwill commented 2025-03-28 21:38:25 +00:00
Author
Contributor
Copy link

@kevin yeah, I intentionally did it that way so the enablement for prod is just a single change.

@kevin yeah, I intentionally did it that way so the enablement for prod is just a single change.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
freeze-break-request

   
post-freeze
No labels
freeze-break-request
post-freeze
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Infrastructure/ansible#2546
No description provided.
Delete branch "openqa-oauth2-stg"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?