Skip to content

Automated cherry pick of #55259#56222

Merged
k8s-github-robot merged 1 commit intokubernetes:release-1.8from
justinsb:automated-cherry-pick-of-#55259-origin-release-1.8
Nov 27, 2017
Merged

Automated cherry pick of #55259#56222
k8s-github-robot merged 1 commit intokubernetes:release-1.8from
justinsb:automated-cherry-pick-of-#55259-origin-release-1.8

Conversation

@justinsb
Copy link
Copy Markdown
Member

Cherry pick of #55259 on release-1.8.

#55259: Tolerate partial discovery in garbage collector

Allow the garbage collector to tolerate partial discovery failures. On a
partial failure, use whatever was discovered, log the failures, and
allow the resync logic to try again later.

Fixes kubernetes#55022.
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 22, 2017
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 22, 2017
@k8s-github-robot k8s-github-robot added the do-not-merge/cherry-pick-not-approved Indicates that a PR is not yet approved to merge into a release branch. label Nov 22, 2017
@justinsb justinsb added the priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. label Nov 22, 2017
@justinsb
Copy link
Copy Markdown
Member Author

cc @jpbetz

@chrislovecnm
Copy link
Copy Markdown
Contributor

/lgtm

Who can approve this?

@thockin this needs priority

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 22, 2017
@janetkuo
Copy link
Copy Markdown
Member

/lgtm

@k8s-github-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: chrislovecnm, janetkuo, justinsb
We suggest the following additional approver: deads2k

Assign the PR to them by writing /assign @deads2k in a comment when ready.

Associated issue: 55259

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

@janetkuo
Copy link
Copy Markdown
Member

/assign @jpbetz would you cherrypick approve this patch? Thanks!

@enisoc enisoc added this to the v1.8 milestone Nov 22, 2017
@justinsb justinsb added kind/bug Categorizes issue or PR as related to a bug. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. labels Nov 22, 2017
@k8s-github-robot
Copy link
Copy Markdown

[MILESTONENOTIFIER] Milestone Pull Request Current

@chrislovecnm @erictune @janetkuo @jpbetz @justinsb

Pull Request Labels
  • sig/api-machinery: Pull Request will be escalated to these SIGs if needed.
  • priority/critical-urgent: Never automatically move pull request out of a release milestone; continually escalate to contributor and SIG through all available channels.
  • kind/bug: Fixes a bug discovered during the current release.
Help

@jpbetz jpbetz added approved Indicates a PR has been approved by an approver from all required OWNERS files. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. labels Nov 27, 2017
@k8s-github-robot
Copy link
Copy Markdown

/test all

Tests are more than 96 hours old. Re-running tests.

@k8s-github-robot k8s-github-robot removed the do-not-merge/cherry-pick-not-approved Indicates that a PR is not yet approved to merge into a release branch. label Nov 27, 2017
@k8s-github-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link
Copy Markdown

Automatic merge from submit-queue.

@k8s-github-robot k8s-github-robot merged commit e002d34 into kubernetes:release-1.8 Nov 27, 2017
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. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants