Skip to content

UPSTREAM: 57422: Rework method of updating atomic-updated data volumes#18165

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
joelsmith:pick-57422
Jan 19, 2018
Merged

UPSTREAM: 57422: Rework method of updating atomic-updated data volumes#18165
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
joelsmith:pick-57422

Conversation

@joelsmith
Copy link
Contributor

What this PR does / why we need it:

This is a backport of kubernetes/kubernetes#57422

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 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.

Which issue(s) this PR fixes
This addresses https://bugzilla.redhat.com/show_bug.cgi?id=1516569 and https://bugzilla.redhat.com/show_bug.cgi?id=1430322

Special notes for your reviewer:

Release note:

Allow volumes to be mounted beneath secret volumes [1430322] [1516569]

@sjenning
Copy link
Contributor

flake #17974 and yum mirror failure
/retest

@joelsmith joelsmith added the kind/bug Categorizes issue or PR as related to a bug. label Jan 18, 2018
@jsafrane
Copy link
Contributor

/assign

@jsafrane
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 19, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: joelsmith, jsafrane

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

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 19, 2018
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 18129, 18152, 17403, 18020, 18165).

@openshift-merge-robot openshift-merge-robot merged commit 91d864d into openshift:master Jan 19, 2018
@joelsmith joelsmith deleted the pick-57422 branch January 19, 2018 15:32
openshift-merge-robot added a commit that referenced this pull request Jan 21, 2018
Automatic merge from submit-queue.

[3.7] UPSTREAM: 57422: Rework method of updating atomic-updated data volumes

See #18165 for background info.
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. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. vendor-update Touching vendor dir or related files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants