Skip to content
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

fix(mesh id): ensure mesh id is available to gateways #17708

Merged
merged 2 commits into from
Oct 9, 2019

Conversation

douglas-reid
Copy link
Contributor

It looks like the update to add the ISTIO_META_MESH_ID env var to sidecar injection template did not also update the gateways. This PR fixes this particular gap.

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

Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>
@douglas-reid douglas-reid requested a review from a team as a code owner October 8, 2019 22:40
@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 8, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 8, 2019
@douglas-reid
Copy link
Contributor Author

/test integ-telemetry-k8s-tests_istio

@howardjohn
Copy link
Member

/retest

@douglas-reid
Copy link
Contributor Author

/test integ-security-k8s-tests_istio

@istio-testing istio-testing merged commit 1068e98 into istio:master Oct 9, 2019
howardjohn added a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
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/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.

None yet

4 participants