We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
vlan
VLAN_PLUS_VID
According to the dracut man page, there are four different forms of VLAN names:
When parsing the parameter, the parsing function assumes that the field uses either the DEV_PLUS_VID form or the DEV_PLUS_VID_NO_PAD form.
DEV_PLUS_VID
DEV_PLUS_VID_NO_PAD
Instead, the function should do the following:
var vlanNumberString string if strings.HasPrefix(vlanName, "vlan") { vlanNumberString = strings.TrimPrefix(vlanName, "vlan") } else { _, vlanNumberString, ok := strings.Cut(vlanName, ".") if !ok { return settings, fmt.Errorf("malformed vlan commandline argument: %s", *vlanSettings) } }
The text was updated successfully, but these errors were encountered:
feat: add parsing of vlanNNNN:ethX style VLAN cmdline args
5ae44c0
Fixes siderolabs#9552 Signed-off-by: Joakim Nohlgård <joakim@nohlgard.se>
b7d650d
Fixes siderolabs#9552 Signed-off-by: Joakim Nohlgård <joakim@nohlgard.se> Signed-off-by: Dmitriy Matrenichev <dmitry.matrenichev@siderolabs.com>
78353f7
f8155c4
Check for invalid VLAN ID numbers during cmdline parsing Fixes siderolabs#9552 Signed-off-by: Joakim Nohlgård <joakim@nohlgard.se> Signed-off-by: Dmitriy Matrenichev <dmitry.matrenichev@siderolabs.com> (cherry picked from commit 78353f7)
Successfully merging a pull request may close this issue.
Bug Report
Description
According to the dracut man page, there are four different forms of VLAN names:
When parsing the parameter, the parsing function assumes that the field uses either the
DEV_PLUS_VID
form or theDEV_PLUS_VID_NO_PAD
form.Instead, the function should do the following:
Logs
Environment
The text was updated successfully, but these errors were encountered: