[TEST - DO NOT MERGE] {Consumption} Bump budgets to 2024-08-01 (aaz-dev-mcp test)#33368
Draft
khang-11 wants to merge 3 commits into
Draft
[TEST - DO NOT MERGE] {Consumption} Bump budgets to 2024-08-01 (aaz-dev-mcp test)#33368khang-11 wants to merge 3 commits into
khang-11 wants to merge 3 commits into
Conversation
|
Hi @khang-11, |
🔄AzureCLI-FullTest
|
❌AzureCLI-BreakingChangeTest
Please submit your Breaking Change Pre-announcement ASAP if you haven't already. Please note:
|
microsoft-github-policy-service
Bot
added
Reservations
khang-11
force-pushed
the
mcp-test-consumption
branch
from
eb4d777 to
8463346
Compare
khang-11
force-pushed
the
mcp-test-consumption
branch
from
8463346 to
8218165
Compare
Collaborator
|
Consumption |
Contributor
|
@khang-11 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
khang-11
force-pushed
the
mcp-test-consumption
branch
from
29820bf to
8c93ce9
Compare
…aliases, flattened --time-period, and curated examples
khang-11
force-pushed
the
mcp-test-consumption
branch
2 times, most recently
from
a683975 to
618171d
Compare
…e tests After regenerating the consumption module against API 2024-08-01, the hand-written budget wrappers in commands.py / custom.py registered a conflicting argument shape (--budget-name with no --scope) and called aaz Create/Show/Delete without the now-required scope. Drop those wrappers, the budget arg context in _params.py, and the now-unused _validators.py / _transformers.py so the aaz-generated 'consumption budget' commands take effect with their --name / --scope arguments. Update _help.py examples to match the new argument shape (azdev linter's faulty_help_example_parameters_rule rejected the old --budget-name examples). Gut test_consumption_commands.py: every existing recording was captured at api-version 2017-11-30 / 2023-05-01 and at the old URL shapes, so cassette playback can no longer match the new requests. Scenarios need to be re-recorded against a live subscription.
khang-11
force-pushed
the
mcp-test-consumption
branch
from
618171d to
6ecff09
Compare
khang-11
force-pushed
the
mcp-test-consumption
branch
from
7d325b7 to
27ab73b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is a test PR. Do not merge. Demonstrates the aaz-dev-mcp server regenerating the entire
consumptionCLI module at API version 2024-08-01, with workspace renames to preserve legacy command names.Result
All 10 existing commands updated in place at the same paths:
consumption budget {list,show,create,update,delete}consumption marketplace listconsumption usage listconsumption reservation detail listconsumption reservation summary listconsumption pricesheet showThe only deletions are budget's
_*_with_rg.pyvariants: the 2024-08-01 spec consolidates subscription + resource-group flows under/{scope}/..., so the separate_with_rggenerators are no longer needed.MCP rename tools (new)
This iteration relies on two new MCP tools added to aaz-dev-mcp:
rename_command_group(name, node_names, new_node_names)rename_command(name, leaf_names, new_leaf_names)The auto-generated swagger names (
usage-detail,reservation-detail,reservation-summary,pricesheet default show) are renamed in the workspace to the legacy CLI names beforegenerate_to_aaz.Companion aaz PR: Azure/aaz#1007