Patch gateway hosts
Package Merge Request
Package Changes
Updates the istio global values, secret-tls.yaml template, gateway.yaml template, and values.yaml template files in chart/templates/istio/controlplane/ to support using two different TLS certificate secrets for the same Gateway resource.
Package MR
N/A
For Issue
Closes #1068 (closed)
BB Processes
Add labels for affected packages so that they are deployed in CI as well as a status label:
Be sure to assign to yourself:
Once it is ready for review switch the status and assign reviewers:
Merge request reports
Activity
added Big Bang Core istio labels
assigned to @seagren.tim
- Resolved by Tim Seagren
Since we'd be deprecating the old way, add a conditional in NOTES.txt to spit out a deprecation warning if the old way is used. I think you'll have to use range there to iterate over each gateway and spit out the warning for each gateway using the deprecated method.
- Last reply by Tim Seagren
mentioned in issue #1068 (closed)
added 1 commit
- a662f498 - updating values.yaml to support old method by default, updating...
added 1 commit
- bf2fde61 - updating defaults to not create nonsense ports
-
@michaelmcleroy changes have been made, whenever you get a second
added statusreview label
@micah.nagel, @BrandenCobb, and @ryan.j.garcia could one of you please take a look at this MR?
added 95 commits
-
bf2fde61...000913d0 - 94 commits from branch
master - e3522acd - Merge branch 'master' into patch-gateway-hosts
-
bf2fde61...000913d0 - 94 commits from branch
added 1 commit
- 36093bcf - replacing the missing TLS certificate for the default case
changed milestone to %1.29.0
mentioned in commit 7a7aed7a