Skip to content

Snapshot reference support#57

Open
linglingye001 wants to merge 3 commits intorelease/v1.6.0from
linglingye/snapshot-ref
Open

Snapshot reference support#57
linglingye001 wants to merge 3 commits intorelease/v1.6.0from
linglingye/snapshot-ref

Conversation

@linglingye001
Copy link
Member

@linglingye001 linglingye001 commented Nov 27, 2025

ref

Base automatically changed from release/v1.4.0 to main December 8, 2025 06:54
@linglingye001 linglingye001 force-pushed the linglingye/snapshot-ref branch from d07e3d1 to a047276 Compare March 3, 2026 05:52
@linglingye001 linglingye001 marked this pull request as ready for review March 3, 2026 06:11
@linglingye001 linglingye001 changed the base branch from main to release/v1.6.0 March 3, 2026 06:11
@linglingye001 linglingye001 requested a review from Copilot March 3, 2026 06:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds "Snapshot Reference" support to the Azure App Configuration Go provider. Snapshot references allow a key-value setting stored in App Configuration to point to a named snapshot — when loaded, the settings from that snapshot are merged into the configuration. The implementation refactors the existing snapshot loading into a shared helper function, adds content-type detection for snapshot references, and emits a tracing tag when they are used.

Changes:

  • Extracts snapshot loading into a reusable loadSnapshotSettings function and adds snapshotSettingsLoader function type to support snapshot reference resolution with dependency injection.
  • Adds detection of snapshotReferenceContentType in loadKeyValues, collects snapshot references, and resolves them via the new loadSettingsFromSnapshotRefs method.
  • Adds UseSnapshotReference tracing option and SnapshotReferenceTag correlation context tag.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
azureappconfiguration/azureappconfiguration.go Core implementation: detection of snapshot reference content type, loadSettingsFromSnapshotRefs, and parseSnapshotReference
azureappconfiguration/settings_client.go Extracts snapshot loading into loadSnapshotSettings helper; adds snapshotSettingsLoader type
azureappconfiguration/internal/tracing/tracing.go Adds SnapshotReferenceTag and UseSnapshotReference option to correlation context header
azureappconfiguration/constants.go Adds snapshotReferenceContentType constant
azureappconfiguration/options.go Minor whitespace fix
azureappconfiguration/snapshot_test.go Adds unit tests for parseSnapshotReference and loadSettingsFromSnapshotRefs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants