Skip to content

{AKS} Fix aks byo appgw test case#4894

Merged
zhoxing-ms merged 5 commits intoAzure:mainfrom
FumingZhang:fuming/fix-appgw-0525
May 25, 2022
Merged

{AKS} Fix aks byo appgw test case#4894
zhoxing-ms merged 5 commits intoAzure:mainfrom
FumingZhang:fuming/fix-appgw-0525

Conversation

@FumingZhang
Copy link
Copy Markdown
Member


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

Fix aks byo appgw test case test_aks_byo_appgw_with_ingress_appgw_addon. See sibling PR #22608 in azure-cli.

Related command

network application-gateway create

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.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@ghost ghost requested review from wangzelin007 and zhoxing-ms May 25, 2022 08:11
@ghost ghost assigned zhoxing-ms May 25, 2022
@ghost ghost added this to the Jun 2022 (2022-07-05) milestone May 25, 2022
@ghost ghost added the Auto-Assign Auto assign by bot label May 25, 2022
@ghost ghost requested a review from yonzhan May 25, 2022 08:11
@ghost ghost added the AKS label May 25, 2022
@FumingZhang
Copy link
Copy Markdown
Member Author

FumingZhang commented May 25, 2022

Queued a pipeline to test this case. The case passed.

@FumingZhang FumingZhang marked this pull request as ready for review May 25, 2022 08:32
@zhoxing-ms zhoxing-ms merged commit e102839 into Azure:main May 25, 2022
@FumingZhang FumingZhang deleted the fuming/fix-appgw-0525 branch June 13, 2022 07:33
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.

2 participants