Skip to content

Commit

Permalink
Clean up OCP 3.11 configurations
Browse files Browse the repository at this point in the history
Signed-off-by: Dale Haiducek <[email protected]>
  • Loading branch information
dhaiducek authored and openshift-merge-bot[bot] committed Feb 14, 2025
1 parent a151d5a commit 8dbf62d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
5 changes: 3 additions & 2 deletions main.go
Original file line number Diff line number Diff line change
Expand Up @@ -214,10 +214,11 @@ func setupLogging() {

klogV := flag.CommandLine.Lookup("v")
if klogV != nil {
var klogLevel int
var klogLevel int64

klogLevel, err = strconv.Atoi(klogV.Value.String())
klogLevel, err = strconv.ParseInt(klogV.Value.String(), 10, 8)
if err != nil {
// #nosec G115 -- ParseInt() guarantees that it's an int8
klogConfig.Level = zap.NewAtomicLevelAt(zapcore.Level(int8(-1 * klogLevel)))
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ metadata:
addon.open-cluster-management.io/hosted-manifest-location: hosting
annotations:
{{- if eq .Values.hostingKubernetesDistribution "OpenShift" }}
# The alpha annotation is required for OpenShift 3.11
service.alpha.openshift.io/serving-cert-secret-name: {{ include "controller.fullname" . }}-metrics
service.beta.openshift.io/serving-cert-secret-name: {{ include "controller.fullname" . }}-metrics
{{- end }}
spec:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ metadata:
addon.open-cluster-management.io/hosted-manifest-location: hosting
annotations:
{{- if eq .Values.hostingKubernetesDistribution "OpenShift" }}
# The alpha annotation is required for OpenShift 3.11
service.alpha.openshift.io/serving-cert-secret-name: {{ include "controller.fullname" . }}-metrics
service.beta.openshift.io/serving-cert-secret-name: {{ include "controller.fullname" . }}-metrics
{{- end }}
spec:
Expand Down

0 comments on commit 8dbf62d

Please sign in to comment.