-
Notifications
You must be signed in to change notification settings - Fork 38.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[1.9] Automated cherry pick of #57422: Rework method of updating atomic-updated data volumes #58458
[1.9] Automated cherry pick of #57422: Rework method of updating atomic-updated data volumes #58458
Conversation
This change affects the way that secret, configmap, downwardAPI and projected volumes (which all use the same underlying code) implement their data update functionality. * Instead of creating a subdirectory hierarchy that itself will contain symlinks to each actual data file, create only symlinks to items in the root of the volume, whether they be files or directories. * Rather than comparing the user-visible data directory to see if an update is needed, compare with the current version of the data directory. * Fix data dir timestamp format year * Create ..data symlink even when a data volume has no data so consumers can have simplified update watch logic.
/test pull-kubernetes-e2e-kops-aws |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: joelsmith, jsafrane Associated issue: #57422 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/assign @mbohlool |
/kind bug |
@jsafrane can you apply |
@joelsmith can you add a more user facing release not to the main PR? something like:
|
@mbohlool done, thanks for the suggestion. |
[MILESTONENOTIFIER] Milestone Removed From Pull Request @joelsmith @jsafrane @mbohlool @kubernetes/sig-node-misc @kubernetes/sig-storage-misc Important: This pull request was missing the |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
Cherry pick of #57422 on release-1.9.
#57422: Rework method of updating atomic-updated data volumes
Fixes #60814 for 1.9