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

manually back ported some enhacements from infra.aap_configuration_extended #15

Conversation

ivarmu
Copy link
Contributor

@ivarmu ivarmu commented Dec 17, 2024

What does this PR do?

Manually backport some enhancements from the infra.aap_configuration_extended that can be usefull for AAP <=2.4

How should this be tested?

Manually tested

Is there a relevant Issue open for this?

resolves redhat-cop/aap_configuration_extended#50

Other Relevant info, PRs, etc

redhat-cop/aap_configuration_extended#50

@djdanielsson
Copy link
Contributor

Have you tested it to ensure it is working?

@ivarmu
Copy link
Contributor Author

ivarmu commented Dec 18, 2024

Have you tested it to ensure it is working?

Yes for AAP2.4! And detected and fixed some minor errors with the surveys (corrected for both streams, but only PR to the 2.4 by now, as still not tested for 2.5)

Copy link
Contributor

@djdanielsson djdanielsson left a comment

Choose a reason for hiding this comment

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

LGTM

@djdanielsson djdanielsson merged commit 870f24b into redhat-cop:devel Dec 18, 2024
8 of 13 checks passed
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.

back porting some updates to infra.controller_configuration
2 participants