-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[SCVMM] Fix VMM Server Connect command 1.1.1 release #7841
[SCVMM] Fix VMM Server Connect command 1.1.1 release #7841
Conversation
️✔️Azure CLI Extensions Breaking Change Test
|
Hi @hsurana06, |
Hi @hsurana06, |
SCVMM |
Thanks for the approval on the az cli PR #7778 yesterday. Added the fix for the failures observed with SCVMM cli v1.1.0 release Looking for a hotfix to get this healthy. |
@kairu-ms - I have updated the release version and history.rst for changelog. Can you help merge this PR and what would be ETA for releasing this as a hotfix once PR is merged? |
[Release] Update index.json for extension [ scvmm ] : https://dev.azure.com/azclitools/release/_build/results?buildId=177941&view=results |
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az scvmm vmmserver connect - There is a fix needed in passing the vmmServerProperties correctly in the body. All calls are facing due to this properties API validation failure.
General Guidelines
azdev style <YOUR_EXT>
locally? (pip install azdev
required)python scripts/ci/test_index.py -q
locally? (pip install wheel==0.30.0
required)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
.