Skip to content

OPEN-4593 - Add tags to space mappings#217

Merged
ahmad-can merged 1 commit into
mainfrom
tag-to-spaces
Jul 7, 2026
Merged

OPEN-4593 - Add tags to space mappings#217
ahmad-can merged 1 commit into
mainfrom
tag-to-spaces

Conversation

@ahmad-can

@ahmad-can ahmad-can commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a Required Spaces column to the MAAS machine tags assignment table.

Changes

  • Documented required spaces for:
    • control
    • compute
    • storage
    • network
    • sunbeam
    • juju-controller

OPEN-4593

@gboutry gboutry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should rather list the Sunbeam Networks naming rather than the MAAS space, as MAAS space naming is to the discretion of the end users / operators, and why we do network to space mapping.

@ahmad-can

Copy link
Copy Markdown
Contributor Author

We should rather list the Sunbeam Networks naming rather than the MAAS space, as MAAS space naming is to the discretion of the end users / operators, and why we do network to space mapping.

@gboutry in that our sunbeam Network mapping is incomplete as it doesn't point to admin space for juju_controller

https://github.com/canonical/snap-openstack/blob/main/sunbeam-python/sunbeam/provider/maas/deployment.py#L119

@gboutry

gboutry commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Then, it's a bug to fix there, rather than to document here IMO.

But, what does admin space represent in your setup?

Why does Juju need both OAM and ADMIN?

Is OAM management and ADMIN internal?

Signed-off-by: Ahmad Hassan <ahmad.hassan@canonical.com>
@ahmad-can

Copy link
Copy Markdown
Contributor Author

I have updated the table with sunbeam network names.

@ahmad-can

Copy link
Copy Markdown
Contributor Author
image

@ahmad-can
ahmad-can requested a review from gboutry June 30, 2026 07:08
@gboutry
gboutry requested a review from hmlanigan July 2, 2026 08:09
@ahmad-can
ahmad-can merged commit 461ec3b into main Jul 7, 2026
3 of 4 checks passed
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.

3 participants