Skip to content

role-sync-controller: Update to version main-4 #8572

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

Merged

Conversation

k8s-on-aws-manager-app[bot]
Copy link
Contributor

  • create secrets-reader Role and bind it to the subjects

Update container-registry.zalando.net/teapot/role-sync-controller to version main-4
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot added the minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call. label Dec 4, 2024
@demonCoder95
Copy link
Member

Testing this change in #8571 to ensure this makes e2e pass on EKS. Hold onto the merge for now.

@demonCoder95
Copy link
Member

The e2e tests passed against the EKS branch, so the upgrade works! 🥳 🎉

@demonCoder95
Copy link
Member

👍

1 similar comment
@mikkeloscar
Copy link
Contributor

👍

@mikkeloscar mikkeloscar merged commit 92441dc into dev Dec 6, 2024
10 checks passed
@mikkeloscar mikkeloscar deleted the container-registry.zalando.net/teapot/role-sync-controller branch December 6, 2024 07:27
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot mentioned this pull request Dec 6, 2024
This was referenced Dec 6, 2024
@k8s-on-aws-manager-app k8s-on-aws-manager-app bot mentioned this pull request Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged/alpha merged/beta merged/eks merged/stable minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants