Skip to content

Commit c9a709a

Browse files
committed
Remove obsolete glog/klog flags from webhook manifests
The webhook code was migrated from glog/klog to Zap logging (via controller-runtime), but the deployment manifests were not updated to reflect this change. This was causing the webhook to fail with "unknown flag: --alsologtostderr" error when the container started, as these flags are no longer supported by the Zap-based logging implementation. Changes: - Removed --alsologtostderr and --v=3 flags from operator-webhook/server.yaml - Removed --logtostderr and --alsologtostderr flags from webhook/server.yaml - Added --zap-log-level=3 to operator-webhook for equivalent verbosity The network-resources-injector webhook (webhook/server.yaml) now uses default Zap logging settings which are appropriate for production use. Signed-off-by: Sebastian Sch <sebassch@gmail.com>
1 parent c063f68 commit c9a709a

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

bindata/manifests/operator-webhook/server.yaml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ spec:
7171
- "--port=6443"
7272
- "--tls-private-key-file=/etc/tls/tls.key"
7373
- "--tls-cert-file=/etc/tls/tls.crt"
74-
- "--alsologtostderr=true"
75-
- "--v=3"
74+
- "--zap-log-level=3"
7675
env:
7776
- name: NAMESPACE
7877
valueFrom:

bindata/manifests/webhook/server.yaml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,6 @@ spec:
7575
- -tls-private-key-file=/etc/tls/tls.key
7676
- -tls-cert-file=/etc/tls/tls.crt
7777
- -insecure=true
78-
- -logtostderr=true
79-
- -alsologtostderr=true
8078
env:
8179
- name: NAMESPACE
8280
valueFrom:

0 commit comments

Comments
 (0)