Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API: Add scope to metadata configuration API #14530

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

kadinsayani
Copy link
Contributor

@kadinsayani kadinsayani commented Nov 25, 2024

Resolves #14125.

This PR adds scope to the metadata configuration API endpoint, /1.0/metadata/configuration.

Summary of changes:

  • Adds api extension, metadata_configuration_scope;
  • Adds Scope field to MetadataConfigurationConfigKey struct;
  • Adds scope to metadata for member specific config keys in lxd/network/driver_{bridge/macvlan/physical/sriov}.go;
  • Updates API docs and metadata.

@github-actions github-actions bot added Documentation Documentation needs updating API Changes to the REST API labels Nov 25, 2024
@kadinsayani
Copy link
Contributor Author

cc. @edlerd

@kadinsayani kadinsayani force-pushed the metadata-configuration-scope branch 3 times, most recently from 791eff4 to 4869da3 Compare November 26, 2024 16:12
@kadinsayani kadinsayani changed the title Add scope to metadata configuration API Network: Add scope to metadata configuration API Nov 26, 2024
@kadinsayani kadinsayani marked this pull request as ready for review November 26, 2024 16:16
@kadinsayani kadinsayani changed the title Network: Add scope to metadata configuration API API: Add scope to metadata configuration API Nov 26, 2024
Copy link
Contributor

@markylaing markylaing left a comment

Choose a reason for hiding this comment

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

We'll need @tomponline or @simondeziel to review the local/global-ness of the config keys but other than that I just have a couple of small nits :)

doc/api-extensions.md Outdated Show resolved Hide resolved
shared/api/metadata_configuration.go Outdated Show resolved Hide resolved
lxd/network/driver_bridge.go Show resolved Hide resolved
lxd/network/driver_macvlan.go Show resolved Hide resolved
lxd/network/driver_macvlan.go Show resolved Hide resolved
lxd/network/driver_macvlan.go Show resolved Hide resolved
@kadinsayani kadinsayani force-pushed the metadata-configuration-scope branch 2 times, most recently from 8a58825 to c49b172 Compare November 27, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Changes to the REST API Documentation Documentation needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Metadata API to indicate cluster member specificity
4 participants