azure-cli-extensions
azure-cli-extensions copied to clipboard
AZ AOSM Extension Refactor
- History.RST and setup.py up to date for release of 2.0.0b
- Large scale refactor of az aosm extension
- Logic reviewed by responsible team
- Good unit test coverage
- Working integration tests using ScenarioTest
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
General Guidelines
- [x] Have you run
azdev style <YOUR_EXT>locally? (pip install azdevrequired) - [x] Have you run
python scripts/ci/test_index.py -qlocally? (pip install wheel==0.30.0required) - [x] My extension version conforms to the Extension version schema
For new extensions:
- [ ] My extension description/summary conforms to the Extension Summary Guidelines.
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.
❌Azure CLI Extensions Breaking Change Test
❌aosm
rule cmd_name rule_message suggest_message ❌ 1007 - ParaRemove aosm nfd build cmd aosm nfd buildremoved parameterforceplease add back parameter forcefor cmdaosm nfd build❌ 1007 - ParaRemove aosm nfd build cmd aosm nfd buildremoved parameterinteractiveplease add back parameter interactivefor cmdaosm nfd build❌ 1007 - ParaRemove aosm nfd build cmd aosm nfd buildremoved parameterorder_paramsplease add back parameter order_paramsfor cmdaosm nfd build❌ 1002 - CmdRemove aosm nfd delete cmd aosm nfd deleteremovedplease confirm cmd aosm nfd deleteremoved❌ 1006 - ParaAdd aosm nfd publish cmd aosm nfd publishadded parameterbuild_output_folderplease remove parameter build_output_folderfor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parameterconfig_fileplease add back parameter config_filefor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parameterdefinition_fileplease add back parameter definition_filefor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parametermanifest_fileplease add back parameter manifest_filefor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parametermanifest_params_fileplease add back parameter manifest_params_filefor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parameterparameters_json_fileplease add back parameter parameters_json_filefor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nfd publish cmd aosm nfd publishremoved parameterskipplease add back parameter skipfor cmdaosm nfd publish❌ 1007 - ParaRemove aosm nsd build cmd aosm nsd buildremoved parameterforceplease add back parameter forcefor cmdaosm nsd build❌ 1002 - CmdRemove aosm nsd delete cmd aosm nsd deleteremovedplease confirm cmd aosm nsd deleteremoved❌ 1006 - ParaAdd aosm nsd publish cmd aosm nsd publishadded parameterbuild_output_folderplease remove parameter build_output_folderfor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parameterconfig_fileplease add back parameter config_filefor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parameterdesign_fileplease add back parameter design_filefor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parametermanifest_fileplease add back parameter manifest_filefor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parametermanifest_params_fileplease add back parameter manifest_params_filefor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parameterparameters_json_fileplease add back parameter parameters_json_filefor cmdaosm nsd publish❌ 1007 - ParaRemove aosm nsd publish cmd aosm nsd publishremoved parameterskipplease add back parameter skipfor cmdaosm nsd publish⚠️ 1006 - ParaAdd aosm nfd build cmd aosm nfd buildadded parameterskip⚠️ 1010 - ParaPropUpdate aosm nfd build cmd aosm nfd buildupdate parameterdefinition_type: updated propertychoicesfrom['cnf', 'vnf']to['cnf', 'vnf', 'vnf-nexus']⚠️ 1010 - ParaPropUpdate aosm nfd generate-config cmd aosm nfd generate-configupdate parameterdefinition_type: updated propertychoicesfrom['cnf', 'vnf']to['cnf', 'vnf', 'vnf-nexus']⚠️ 1009 - ParaPropRemove aosm nfd generate-config cmd aosm nfd generate-configupdate parameteroutput_file: removed propertydefault=input.json⚠️ 1010 - ParaPropUpdate aosm nfd publish cmd aosm nfd publishupdate parameterdefinition_type: updated propertychoicesfrom['cnf', 'vnf']to['cnf', 'vnf', 'vnf-nexus']⚠️ 1009 - ParaPropRemove aosm nsd generate-config cmd aosm nsd generate-configupdate parameteroutput_file: removed propertydefault=input.json⚠️ 1006 - ParaAdd aosm nsd publish cmd aosm nsd publishadded parameterno_subscription_permissions⚠️ 1011 - SubgroupAdd aosm publisher sub group aosm publisheradded
Hi @jordlay, 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.
Hi @jordlay, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.
Thank you for your contribution! We will review the pull request and get back to you soon.
@jordlay ,
- See https://github.com/Azure/azure-cli/blob/release/doc/extensions/versioning_guidelines.md . According to that the version we're releasing should be 2.0.0b1. It's a new preview version with breaking changes.
- The existing published version is still marked as preview - good :)
{ "experimental": false, "installed": true, "name": "aosm", "preview": true, "summary": "Microsoft Azure Command-Line Tools Aosm Extension", "version": "1.0.0b3" },
:warning: Release Suggestions
Module: aosm
- Update
VERSIONto1.0.0b4insrc/aosm/setup.py
Notes
- Stable/preview tag is inherited from last release. If needed, please add
stable/previewlabel to modify it. - Major/minor/patch/pre increment of version number is calculated by pull request code changes automatically. If needed, please add
major/minor/patch/prelabel to adjust it. - For more info about extension versioning, please refer to Extension version schema
@kairu-ms , we've added examples as requested, could you review again, please?
Please fix CI issues
/azp run
Azure Pipelines successfully started running 2 pipeline(s).
Thanks @kairu-ms . Do we need to do anything else for this to merge, or is there another button you need to press?
Thanks @kairu-ms . Do we need to do anything else for this to merge, or is there another button you need to press?
PR merged
[Release] Update index.json for extension [ aosm ] : https://dev.azure.com/azclitools/release/_build/results?buildId=159847&view=results