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

lxd_metadata: annotate remaining network properties #13355

Merged

Conversation

@github-actions github-actions bot added Documentation Documentation needs updating API Changes to the REST API labels Apr 19, 2024
Copy link

Heads up @ru-fu - the "Documentation" label was applied to this issue.

Copy link
Contributor

@ru-fu ru-fu left a comment

Choose a reason for hiding this comment

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

I've added my commits - please squash accordingly. :)

There's also one field missing, see comment.

doc/howto/network_ovn_peers.md Show resolved Hide resolved
@gabrielmougard
Copy link
Contributor Author

@ru-fu updated

shared/api/network_peer.go Outdated Show resolved Hide resolved
shared/api/network_peer.go Show resolved Hide resolved
@tomponline
Copy link
Member

lxd metadata permissions issues fixed in 2bda09e

@gabrielmougard gabrielmougard force-pushed the feat/annotate-network-properties branch from fc77c68 to 4d5081b Compare May 2, 2024 12:19
@gabrielmougard
Copy link
Contributor Author

@ru-fu udpated

@ru-fu
Copy link
Contributor

ru-fu commented May 2, 2024

#13379 has now been merged - so when bringing in those changes, #13379 (comment) and #13379 (comment) must be updated.

@tomponline
Copy link
Member

Needs a rebase please

gabrielmougard and others added 8 commits May 3, 2024 16:43
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Co-authored-by: Ruth Fuchss <ruth.fuchss@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Co-authored-by: Ruth Fuchss <ruth.fuchss@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Co-authored-by: Ruth Fuchss <ruth.fuchss@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Co-authored-by: Ruth Fuchss <ruth.fuchss@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Co-authored-by: Ruth Fuchss <ruth.fuchss@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
Signed-off-by: Gabriel Mougard <gabriel.mougard@canonical.com>
@gabrielmougard gabrielmougard force-pushed the feat/annotate-network-properties branch from 4d5081b to 84cc9d8 Compare May 3, 2024 15:04
@gabrielmougard
Copy link
Contributor Author

@ru-fu @tomponline updated

@tomponline tomponline merged commit 099e4df into canonical:main May 3, 2024
28 checks passed
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.

3 participants