Skip to content

add privacy notice and command to set update check false#2774

Merged
balopat merged 4 commits intomasterfrom
add_privacy_policy
Aug 31, 2019
Merged

add privacy notice and command to set update check false#2774
balopat merged 4 commits intomasterfrom
add_privacy_policy

Conversation

@tejal29
Copy link
Copy Markdown
Contributor

@tejal29 tejal29 commented Aug 30, 2019

Adding a config for turning off update check for skaffold

Test Notes:

  1. Build a non dirty version with version set to < 0.37.0
GOOS=linux GOARCH=amd64 CGO_ENABLED=1 go install -tags "osusergo netgo static_build" -ldflags " -X github.com/GoogleContainerTools/skaffold/pkg/skaffold/version.version=v0.30.0 -X github.com/GoogleContainerTools/skaffold/pkg/skaffold/version.buildDate=2019-08-30T10:29:55Z -X github.com/GoogleContainerTools/skaffold/pkg/skaffold/version.gitCommit=13a5d595d7bcc760d00460e80dd78fa0f676d1ac -X github.com/GoogleContainerTools/skaffold/pkg/skaffold/version.gitTreeState=dirty  -extldflags \"-static\"" -gcflags "all=-trimpath=/usr/local/google/home/tejaldesai/workspace/skaffold1" -asmflags "all=-trimpath=/usr/local/google/home/tejaldesai/workspace/skaffold1" github.com/GoogleContainerTools/skaffold/cmd/skaffold
  1. Verify skaffold version is < 0.37.0 and GitTreeState is non-dirty to by-pass this condition https://github.com/GoogleContainerTools/skaffold/blob/master///pkg/skaffold/update/update.go#L42
$ skaffold version -v=debug
INFO[0000] Skaffold &{Version:t0.40.0 ConfigVersion:skaffold/v1beta14 GitVersion: GitCommit:13a5d595d7bcc760d00460e80dd78fa0f676d1ac GitTreeState:test BuildDate:2019-08-30T10:29:55Z GoVersion:go1.12.5 Compiler:gc Platform:linux/amd64} 
v0.30.0
  1. run skaffold deploy in examples/getting-started and verify we see the message for newer version
$ cd examples/geting-started
$ skaffold deploy
Starting deploy...
kubectl client version: 1.11+
kubectl version 1.12.0 or greater is recommended for use with Skaffold
 - pod/getting-started configured
Deploy complete in 949.223543ms
There is a new version (0.37.0) of Skaffold available. Download it at https://storage.googleapis.com/skaffold/releases/latest/skaffold-linux-amd64
  1. Now disable the update-check config via command line
$skaffold config set -g update-check false
set global value update-check to false
  1. Run skaffold deploy again and make sure there is no message for newer skaffold version.
tejaldesai@@getting-started (add_privacy_policy)$ skaffold deploy 
Starting deploy...
kubectl client version: 1.11+
kubectl version 1.12.0 or greater is recommended for use with Skaffold
 - pod/getting-started configured
Deploy complete in 1.170797454s

You can test all scenarios in the unit test.

@tejal29 tejal29 added meta/docs-preview docs-modifications runs the docs preview service on the given PR labels Aug 30, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment, please see controller logs for details.

1 similar comment
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment, please see controller logs for details.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Aug 30, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment, please see controller logs for details.

@balopat balopat added docs-modifications runs the docs preview service on the given PR and removed meta/docs-preview labels Aug 30, 2019
@container-tools-bot
Copy link
Copy Markdown

Please visit http://35.236.69.194:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Aug 30, 2019
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 30, 2019

Codecov Report

Merging #2774 into master will increase coverage by 0.02%.
The diff coverage is 88.88%.

Impacted Files Coverage Δ
pkg/skaffold/update/update.go 34.61% <100%> (+8.52%) ⬆️
pkg/skaffold/config/util.go 65.47% <100%> (+2.18%) ⬆️
cmd/skaffold/app/cmd/flags.go 100% <100%> (ø) ⬆️
cmd/skaffold/app/cmd/cmd.go 68% <33.33%> (-0.26%) ⬇️

@balopat balopat added docs-modifications runs the docs preview service on the given PR and removed meta/docs-preview labels Aug 30, 2019
@container-tools-bot
Copy link
Copy Markdown

Please visit http://34.94.144.61:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Aug 30, 2019
@tejal29 tejal29 force-pushed the add_privacy_policy branch from 3c8f447 to 0e61652 Compare August 30, 2019 23:36
@balopat
Copy link
Copy Markdown
Contributor

balopat commented Aug 31, 2019

Lgtm, I bounced the Travis integration job - it flaked out on an rules_docker download! You are unlucky with CI:)))

@balopat balopat merged commit 4eb250b into master Aug 31, 2019
@tejal29
Copy link
Copy Markdown
Contributor Author

tejal29 commented Aug 31, 2019

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
@dgageot dgageot deleted the add_privacy_policy branch September 9, 2019 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants