Skip to content
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.

Sign up for GitHub

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

Jump to bottom

1.5 #94707

Closed
wants to merge 534 commits into from
Closed

1.5 #94707

wants to merge 534 commits into from

Conversation

yangcheng-dev
Copy link

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Kubernetes Submit Queue and others added 30 commits January 25, 2017 22:29
Merge pull request kubernetes#40498 from lavalamp/automated-cherry-pi…
0bd0b54 
…ck-of-#40497-upstream-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#40497

Cherry pick of kubernetes#40497 on release-1.5.

kubernetes#40497: Make HandleError prevent hot-loops
Merge pull request kubernetes#40501 from lavalamp/fix-rel-branch
19f83fc 
Automatic merge from submit-queue

Actually fix local-cluster-up on 1.5 branch

Fixes kubernetes#38847 (for real)
@lavalamp
Adjust global log limit to 1ms
e4f22b5
Merge pull request kubernetes#40553 from lavalamp/cp-40497
64029a2 
Automatic merge from submit-queue

Adjust global log limit to 1ms

Per further discussion, we adjusted the limit. Making this a separate PR since the CP already merged.
@0xmichalis
controller: old pods should block deployment completeness
b1882b6
@gnufied
Remove duplicate calls to describeInstance from aws
9903bd8 
This change removes all duplicate calls to describeInstance
from aws volume code path.
@divyenpatel
fix for vSphere DeleteVolume
74442bb
Merge pull request kubernetes#40191 from ericchiang/automated-cherry-…
9faa2a7 
…pick-of-#38167-upstream-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#38167

Cherry pick of kubernetes#38167 on release-1.5.

kubernetes#38167: rework oidc client auth provider
Merge pull request kubernetes#40685 from gnufied/automated-cherry-pic…
d77b7f6 
…k-of-#39842-upstream-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#39842

Cherry pick of kubernetes#39842 on release-1.5.

kubernetes#39842: Remove duplicate calls to describeInstance from aws
Merge pull request kubernetes#40625 from kargakis/cherry-pick-to-15
1cb54b5 
Automatic merge from submit-queue

Cherry-pick of "controller: old pods should block deployment completeness" to 1.5

@janetkuo the cherry-pick of kubernetes#40261 wasn't clean but I resolved any conflicts, saw the unit test pass, and additionally verified that it works for 1.5 as well as for 1.6 (though the unit test is enough). ptal
Merge pull request kubernetes#40693 from vmware/automated-cherry-pick…
8bbf2a9 
…-of-#40538-kubernetes-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#40538

Cherry pick of kubernetes#40538 on release-1.5.

kubernetes#40538: fix for vSphere DeleteVolume
Merge pull request kubernetes#40321 from zmerlynn/automated-cherry-pi…
92a0302 
…ck-of-#40094-upstream-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#40094

Cherry pick of kubernetes#40094 on release-1.5.

kubernetes#40094: Bump GCE to container-vm-v20170117
@piosz
Bumped version of addon resizer
a478a3c
@piosz
Bumped Heapster deployment version
f312683 
This change is required to make sure that addon manager will update it
fix GetVolumeInUse() function
2a5dcda 
Since we just want to get volume name info, each volume name just need to added once. desiredStateOfWorld.GetVolumesToMount() will return volume and pod binding info,
if one volume is mounted to several pods, the volume name will be return several times. That is not what we want in this function.
We can add a new function to only get the volume name info or judge whether the volume name is added to the desiredVolumesMap array.
@brendandburns
Fix merge, remove s390x
1c6cffc
fix copy.sh for osx
0c8dd97
update staging client
c60b997
Merge pull request kubernetes#40507 from brendandburns/automated-cher…
9827d09 
…ry-pick-of-#38919-upstream-release-1.5

Automatic merge from submit-queue

Add a KUBERNETES_NODE_* section to build kubelet/kube-proxy for windows

@saad-ali @luxas @pires
@codablock
Set NetworkInterfaces and Subnets to nil before updating Azure securi…
b44408d 
…ty groups
@codablock
Update go-autorest library to v7.2.3
1d59087 
To include Azure/go-autorest#107
Improves kubernetes#35180
@codablock
Enable lazy initialization of ext3/ext4 filesystems
09fbd99
@zmerlynn
Bump GCE to container-vm-v20170201
9ee80cf 
Addresses CVE-2016-9962
Merge pull request kubernetes#40791 from vwfs/automated-cherry-pick-o…
49a3cef 
…f-#40405-origin-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#40405

Cherry pick of kubernetes#40405 on release-1.5.

kubernetes#40405: Set NetworkInterfaces and Subnets to nil before updating
Merge pull request kubernetes#40792 from vwfs/automated-cherry-pick-o…
736800e 
…f-#40082-origin-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#40082

Cherry pick of kubernetes#40082 on release-1.5.

kubernetes#40082: Update go-autorest library to v7.2.3
Merge pull request kubernetes#40734 from NickrenREN/cherry-pick-1.5
cd2e2ff 
Automatic merge from submit-queue

Automated cherry pick of kubernetes#38443

Cherry pick of kubernetes#38443 on release-1.5.

kubernetes#38443: fix GetVolumeInUse() function
Merge pull request kubernetes#40301 from deads2k/rbac-42-clientcerts
d014455 
Automatic merge from submit-queue

present request header cert CA

Fixes kubernetes#40278

```release-note
Fixes request header authenticator by presenting the request header client CA so that the front proxy will authenticate using its client certificate.
```
Merge pull request kubernetes#40733 from piosz/nanny
b15f936 
Automatic merge from submit-queue

Bumped version of addon resizer

This is cherry-pick of kubernetes#40707. This PR is a low risk change, since it only reduces the verbosity of pod-nanny.
Merge pull request kubernetes#40228 from jessfraz/automated-cherry-pi…
41f9ee6 
…ck-of-#39831-origin-release-1.5

Automatic merge from submit-queue

Automated cherry pick of kubernetes#39831

Cherry pick of kubernetes#39831 on release-1.5.

kubernetes#39831: Check if error is Status in result.Stream()

```release-note
Check if error is Status in result.Stream()
```
@codablock
Set custom PollingDelay of 5 seconds for Azure VirtualMachinesClient
4f973c4 
The default polling delay of 1 minute results in very long delays when
an Azure Disk is attached to a node. It gets worse as go-autorest
doubles the default delay to 2 minutes.

Please see: kubernetes#35180 (comment)

Only the PollingDelay for VirtualMachinesClient is modified here to
avoid too much pressure on Azure quotas.
@k8s-ci-robot
Copy link
Contributor

@yangcheng-icbc: 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.

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/test-infra repository.

@k8s-ci-robot
Copy link
Contributor

@yangcheng-icbc: This cherry pick PR is for a release branch and has not yet been approved by Release Managers.
Adding the do-not-merge/cherry-pick-not-approved label.

To merge this cherry pick, it must first be approved (/lgtm + /approve) by the relevant OWNERS.

AFTER it has been approved by code owners, please ping the kubernetes/release-managers team in a comment to request a cherry pick review.

(For details on the patch release process and schedule, see the Patch Releases page.)

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/test-infra repository.

@k8s-ci-robot k8s-ci-robot added do-not-merge/cherry-pick-not-approved Indicates that a PR is not yet approved to merge into a release branch. 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 Sep 11, 2020
@k8s-ci-robot
Copy link
Contributor

Keywords which can automatically close issues and at(@) mentions are not allowed in commit messages.

The list of commits with invalid commit messages:

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/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. do-not-merge/blocked-paths Indicates that a PR should not merge because it touches files in blocked paths. labels Sep 11, 2020
@k8s-ci-robot
Copy link
Contributor

@yangcheng-icbc: Adding label: do-not-merge/blocked-paths because PR changes a protected file.

Reasons for blocking this PR:

[examples/ has moved to https://github.com/kubernetes/examples/]

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/test-infra repository.

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. needs-kind Indicates a PR lacks a `kind/foo` label and requires one. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Sep 11, 2020
@k8s-ci-robot
Copy link
Contributor

@yangcheng-icbc: PR needs rebase.

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/test-infra repository.

@k8s-ci-robot k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. needs-priority Indicates a PR lacks a `priority/foo` label and requires one. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Sep 11, 2020
@k8s-ci-robot
Copy link
Contributor

Hi @yangcheng-icbc. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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/test-infra repository.

@k8s-ci-robot
Copy link
Contributor

@yangcheng-icbc: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

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/test-infra repository.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/contains-merge-commits Indicates a PR which contains merge commits. label Sep 11, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: yangcheng-icbc
To complete the pull request process, please assign smarterclayton
You can assign the PR to them by writing /assign @smarterclayton in a comment when ready.

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

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
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

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/test-infra repository. I understand the commands that are listed here.

1 similar comment
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

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/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Sep 11, 2020
@neolit123
Copy link
Member

neolit123 commented Sep 13, 2020 •
edited

please re-open this PR once the rebase is fixed and description is added.
/close

@k8s-ci-robot
Copy link
Contributor

@neolit123: Closed this PR.

In response to this:

please re-open this PR once the rebase is fixed.
/close

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/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liggitt liggitt Awaiting requested review from liggitt

@msau42 msau42 Awaiting requested review from msau42

Assignees
No one assigned
Labels
cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. do-not-merge/blocked-paths Indicates that a PR should not merge because it touches files in blocked paths. do-not-merge/cherry-pick-not-approved Indicates that a PR is not yet approved to merge into a release branch. do-not-merge/contains-merge-commits Indicates a PR which contains merge commits. do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. needs-kind Indicates a PR lacks a `kind/foo` label and requires one. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. needs-priority Indicates a PR lacks a `priority/foo` label and requires one. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet