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

Remove dns-controller-manager subcomponent from release vector #337

Merged
merged 6 commits into from
Oct 21, 2024

Conversation

Gerrit91
Copy link
Contributor

@Gerrit91 Gerrit91 commented Oct 16, 2024

Description

The shoot-dns extension ships with the proper version of this component. In order not to have to maintain this dependency anymore, we remove the override from our release vector. Instead an operator can now manually provide an overwrite.

Required Actions

The deployment for the shoot-dns-service extension does not use the overwrite from our release vector for the dns-controller-manager subcomponent anymore. Operators who use a custom overwrite for this component now need to provide it through the new variables `gardener_shoot_dns_service_dns_controller_manager_image_name` and `gardener_shoot_dns_service_dns_controller_manager_image_tag`.

The shoot-dns extension ships with the proper version of this component. We do not need to allow overrides for this.
@Gerrit91 Gerrit91 changed the title Remove dns controller manager Remove dns-controller-manager from release vector Oct 16, 2024
@Gerrit91 Gerrit91 changed the title Remove dns-controller-manager from release vector Remove dns-controller-manager subcomponent from release vector Oct 16, 2024
@Gerrit91 Gerrit91 marked this pull request as ready for review October 16, 2024 11:16
@Gerrit91 Gerrit91 requested a review from a team as a code owner October 16, 2024 11:16
@robertvolkmann
Copy link
Contributor

robertvolkmann commented Oct 18, 2024

Overwriting the image vector has not the desired effect because the image of the dns-controller-manager can only be configured by the removed value, see
https://github.com/gardener/gardener-extension-shoot-dns-service/blob/release-v1.47/charts/gardener-extension-shoot-dns-service/templates/dnsman-deployment.yaml#L50

Gerrit91 and others added 3 commits October 21, 2024 09:35
…oller-deployment.yaml

Co-authored-by: Robert Volkmann <20912167+robertvolkmann@users.noreply.github.com>
@Gerrit91 Gerrit91 merged commit 5aab58c into master Oct 21, 2024
1 check passed
@Gerrit91 Gerrit91 deleted the remove-dns-controller-manager branch October 21, 2024 11:06
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.

2 participants