Bug 2168749
| Summary: | Rename of Network Interface duplicates it, breaks VM start | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Container Native Virtualization (CNV) | Reporter: | Germano Veit Michel <gveitmic> | ||||||
| Component: | User Experience | Assignee: | Ugo Palatucci <upalatuc> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Guohua Ouyang <gouyang> | ||||||
| Severity: | medium | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | 4.12.0 | CC: | gouyang, tnisan, upalatuc | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | 4.14.0 | ||||||||
| Hardware: | x86_64 | ||||||||
| OS: | Linux | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | |||||||||
| : | 2182006 (view as bug list) | Environment: | |||||||
| Last Closed: | 2023-11-08 14:05:04 UTC | Type: | Bug | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 2182006 | ||||||||
| Attachments: |
|
||||||||
|
Description
Germano Veit Michel
2023-02-10 00:09:25 UTC
Created attachment 1943205 [details]
admission webhook error on UI
I could see the error is displaying on UI while rename the Nic Name on CNV 4.12.1-25:
admission webhook "mutatevirtualmachines.kubemacpool.io" denied the request: Failed to update virtual machine allocation error: failed to allocate requested mac address
(In reply to Guohua Ouyang from comment #1) > Created attachment 1943205 [details] > admission webhook error on UI > > I could see the error is displaying on UI while rename the Nic Name on CNV > 4.12.1-25: > admission webhook "mutatevirtualmachines.kubemacpool.io" denied the request: > Failed to update virtual machine allocation error: failed to allocate > requested mac address Yup. And I see your VM is running in the screenshot. If you do with the VM down then it does not try to allocate mac and you will get the duplicate NIC. But later VM will fail to start, with same mac address error. Created attachment 1943207 [details]
rename nic with VM is down
Could still see the admission webhook error denied the request while VM is down.
(In reply to Guohua Ouyang from comment #3) > Created attachment 1943207 [details] > rename nic with VM is down > > Could still see the admission webhook error denied the request while VM is > down. My bad. Do it in the VM customisation dialog, while creating the VM. There it will go duplicate, but the problem is the same. Also same problem when trying to rename the Pod NIC on VM customisation. It will try to create duplicate and then fail as only 1 Pod network is allowed. I could reproduce the issue while creating the VM via customize mode, the problem is editing a NIC is not saved to the original NIC but created a new NIC. It looks #2158063 and this bug are caused by the this problem. Exactly, it seems to try to create a new NIC, triggering all these problems. Thanks (In reply to Guohua Ouyang from comment #6) > It looks #2158063 and this bug are caused by the this problem. Actually I did some more testing and I think they are different. Because the error on that bug is the error when 2 POD network NICs are attached, but there the Edit changes the 2nd NIC to Bridge. If the edit was just doing a copy/duplicate and changing to Bridge it would not hit that error, because the original NIC would be Pod Net and the copy would be Bridge. The maintainers will know better. @gouyang we've completely refactored the network interface modal. I can't reproduce this issue and i think we resolved that with the refactor (In reply to Ugo Palatucci from comment #9) > @gouyang we've completely refactored the network interface modal. > > I can't reproduce this issue and i think we resolved that with the refactor Could not reproduce on 4.13 too. While renaming the nic will see the error from backend, but it's not the UI issue and I guess it's not supported by the backend. admission webhook "mutatevirtualmachines.kubemacpool.io" denied the request: Failed to update virtual machine allocation error: failed to allocate requested mac address @Germano, wdyt? (In reply to Guohua Ouyang from comment #11) > While renaming the nic will see the error from backend, but it's not the UI > issue and I guess it's not supported by the backend. > admission webhook "mutatevirtualmachines.kubemacpool.io" denied the request: > Failed to update virtual machine allocation error: failed to allocate > requested mac address > > @Germano, wdyt? Is this on editing an already created VM, or in the VM customization dialog? Both were sort of broken... Also, for best user experience, I think: * If it cannot be renamed for already created NIC/VM, then don't let the user try to rename it in the UI. * But in customization dialog, it should allow to rename it, the VM is not created yet. What you you think? (In reply to Germano Veit Michel from comment #12) > (In reply to Guohua Ouyang from comment #11) > > Is this on editing an already created VM, or in the VM customization dialog? > > Both were sort of broken... > > Also, for best user experience, I think: > * If it cannot be renamed for already created NIC/VM, then don't let the > user try to rename it in the UI. Ugo, do you like to disable the name field while editing the existing Nic? > * But in customization dialog, it should allow to rename it, the VM is not > created yet. Checked on 4.13, it allow to rename the nic name in customization dialog. > > What you you think? Yes we can do that verified on upstream latest Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Important: OpenShift Virtualization 4.14.0 Images security and bug fix update), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2023:6817 |