Skip to content

The current global hub and tag defaults are only for dev#5131

Merged
istio-testing merged 1 commit intorelease-1.3from
elfinhe-patch-2
Oct 11, 2019
Merged

The current global hub and tag defaults are only for dev#5131
istio-testing merged 1 commit intorelease-1.3from
elfinhe-patch-2

Conversation

@elfinhe
Copy link
Copy Markdown
Member

@elfinhe elfinhe commented Oct 11, 2019

In release this is changed to below:

global:
  # Default hub for Istio images.
  # Releases are published to docker hub under 'istio' project.
  # Daily builds from prow are on gcr.io
  hub: docker.io/istio

  # Default tag for Istio images.
  tag: 1.3.2

Not sure how to fix, but having the dev defaults here may be misleading.

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

In release this is changed to below:
```yaml
global:
  # Default hub for Istio images.
  # Releases are published to docker hub under 'istio' project.
  # Daily builds from prow are on gcr.io
  hub: docker.io/istio

  # Default tag for Istio images.
  tag: 1.3.2
```
Not sure how to fix, but having the dev defaults here may be misleading.
@elfinhe elfinhe requested a review from a team as a code owner October 11, 2019 16:46
@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 Oct 11, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 11, 2019
@istio-testing istio-testing merged commit b6dfa8d into release-1.3 Oct 11, 2019
@istio-testing istio-testing deleted the elfinhe-patch-2 branch October 11, 2019 21:04
@elfinhe elfinhe added the cherrypick/master Set this label on a PR to auto-merge from a release branch to master label Oct 11, 2019
@elfinhe
Copy link
Copy Markdown
Member Author

elfinhe commented Oct 17, 2019

/cherrypick master

@istio-testing
Copy link
Copy Markdown
Contributor

@elfinhe: new pull request created: #5172

Details

In response to this:

/cherrypick master

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.

charili pushed a commit to charili/istio.io that referenced this pull request Nov 10, 2019
In release this is changed to below:
```yaml
global:
  # Default hub for Istio images.
  # Releases are published to docker hub under 'istio' project.
  # Daily builds from prow are on gcr.io
  hub: docker.io/istio

  # Default tag for Istio images.
  tag: 1.3.2
```
Not sure how to fix, but having the dev defaults here may be misleading.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherrypick/master Set this label on a PR to auto-merge from a release branch to master cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants