Skip to content

Add enable ASM egress gateway commands for az aks mesh#6768

Merged
yanzhudd merged 20 commits intoAzure:mainfrom
nshankar13:nshankar/add-asm-egress-cli-commands
Sep 19, 2023
Merged

Add enable ASM egress gateway commands for az aks mesh#6768
yanzhudd merged 20 commits intoAzure:mainfrom
nshankar13:nshankar/add-asm-egress-cli-commands

Conversation

@nshankar13
Copy link
Contributor


This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

nshankar added 2 commits September 15, 2023 10:21
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Sep 15, 2023

⚠️Azure CLI Extensions Breaking Change Test
⚠️aks-preview
rule cmd_name rule_message suggest_message
⚠️ 1001 - CmdAdd aks mesh disable-egress-gateway cmd aks mesh disable-egress-gateway added
⚠️ 1001 - CmdAdd aks mesh enable-egress-gateway cmd aks mesh enable-egress-gateway added

@azure-client-tools-bot-prd
Copy link

Hi @nshankar13,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@yonzhan
Copy link
Collaborator

yonzhan commented Sep 15, 2023

Thank you for your contribution! We will review the pull request and get back to you soon.

nshankar added 15 commits September 15, 2023 15:39
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
nshankar added 2 commits September 16, 2023 10:59
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
@nshankar13
Copy link
Contributor Author

@andyliuliming @FumingZhang @zhoxing-ms @yanzhudd @yonzhan need some urgent reviews on this as we are trying to meet 9/19 deadline for CLI changes and to get this into the GA API changes in azure-cli/acs as well.


@AllowLargeResponse()
@AKSCustomResourceGroupPreparer(random_name_length=17, name_prefix='clitest', location='westus2')
def test_aks_azure_service_mesh_with_egress_gateway(self, resource_group, resource_group_location):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Queued live test to validate the change.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test passed

Copy link
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: nshankar <nshankar@microsoft.ghe.com>
@yanzhudd
Copy link
Contributor

It is suggested to add some details of changes in #6768 (comment)

@yanzhudd
Copy link
Contributor

Please add change logs in HISTORY.rst. If you want to release a new version, please change version in setup.py file.

@nshankar13
Copy link
Contributor Author

nshankar13 commented Sep 18, 2023

@FumingZhang @yanzhudd updated HISTORY.rst and setup.py.

@yanzhudd yanzhudd merged commit 7127433 into Azure:main Sep 19, 2023
@azclibot
Copy link
Collaborator

[Release] Update index.json for extension [ aks-preview ] : https://dev.azure.com/azclitools/internal/_build/results?buildId=91341&view=results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AKS Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants