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
no dns service installed if seed has taint disabledDNS #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm, just a minor suggestion
pkg/controller/actuator.go
Outdated
@@ -104,6 +107,12 @@ func (a *actuator) Reconcile(ctx context.Context, ex *extensionsv1alpha1.Extensi | |||
return err | |||
} | |||
|
|||
if gardencorev1beta1helper.TaintsHave(cluster.Seed.Spec.Taints, gardencorev1beta1.SeedTaintDisableDNS) || | |||
cluster.Shoot.Spec.DNS == nil { | |||
a.logger.Info("no shoot dns service installed", "shoot", ex.Namespace) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe add the reason here so that readers of the logs know why :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
already done, see commit below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was talking about the log message, not the comment in the code (which is also good to add, of course) :) - can you add it in the log msg as well + squash the commits?
6d5b89c
to
82a912e
Compare
82a912e
to
dcff5f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! :)
/lgtm
[ci:component:github.com/gardener/cert-management:v0.2.13->v0.2.14]
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #4
Special notes for your reviewer:
Release note: