Skip to content

feat: add networkBridge to node model#248

Merged
runleveldev merged 2 commits intomainfrom
rgingras/feature/246-selectable-network-bridge-for-nodes
Mar 23, 2026
Merged

feat: add networkBridge to node model#248
runleveldev merged 2 commits intomainfrom
rgingras/feature/246-selectable-network-bridge-for-nodes

Conversation

@runleveldev
Copy link
Collaborator

This pull request adds support for configuring the Proxmox network bridge (networkBridge) used when creating containers. The change introduces a new networkBridge field to the Node model, database schema, and associated forms and routes, defaulting to 'vmbr0'. This allows users to specify which Proxmox bridge should be used per node, providing greater flexibility and clarity.

Database and Model Updates:

  • Added a new non-nullable networkBridge column (default: 'vmbr0') to the Nodes table via a Sequelize migration.
  • Updated the Node model to include the networkBridge attribute with the appropriate type and default value.

API and Form Handling:

  • Updated the node creation (POST /), import (POST /import), and update (PUT /:id) routes to accept and handle the networkBridge field, defaulting to 'vmbr0' if not provided. [1] [2] [3] [4] [5]
  • Added a new input field for networkBridge to the node form UI, pre-filled with the current value or 'vmbr0' by default.

Documentation:

  • Updated the database schema diagram and documentation to reflect the new networkBridge field and its default value. [1] [2]

@runleveldev runleveldev linked an issue Mar 19, 2026 that may be closed by this pull request
Copy link
Collaborator

@cmyers-mieweb cmyers-mieweb left a comment

Choose a reason for hiding this comment

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

LGTM

@runleveldev runleveldev merged commit c56076b into main Mar 23, 2026
5 of 6 checks passed
@runleveldev runleveldev deleted the rgingras/feature/246-selectable-network-bridge-for-nodes branch March 23, 2026 19:10
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.

Selectable network bridge for nodes

2 participants