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.
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
Update fortio version to latest #2596
Conversation
Not clear why Gopkg.lock changed so much
@ldemailly PR needs rebase |
# Conflicts: # Gopkg.lock
istio.VERSION
Outdated
@@ -13,6 +13,6 @@ export AUTH_DEBIAN_URL="https://storage.googleapis.com/istio-artifacts/auth/d10e | |||
export PILOT_DEBIAN_URL="https://storage.googleapis.com/istio-artifacts/pilot/3101ea9d82a5f83b699c2d3245b371a19fa6bef4/artifacts/debs" | |||
export PROXY_DEBIAN_URL="https://storage.googleapis.com/istio-artifacts/proxy/2458e4e27bc588df858711b66de0970183e0cf66/artifacts/debs" | |||
export FORTIO_HUB="docker.io/istio" | |||
export FORTIO_TAG="0.4.2" | |||
export FORTIO_TAG="0.6.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need this, and for what ?
(this goes for the entire file)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's used in e2e still afaik, for instance:
tests/e2e/tests/simple/simple1_test.go: tag := os.Getenv("FORTIO_TAG")
and in updateVersion
# Conflicts: # Gopkg.lock
Tried ``` dep ensure --update istio.io/fortio Warning: Gopkg.lock is out of sync with Gopkg.toml or the project's imports. ``` But it did change everything in the lock file anyway
@ldemailly PR needs rebase |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: douglas-reid The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/test all [submit-queue is verifying that this PR is safe to merge] |
/test istio-presubmit |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
Not clear why Gopkg.lock changed so much