Skip to content

Remove deprecated klog flags from csi-addons containers#444

Merged
Madhu-1 merged 1 commit intoceph:mainfrom
iPraveenParihar:csi-addons-zap-log-flags
Apr 6, 2026
Merged

Remove deprecated klog flags from csi-addons containers#444
Madhu-1 merged 1 commit intoceph:mainfrom
iPraveenParihar:csi-addons-zap-log-flags

Conversation

@iPraveenParihar
Copy link
Copy Markdown
Contributor

Describe what this PR does

csi-addons has moved from klog to zap logging (see kubernetes-csi-addons#1006).
The --logtostderr and --alsologtostderr flags are now accepted as no-ops in the sidecar.
Remove these deprecated flags from the controller plugin and node plugin csi-addons container args.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

csi-addons has moved from klog to zap logging (see
kubernetes-csi-addons#1006). The --logtostderr and --alsologtostderr
flags are now accepted as no-ops in the sidecar. Remove these
deprecated flags from the controller plugin and node plugin
csi-addons container args.

Signed-off-by: Praveen M <m.praveen@ibm.com>
@iPraveenParihar iPraveenParihar marked this pull request as ready for review March 23, 2026 12:09
@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

This needs to be merged after we have a CSI-Addons release?
cc @Madhu-1

@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

/hold
We might need csi-addons release before ceph-csi-operator takes this change.

@Madhu-1 Madhu-1 merged commit 7067b01 into ceph:main Apr 6, 2026
17 checks passed
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.

3 participants