Skip to content

[AKS] refine creating TA binding commands#5572

Merged
zhoxing-ms merged 7 commits into
Azure:mainfrom
YitongFeng-git:yife/ta
Nov 24, 2022
Merged

[AKS] refine creating TA binding commands#5572
zhoxing-ms merged 7 commits into
Azure:mainfrom
YitongFeng-git:yife/ta

Conversation

@YitongFeng-git
Copy link
Copy Markdown
Contributor

@YitongFeng-git YitongFeng-git commented Nov 23, 2022


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.
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.

Tests

https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=64716041&view=logs&j=824f9585-43e9-59d0-7652-5251391c2ec5&t=281345dd-f8d7-5db6-4773-d5fdc691cb8d

@ghost ghost requested review from wangzelin007 and zhoxing-ms November 23, 2022 06:43
@ghost ghost assigned zhoxing-ms Nov 23, 2022
@ghost ghost added this to the Nov 2022 (2022-12-06) milestone Nov 23, 2022
@ghost ghost added the Auto-Assign Auto assign by bot label Nov 23, 2022
@ghost ghost requested a review from yonzhan November 23, 2022 06:43
@ghost ghost added the AKS label Nov 23, 2022
Copy link
Copy Markdown
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.

Queued a pipeline to test newly added test cases. It would also generate the recording files you'll need to commit to pass the CI if the test passed.

Comment thread src/aks-preview/azext_aks_preview/custom.py Outdated
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Nov 23, 2022

AKS

@YitongFeng-git YitongFeng-git force-pushed the yife/ta branch 6 times, most recently from 693038b to a95329e Compare November 24, 2022 03:04
Comment thread src/aks-preview/azext_aks_preview/tests/latest/test_aks_commands.py Outdated
Comment thread src/aks-preview/azext_aks_preview/tests/latest/test_aks_commands.py Outdated
Copy link
Copy Markdown
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

@zhoxing-ms zhoxing-ms merged commit 0344559 into Azure:main Nov 24, 2022
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.

4 participants