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 support for subscription activation keys to trusted artifacts #1173

Merged
merged 3 commits into from
Jul 18, 2024

Conversation

brianwcook
Copy link
Contributor

@brianwcook brianwcook commented Jul 18, 2024

… tasks, including buildah-remoate-oci-ta. It also disables podman subscription-manager integration on all remote builders so that subscription-manager commands can be used to configure which repos are enabled.

i had recently had a couple interactions with people smart about subscription mgmt and they all said that using certs the way we are is wrong and eventually will lead to random build failures due to revoked certs. Activation keys solves that problems, and it solves the problem that Camilla was having in this thread, so I just went ahead and did it.

ref slack thread https://redhat-internal.slack.com/archives/C04PZ7H0VA8/p1721226430548109

…artifacts tasks, including buildah-remoate-oci-ta. It also disables podman subscription-manager integration on all remote builders so that subscripition-manager commands can be used to configure which repos are enabled.
@brianwcook brianwcook requested review from lcarva, arewm and rcerven July 18, 2024 02:33
@brianwcook
Copy link
Contributor Author

@brianwcook
Copy link
Contributor Author

I guess the changes need to be moved from TA tasks to the PVC based ones.

@brianwcook
Copy link
Contributor Author

@rcerven if you are able to help get this in shape to merge, I think it will solve a lot of problems. thanks :)

@chmeliik chmeliik added this pull request to the merge queue Jul 18, 2024
Merged via the queue into konflux-ci:main with commit f3ac40b Jul 18, 2024
7 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.

3 participants