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

salt.states.smartos.vm_present fails when vrrp_vrid is specified #56244

Closed
sjorge opened this issue Feb 25, 2020 · 1 comment
Closed

salt.states.smartos.vm_present fails when vrrp_vrid is specified #56244

sjorge opened this issue Feb 25, 2020 · 1 comment
Assignees
Labels
fixed-pls-verify fix is linked, bug author to confirm fix Proxy-Minion v3000.1 vulnerable version
Milestone

Comments

@sjorge
Copy link
Contributor

sjorge commented Feb 25, 2020

Issue for #55906 as requested

Description

SmartOS supports VRRP configuration for native zones, when adding a nic with vrrp_vrid specified.

The mac for these nics is automatically set to the proper mac as are based on the VRID. As we are keying the nics dict on the mac key this causes issues.

Workaround

Calculate the correct mac and use that as the key for the nics dict.

Solution

In #55906 we will replace the key with the proper mac when a vrrp_vrid is specified. When chacging VRID's the state will remove the old nic and then add a new one with the proper mac + vrid combo copying all other configuration.

@Akm0d Akm0d added fixed-pls-verify fix is linked, bug author to confirm fix and removed needs-triage labels Apr 14, 2020
@Akm0d Akm0d added this to the Approved milestone Apr 14, 2020
@sagetherage
Copy link
Contributor

this is fixed in 3000.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-pls-verify fix is linked, bug author to confirm fix Proxy-Minion v3000.1 vulnerable version
Projects
None yet
Development

No branches or pull requests

5 participants