Skip to content

fix: correct storageclass annotations and node dnsPolicy in latest chart#1103

Merged
andyzhangx merged 1 commit intokubernetes-csi:release-4.13from
andyzhangx:fix-latest-chart-issues-4.13
Apr 19, 2026
Merged

fix: correct storageclass annotations and node dnsPolicy in latest chart#1103
andyzhangx merged 1 commit intokubernetes-csi:release-4.13from
andyzhangx:fix-latest-chart-issues-4.13

Conversation

@andyzhangx
Copy link
Copy Markdown
Member

What type of PR is this?
/kind bug

What this PR does / why we need it:
Cherry-pick of #1102 for release-4.13 branch.

  1. storageclass.yaml: annotations: key was rendered outside the with block, causing annotations: null when .Values.storageClass.annotations is unset
  2. csi-nfs-node.yaml: node DaemonSet dnsPolicy was using .Values.controller.dnsPolicy instead of .Values.node.dnsPolicy

Changes:

  • storageclass.yaml: move annotations: inside the with block
  • csi-nfs-node.yaml: use .Values.node.dnsPolicy
  • Regenerated helm chart tgz

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Apr 17, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andyzhangx

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 17, 2026
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 17, 2026
@andyzhangx andyzhangx removed the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Apr 17, 2026
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 18, 2026
Cherry-pick of PR kubernetes-csi#1102 for release-4.13 branch.

- Move annotations key inside with block to avoid rendering null annotations
- Fix node DaemonSet dnsPolicy to use node.dnsPolicy instead of controller.dnsPolicy
@andyzhangx andyzhangx force-pushed the fix-latest-chart-issues-4.13 branch from b902b9f to d40b8e4 Compare April 19, 2026 01:19
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 19, 2026
@andyzhangx andyzhangx merged commit 64071c5 into kubernetes-csi:release-4.13 Apr 19, 2026
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants