Description of problem: [UI] - Can't make any changes in custom mode field after pressed on 'OK' button one time. Version-Release number of selected component (if applicable): 4.1.1.2-0.1.el7 How reproducible: 100% Steps to Reproduce: 1. Create bond in ui setup networks with custom bond mode 'mode=1' and press 'OK' 2. Press the edit pencil of the bond again and press 'OK' again (can't) 3. Press 'Cancel' and 'OK' on the setup network sdialog 4. Try to edit the bond with different mode Actual results: 2. Field colored in red 4. Field colored in red Expected results: Should work as expected Additional info: Attaching record to describe the issue
Created attachment 1255474 [details] record
Dominik, Failing QA, as it partially fixed. Steps to Reproduce: 1. Create bond in ui setup networks with custom bond mode 'mode=1' and press 'OK' 2. Press the edit pencil of the bond again and press 'OK' again (can) 3. Press 'OK' on the setup networks dialog 4. Try to edit the bond with different mode Actual results: 2. Fixed 4. Field colored in red. Doesn't fixed. It is still not possible to change the custom mode after it was created once.
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.
Created attachment 1260289 [details] new record
The reason for 4. in comment #2 is a leading space in the custom bonding options. It has the same reason like problem 1) in https://bugzilla.redhat.com/show_bug.cgi?id=1412563 , which is already fixed in the master.
Michael, please VERIFY this bug, while keeping a miimon=NNN value. The leading space issue is not related to this webadmin bug. If you find the leading space issue too troublesome, please file a new bug for it and we can consider backporting it (though we do not want to).
Dan, How can i verify it , if the gap causing to this issue in the first place? It is related, the gap preventing from changing the mode to something else.
If you set miimon=200 on your custom options, you won't see the leading space. The UI issue solved by this patch would have remained in earlier versions, but solved in 4.1.1.3
Fine, as you wish. New bug reported for the gap issue. See bz 1429810 Verified on - 4.1.1.3-0.1.el7