Adding enable ultra ssd, eviction policy, and spot max price support for machine#9696
Conversation
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| aks machine add | cmd aks machine add added parameter enable_ultra_ssd |
||
| aks machine add | cmd aks machine add added parameter eviction_policy |
||
| aks machine add | cmd aks machine add added parameter spot_max_price |
|
Hi @erinborders, |
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
|
Hi @erinborders Release SuggestionsModule: aks-preview
Notes
|
CodeGen Tools Feedback CollectionThank you for using our CodeGen tool. We value your feedback, and we would like to know how we can improve our product. Please take a few minutes to fill our codegen survey |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
FumingZhang
left a comment
There was a problem hiding this comment.
Could you add some unit tests? It would also be helpful to include a scenario test (e2e test) in test_aks_commands.py.
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
…sions into erinborders/machine-preview
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
finally got the test to pass and added the recording file to the PR |
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
src/aks-preview/HISTORY.rst
Outdated
| +++++++ | ||
| * `az aks create/update`: Add `--enable-app-routing-istio` / `--disable-app-routing-istio` (short: `--enable-ari` / `--disable-ari`) flags to enable or disable Istio as a Gateway API implementation for App Routing. | ||
| * `az aks approuting gateway istio enable/disable`: Add new subcommands to enable or disable the Istio Gateway API implementation for App Routing on an existing cluster. | ||
| * `az aks machine add`: Add `--spot-max-price` flag support to set the max price (in US Dollars) you are willing to pay for spot instances on a machine. |
There was a problem hiding this comment.
Please move your newly added history notes to the Pending section or create a new version tag and update the value of VERSION in setup.py accordingly.
| operation_group="machines" | ||
| ) | ||
| machineBillingProfile = MachineBillingProfile( | ||
| spot_max_price=spot_max_price |
There was a problem hiding this comment.
There is a normalization of NaN in the AP handling logic. Since the machine is a new feature, it may not yet have comprehensive unit and scenario tests to ensure that NaN values are correctly passed to RP and handled appropriately.
…sions into erinborders/machine-preview
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
src/aks-preview/HISTORY.rst
Outdated
| * `az aks machine add`: Add `--spot-max-price` flag support to set the max price (in US Dollars) you are willing to pay for spot instances on a machine. | ||
| * `az aks machine add`: Add `--eviction-policy` flag support to set the eviction policy for a machine. | ||
| * `az aks machine add`: Add `--enable-ultra-ssd` flag support to enable ultra ssd on a machine. |
There was a problem hiding this comment.
Please do not add new history notes to a released version. Either choose a new version like 19.0.0b30 or move it to Pending.
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az aks machine add
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally? (pip install wheel==0.30.0required)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.jsonautomatically.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.