Skip to content

Conversation

sdake
Copy link
Member

@sdake sdake commented Aug 22, 2019

move from master-latest-release to release-1.3-latest-daily

Please provide a description for what this PR is for.

And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

move from master-latest-release to release-1.3-latest-daily
@sdake sdake requested a review from a team August 22, 2019 15:23
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Aug 22, 2019
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 22, 2019
@sdake sdake requested a review from howardjohn August 22, 2019 15:23
Makefile Outdated
@@ -27,9 +27,9 @@ ISTIO_DOCKER_HUB ?= docker.io/istio
export ISTIO_DOCKER_HUB
ISTIO_GCS ?= istio-release/releases/$(VERSION)
ISTIO_URL ?= https://storage.googleapis.com/$(ISTIO_GCS)
ISTIO_CNI_HUB ?= gcr.io/istio-release
ISTIO_CNI_HUB ?= gcr.io/istio-testing
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this does not contain the daily builds, so this will break things

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We publish daily builds to gcr.io/istio-release

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@howardjohn daily builids are published to istio-release only for cni, or for all repos? (my question is around the proxy push specifically).

@howardjohn howardjohn mentioned this pull request Aug 22, 2019
58 tasks
@sdake sdake dismissed howardjohn’s stale review August 23, 2019 09:48

stale review.

@istio-testing
Copy link
Collaborator

istio-testing commented Aug 23, 2019

@sdake: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
integ-new-install-k8s-presubmit-tests-release-1.3 4f831bb link /test integ-new-install-k8s-presubmit-tests-release-1.3

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.

@istio-testing istio-testing merged commit 261e06f into istio:release-1.3 Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants