add privacy notice and command to set update check false#2774
Merged
add privacy notice and command to set update check false#2774
Conversation
|
Error creating deployment, please see controller logs for details. |
1 similar comment
|
Error creating deployment, please see controller logs for details. |
|
Error creating deployment, please see controller logs for details. |
balopat
suggested changes
Aug 30, 2019
|
Please visit http://35.236.69.194:1313 to view changes to the docs. |
Codecov Report
|
|
Please visit http://34.94.144.61:1313 to view changes to the docs. |
3c8f447 to
0e61652
Compare
balopat
approved these changes
Aug 31, 2019
Contributor
|
Lgtm, I bounced the Travis integration job - it flaked out on an rules_docker download! You are unlucky with CI:))) |
Contributor
Author
|
Thanks @balopat on following up! Have a great weekend! |
balopat
added a commit
that referenced
this pull request
Sep 4, 2019
add privacy notice and command to set update check false
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adding a config for turning off update check for skaffold
Test Notes:
GitTreeStateis non-dirty to by-pass this condition https://github.com/GoogleContainerTools/skaffold/blob/master///pkg/skaffold/update/update.go#L42You can test all scenarios in the unit test.