-
Notifications
You must be signed in to change notification settings - Fork 174
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
[1.18]Fix cloud-sdk:slim #1594
Merged
sergii-ssh
merged 2 commits into
GoogleCloudPlatform:staging-1.18
from
sergii-ssh:bump-1-18
Mar 11, 2024
Merged
[1.18]Fix cloud-sdk:slim #1594
sergii-ssh
merged 2 commits into
GoogleCloudPlatform:staging-1.18
from
sergii-ssh:bump-1-18
Mar 11, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wonjekang
approved these changes
Mar 11, 2024
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.
sergii-ssh
added a commit
that referenced
this pull request
Mar 11, 2024
* Merge main to staging-1.18 (#1494) * 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 <[email protected]> Co-authored-by: Clay <[email protected]> * Bump revision 1.18.4-asm.0 (#1505) * Populate MonitoringConfig values to GCP_METADATA (#1511) (#1512) * 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 (#1525) * Fix the issue for null value of monitoringConfig * Use fleet project ID for monitoringConfig * Organize kpt files even in offline mode (#1532) * Bump revision 1.18.5-asm.2 (#1536) * Detect old default ingress gateway earlier in installation flow (#1545) * [1.18] bump revision (#1548) * Bump revision 1.18.7-asm.0 (#1561) * Bump revision 1.18.7-asm.0 * Disable SC2317 since we use trap * Bump revision 1.18.7-asm.5 (#1571) * Bump revision 1.18.7-asm.5 * Update google-cloud packages * Update revision 1.18.7-asm.4 (#1581) * Revision 1.18.7-asm.11 (#1589) * [1.18]Fix cloud-sdk:slim (#1594) * Fix cloud-sdk:slim * remove whitespace --------- Co-authored-by: Wonje Kang <[email protected]> Co-authored-by: Clay <[email protected]>
sergii-ssh
added a commit
that referenced
this pull request
Apr 24, 2024
* Merge main to staging-1.18 (#1494) * 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 <[email protected]> Co-authored-by: Clay <[email protected]> * Bump revision 1.18.4-asm.0 (#1505) * Populate MonitoringConfig values to GCP_METADATA (#1511) (#1512) * 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 (#1525) * Fix the issue for null value of monitoringConfig * Use fleet project ID for monitoringConfig * Organize kpt files even in offline mode (#1532) * Bump revision 1.18.5-asm.2 (#1536) * Detect old default ingress gateway earlier in installation flow (#1545) * [1.18] bump revision (#1548) * Bump revision 1.18.7-asm.0 (#1561) * Bump revision 1.18.7-asm.0 * Disable SC2317 since we use trap * Bump revision 1.18.7-asm.5 (#1571) * Bump revision 1.18.7-asm.5 * Update google-cloud packages * Update revision 1.18.7-asm.4 (#1581) * Revision 1.18.7-asm.11 (#1589) * [1.18]Fix cloud-sdk:slim (#1594) * Fix cloud-sdk:slim * remove whitespace * Bump revision 1.18.7-asm.21 --------- Co-authored-by: Wonje Kang <[email protected]> Co-authored-by: Clay <[email protected]>
sergii-ssh
added a commit
that referenced
this pull request
Jun 5, 2024
* Merge main to staging-1.18 (#1494) * 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 <[email protected]> Co-authored-by: Clay <[email protected]> * Bump revision 1.18.4-asm.0 (#1505) * Populate MonitoringConfig values to GCP_METADATA (#1511) (#1512) * 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 (#1525) * Fix the issue for null value of monitoringConfig * Use fleet project ID for monitoringConfig * Organize kpt files even in offline mode (#1532) * Bump revision 1.18.5-asm.2 (#1536) * Detect old default ingress gateway earlier in installation flow (#1545) * [1.18] bump revision (#1548) * Bump revision 1.18.7-asm.0 (#1561) * Bump revision 1.18.7-asm.0 * Disable SC2317 since we use trap * Bump revision 1.18.7-asm.5 (#1571) * Bump revision 1.18.7-asm.5 * Update google-cloud packages * Update revision 1.18.7-asm.4 (#1581) * Revision 1.18.7-asm.11 (#1589) * [1.18]Fix cloud-sdk:slim (#1594) * Fix cloud-sdk:slim * remove whitespace * Bump revision 1.18.7-asm.21 * Bump rev 1.18.7-asm.26 --------- Co-authored-by: Wonje Kang <[email protected]> Co-authored-by: Clay <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding
--allow-releaseinfo-change
flag as per GoogleCloudPlatform/cloud-sdk-docker#378