Skip to content

Update cli tools#3553

Merged
balopat merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:update-cli-tools
Jan 27, 2020
Merged

Update cli tools#3553
balopat merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:update-cli-tools

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Jan 22, 2020

This is a follow up of #3527

nicknovitski and others added 2 commits January 22, 2020 07:11
@nicknovitski @dgageot
Update CLI tools in docker image
9310225 
Helm purposely not updated due to breaking integration tests.

kubectl purposely only updated two major versions, to retain compatibility with
more clusters.
@dgageot
Update Dockerfile
917609f 
Signed-off-by: David Gageot <david@gageot.net>
@googlebot
Copy link
Copy Markdown

googlebot commented Jan 22, 2020

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@dgageot dgageot mentioned this pull request Jan 22, 2020
Closed
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2020 •
edited
Loading

Codecov Report

Merging #3553 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted Files Coverage Δ
...affold/kubernetes/portforward/kubectl_forwarder.go 65.85% <0%> (-2.44%) ⬇️

@dgageot
Copy link
Copy Markdown
Contributor Author

dgageot commented Jan 22, 2020

@nicknovitski can you give your consent that I use your commit?

@balopat balopat self-assigned this Jan 22, 2020
@dgageot
Copy link
Copy Markdown
Contributor Author

dgageot commented Jan 23, 2020

ping @nicknovitski

@nicknovitski
Copy link
Copy Markdown
Contributor

nicknovitski commented Jan 24, 2020

Ah, here it is. Yeah, I consent to having my commit used in this PR. Thanks!

@balopat balopat added cla: yes and removed cla: no labels Jan 27, 2020
@googlebot
Copy link
Copy Markdown

googlebot commented Jan 27, 2020

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@balopat balopat merged commit 2c048c2 into GoogleContainerTools:master Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nkubala nkubala Awaiting requested review from nkubala

@tejal29 tejal29 Awaiting requested review from tejal29

1 more reviewer

@balopat balopat balopat approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@balopat balopat

Labels

cla: yes pr/lgtm size/S

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dgageot @googlebot @nicknovitski @balopat