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

add missing ah_token option for community repository #238

Merged
merged 1 commit into from
Mar 13, 2023
Merged

add missing ah_token option for community repository #238

merged 1 commit into from
Mar 13, 2023

Conversation

dbk-rabel
Copy link
Contributor

What does this PR do?

Commit 5e5f76d did add ah_token for the certified repository, but not for community repository. This commit adds that.

How should this be tested?

Use the simple example from https://github.com/redhat-cop/ah_configuration/blob/devel/roles/repository/README.md with ah_token as authentication method.

Is there a relevant Issue open for this?

Didn't find one.

Other Relevant info, PRs, etc

@dbk-rabel
Copy link
Contributor Author

By the way: I have no idea, why the automatic tests could be failing on this :-(

Copy link
Contributor

@Tompage1994 Tompage1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks for the contribution

Commit 5e5f76d did add ah_token for the certified repository, but not for community repository. This commit adds that.
@Tompage1994 Tompage1994 enabled auto-merge (squash) March 13, 2023 13:22
@Tompage1994
Copy link
Contributor

CI broken but inconsequential of this. Will merge this and raise a separate issue to fix CI

@sean-m-sullivan sean-m-sullivan merged commit 6b6f8e9 into ansible:devel Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants