[AppConfig] az appconfig kv: Add snapshot reference support#33278
Open
ChristineWanjau wants to merge 7 commits intoAzure:devfrom
Open
[AppConfig] az appconfig kv: Add snapshot reference support#33278ChristineWanjau wants to merge 7 commits intoAzure:devfrom
az appconfig kv: Add snapshot reference support#33278ChristineWanjau wants to merge 7 commits intoAzure:devfrom
Conversation
❌AzureCLI-FullTest
|
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| appconfig kv list | cmd appconfig kv list added parameter resolve_snapshot_references |
||
| appconfig kv set-snapshot-reference | cmd appconfig kv set-snapshot-reference added |
Collaborator
|
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>
|
microsoft-github-policy-service
Bot
requested review from
avanigupta,
jsntcy,
yanzhudd,
yonzhan and
zhoxing-ms
microsoft-github-policy-service
Bot
added
App Configuration
Auto-Assign
ChristineWanjau
changed the title
az appconfig kv: Add snapshot reference supportaz appconfig kv: Add snapshot reference support
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Azure App Configuration snapshot reference support to az appconfig kv by introducing a new command to create/update snapshot reference key-values and a new list flag to expand references into the snapshot’s key-values.
Changes:
- Add
az appconfig kv set-snapshot-referenceto write snapshot reference key-values with the appropriate content type/value format. - Add
az appconfig kv list --resolve-snapshot-referencesto expand snapshot references into the referenced snapshot’s key-values (in sequence). - Add constants, CLI params/validators/help text, and scenario tests for the new behaviors.
Reviewed changes
Copilot reviewed 7 out of 10 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
src/azure-cli/azure/cli/command_modules/appconfig/tests/latest/test_appconfig_snapshot_reference_commands.py |
Adds scenario coverage for setting snapshot references and listing with reference resolution. |
src/azure-cli/azure/cli/command_modules/appconfig/keyvalue.py |
Implements set_snapshot_reference and snapshot-reference expansion in kv list. |
src/azure-cli/azure/cli/command_modules/appconfig/commands.py |
Registers the new appconfig kv set-snapshot-reference command. |
src/azure-cli/azure/cli/command_modules/appconfig/_validators.py |
Adds validation for --snapshot-name. |
src/azure-cli/azure/cli/command_modules/appconfig/_params.py |
Wires new command arguments and the --resolve-snapshot-references flag. |
src/azure-cli/azure/cli/command_modules/appconfig/_help.py |
Documents the new command and list flag with examples. |
src/azure-cli/azure/cli/command_modules/appconfig/_constants.py |
Introduces snapshot reference constants (content type + JSON key). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
added 2 commits
April 30, 2026 15:02
mrm9084
reviewed
Apr 30, 2026
| @@ -0,0 +1,180 @@ | |||
| # -------------------------------------------------------------------------------------------- | |||
Member
There was a problem hiding this comment.
Commenting here as I don't want to in the recording. I think test_azconfig_kv_list_resolve_snapshot_ref.yaml must have been accidentally added as there is no test for it. test_azconfig_kv_list_resolve_snapshot_references is a test. I'm assuming it's an old name.
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
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.
Related command
az appconfig kv set-snapshot-reference (new)
az appconfig kv list --resolve-snapshot-references (new flag)
Description
New command — az appconfig kv set-snapshot-reference
Creates or updates a snapshot reference key-value at the given key (and optional label) pointing at a snapshot by name.
New flag — az appconfig kv list --resolve-snapshot-references
When set, every snapshot reference returned by the list is replaced in sequence by the key-values of the snapshot it points to. Non-reference key-values pass through unchanged.
Testing Guide
History Notes
[Component Name 1]
az appconfig kv set-snapshot-reference: Adds support to create a snapshot reference key-value[Component Name 2]
az appconfig kv list: Adds support to list key-values from a snapshot referenceThis checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.