-
Notifications
You must be signed in to change notification settings - Fork 161
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
Merge staging-1.18 to release-1.18 #1577
Conversation
* Bump revision 1.18.2-asm.0 (#1481) * Update release script (#1483) * Use top level `istioctl` commands for `create-remote-secret` and `tag` (#1484) Patches for istio/istio#46051 and istio/istio#46257 * Revert Off-GCP metadata population (#1485) * Add --working-dir alias for output directory * Add --working-dir alias for output directory * Update test case to utilize --working-dir alist for output dir * Populate GCP_METADATA by membership monitoring config * Add local variables for lint * Address line-variable error * Revert "Populate GCP_METADATA by membership monitoring config" This reverts commit b3372bd. * Do not support channel selection via CLI args (#1488) Allowing this gives the wrong impression that GKE and ASM channels are independent, which is useless at best and dangerous at worst. We shouldn't be making it easier for customers to run nonstandard environments. * Update verbose help text for consistency with public docs (#1487) * Support the regionalized membership name parsing (#1490) * Revert "Do not support channel selection via CLI args (#1488)" This reverts commit 8c09b8f. Will re-commit when the channel labeling issue is solved a bit cleaner. * Bump revision 1.18.2-asm.4 (#1493) --------- Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com> Co-authored-by: Clay <ctpence@google.com>
* Populate MonitoringConfig values to GCP_METADATA (#1511) * Populate MonitoringConfig values to GCP_METADATA (#1498) * Populate MonitoringConfig values to GCP_METADATA * Add quotes to fix build lint error * Add quotes to fix build lint error * Resolve minor problem and code optimization * Add double-quotes for lint * Update lint changes * Update the logic to identify empty monitoring config * Update the logic to identify empty monitoring config * Remove unnecessary empty line
* Fix the issue for null value of monitoringConfig * Use fleet project ID for monitoringConfig
* Bump revision 1.18.7-asm.0 * Disable SC2317 since we use trap
* Bump revision 1.18.7-asm.5 * Update google-cloud packages
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
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
No description provided.