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 vbmctl to create VM with multiple network interfaces #2207

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mquhuy
Copy link
Member

@mquhuy mquhuy commented Jan 21, 2025

vbmctl currently creates VM with 1 network interface. This is enough for BMO E2E, but to be able to have VMs that work for other purposes, having the flexibility to have multiple networks is required.

@metal3-io-bot metal3-io-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 21, 2025
@metal3-io-bot metal3-io-bot requested review from Rozzii and zaneb January 21, 2025 10:25
@metal3-io-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zaneb for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 21, 2025
@mquhuy mquhuy force-pushed the mquhuy/vbmctl-multiple-networks branch 2 times, most recently from d575d2e to a3de0e6 Compare January 21, 2025 10:56
vbmctl currently creates VM with 1 network interface. This is enough for
BMO E2E, but to be able to have VMs that work for other purposes, having
the flexibility to have multiple networks is required.

Signed-off-by: Huy Mai <[email protected]>
@mquhuy mquhuy force-pushed the mquhuy/vbmctl-multiple-networks branch from a3de0e6 to a5e5470 Compare January 21, 2025 10:59
@mquhuy mquhuy changed the title [WIP] 🌱 Allow vbmctl to create VM with multiple network interfaces 🌱 Allow vbmctl to create VM with multiple network interfaces Jan 21, 2025
@metal3-io-bot metal3-io-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants