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

skipper: update hostname-credentials-controller #7120

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

AlexanderYastrebov
Copy link
Member

The logic previously implemented by secret-combiner was moved into hostname-credentials-controller.

@AlexanderYastrebov AlexanderYastrebov added the major Major feature changes or updates, e.g. feature rollout to a new country, new API calls. label Mar 4, 2024
@@ -4,6 +4,20 @@ pre_apply:
namespace: kube-system
kind: Deployment

# TODO: remove after rollout
Copy link
Member

Choose a reason for hiding this comment

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

link to PR would be good

@szuecs
Copy link
Member

szuecs commented Mar 4, 2024

👍

The logic previously implemented by secret-combiner was moved into
hostname-credentials-controller.

Signed-off-by: Alexander Yastrebov <[email protected]>
@AlexanderYastrebov AlexanderYastrebov force-pushed the skipper/update-hostname-credentials branch from 7755b14 to 5ee1f4d Compare March 4, 2024 18:06
@szuecs
Copy link
Member

szuecs commented Mar 4, 2024

👍

1 similar comment
@AlexanderYastrebov
Copy link
Member Author

👍

@AlexanderYastrebov AlexanderYastrebov merged commit 82770db into dev Mar 4, 2024
10 checks passed
@AlexanderYastrebov AlexanderYastrebov deleted the skipper/update-hostname-credentials branch March 4, 2024 20:47
AlexanderYastrebov added a commit that referenced this pull request Mar 5, 2024
Follow up on #5851 and #7120

Signed-off-by: Alexander Yastrebov <[email protected]>
@AlexanderYastrebov AlexanderYastrebov mentioned this pull request Mar 5, 2024
1 task
AlexanderYastrebov added a commit that referenced this pull request Mar 5, 2024
Follow up on #5851, #5857 and #7120

Signed-off-by: Alexander Yastrebov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major Major feature changes or updates, e.g. feature rollout to a new country, new API calls. merged/alpha merged/beta merged/stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants