Skip to content

Change the method to get the ApiServer hostname#1

Merged
a7vicky merged 1 commit intomasterfrom
garbagego-patch-1
May 12, 2020
Merged

Change the method to get the ApiServer hostname#1
a7vicky merged 1 commit intomasterfrom
garbagego-patch-1

Conversation

@a7vicky
Copy link
Copy Markdown
Owner

@a7vicky a7vicky commented May 12, 2020

The kube-apiserver operator is going to degraded state while changing ApiServer URL in Infrastructure config.

Note: If there is no port specified

Ref:
https://github.com/openshift/cluster-kube-apiserver-operator/blob/master/pkg/operator/certrotationcontroller/externalloadbalancer.go#L24

I0511 16:20:50.553418 1 servicehostname.go:40] syncing servicenetwork hostnames: [172.30.0.1 kubernetes kubernetes.default kubernetes.default.svc kubernetes.default.svc.cluster.local openshift openshift.default openshift.default.svc openshift.default.svc.cluster.local]
I0511 16:20:50.553446 1 panic.go:679] Finished waiting for CertRotation
I0511 16:20:50.553456 1 panic.go:679] Shutting down CertRotation
panic: runtime error: slice bounds out of range [:-1]

goroutine 271 [running]:
github.com/openshift/cluster-kube-apiserver-operator/pkg/operator/certrotationcontroller.(*CertRotationController).syncExternalLoadBalancerHostnames(0xc000a980a0, 0x0, 0x0)
github.com/openshift/cluster-kube-apiserver-operator@/pkg/operator/certrotationcontroller/externalloadbalancer.go:24 +0x295

@a7vicky a7vicky merged commit 930a4b3 into master May 12, 2020
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.

1 participant