-
Notifications
You must be signed in to change notification settings - Fork 7.9k
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
Configure mesh.sidecarToTelemetrySessionAffinity with existing helm option #16863
Conversation
…ssionAffinityEnabled is specified
Do we also need to set a pilot variable ? |
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.
/lgtm
pilot env variable is 1.1 only, so no need to set anything for pilot.
/test istio-unit-tests-master |
/test istio-racetest-master |
/test integ-security-k8s-presubmit-tests-master |
In response to a cherrypick label: #16863 failed to apply on top of branch "release-1.2":
|
In response to a cherrypick label: new pull request created: #16897 |
* istio#16223 * istio#16272 * istio#16187 * istio#16466 * istio#16634 * istio#16594 * istio#16666 * istio#16483 * istio#16820 * istio#16842 * istio#16852 * istio#16835 * istio#16863 * istio#16892 * istio#16991 * istio#16957 * istio#17013 * istio#17134 * istio#17155 * istio#17235 * istio#17342 * istio#17477 * istio#17615 * istio#17334 * istio#17708 * istio#17737 * Fix injection template * Fix quoting * Fix test values * Add accidentally deleted affinity
helm option
Values.mixer.telemetry.sessionAffinityEnabled
has existed for some time. It did not turn on the mesh config flag before. This PR turns the meshconfig flag ifValues.mixer.telemetry.sessionAffinityEnabled
is specified.fixes #16862