Skip to content

Support netbox config generation for manila devices - #160

Open
sebageek wants to merge 2 commits into
stable/2024.1-m3from
config-gen-manila-support
Open

sebageek wants to merge 2 commits into
stable/2024.1-m3from
config-gen-manila-support

Conversation

@sebageek

@sebageek sebageek commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

With manila the individual node servers are directly connected to the
switch in NetBox. Neither the devices nor the parent device have any
cluster config whatsoever. To find all devices we need to find the
parent device via parent_bay and then look at all the devicebays from
there (close, but not similar to how we handle loadbalancers). We also
filter on the manila tag being present on the parent device.

@sebageek
sebageek requested review from a team as code owners August 26, 2026 11:20
@sebageek sebageek changed the title Config gen manila support Support netbox config generation for manila devices Aug 26, 2026
@sebageek
sebageek force-pushed the config-gen-manila-support branch from 908fdcf to 86b9167 Compare August 26, 2026 11:31
Loadbalancer requires some more handling on device discovery. As we need
to do something similar for manila we move the code for loadbalancers
into its own method to unclutter get_connected_devices() a bit.
With manila the individual node servers are directly connected to the
switch in NetBox. Neither the devices nor the parent device have any
cluster config whatsoever. To find all devices we need to find the
parent device via parent_bay and then look at all the devicebays from
there (close, but not similar to how we handle loadbalancers). We also
filter on the manila tag being present on the parent device.
@sebageek
sebageek force-pushed the config-gen-manila-support branch from 86b9167 to f98a845 Compare September 14, 2026 14:50

@occamshatchet occamshatchet 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.

Looks good to me.

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