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

Allow configuration of vm bridge #21

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

Conversation

oliverwelter
Copy link

The naming of the network bridge might differ across proxmox installations because of different naming conventions or because of using multiple networks. Externalizing the bridge name for the setup of the vm template as an environment variable avoids applying changes to the shell script directly.

The storage requirements for the debian system and the packages which are installed during the template creation are exceeding the default of 4.3G, increasing the disk to 4.6G solves it.

Using 4.3G as disk size leads to error message indicating critically low value during package installation.
Copy link
Owner

@christensenjairus christensenjairus left a comment

Choose a reason for hiding this comment

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

This is great! Thank you so much for helping out

I'll merge as soon as you're done with my nitpicks.

@@ -72,14 +72,16 @@ ETCD_VERSION=3.5.16
# template creation process.

### Debian 12 Image (Bookworm)
TEMPLATE_VM_BRIDGE=vmbr0

Choose a reason for hiding this comment

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

Could you place this up near line 12?

Copy link
Owner

@christensenjairus christensenjairus Jan 12, 2025

Choose a reason for hiding this comment

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

Could you also add the TEMPLATE_VM_BRIDGE variable somewhere near line 280 in clustercreator.sh? To ensure that the script does not run if the user did not define it.


### Ubuntu 24.04 LTS Image (Noble Numbat)
#TEMPLATE_VM_BRIDGE=vmbr0

Choose a reason for hiding this comment

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

When the other line is moved you can remove this

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