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

istioctl: remove create-remote-secret and remote-clusters in experimental #46051

Merged
merged 2 commits into from Jul 19, 2023

Conversation

hanxiaop
Copy link
Member

@hanxiaop hanxiaop commented Jul 18, 2023

Please provide a description of this PR:
They have been moved to istioctl almost two years ago, and I just submitted istio/istio.io#13589 to update the doc instructions.

@hanxiaop hanxiaop requested a review from a team as a code owner July 18, 2023 07:12
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 18, 2023
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 18, 2023
@istio-testing istio-testing merged commit 62238c3 into istio:master Jul 19, 2023
27 checks passed
sergii-ssh added a commit to sergii-ssh/anthos-service-mesh-packages that referenced this pull request Aug 10, 2023
zerobfd pushed a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Aug 10, 2023
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Sep 5, 2023
* 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>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Sep 6, 2023
* 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>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Oct 11, 2023
* 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* Bump revision 1.18.4-asm.0 (#1505)

---------

Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Nov 14, 2023
* 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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)

---------

Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Dec 12, 2023
* 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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)

---------

Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Jan 16, 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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

---------

Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages that referenced this pull request Feb 9, 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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)

---------

Co-authored-by: Wonje Kang <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
sergii-ssh added a commit to GoogleCloudPlatform/anthos-service-mesh-packages 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>

* 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 <96211823+wonjekang@users.noreply.github.com>
Co-authored-by: Clay <ctpence@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants