Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1403653 - [RFE] Should accept any bond name starting with bond
[RFE] Should accept any bond name starting with bond
Status: NEW
Product: ovirt-engine
Classification: oVirt
Component: BLL.Network (Show other bugs)
---
Unspecified Unspecified
low Severity low (vote)
: ovirt-4.3.0
: ---
Assigned To: Ales Musil
Michael Burman
: FutureFeature
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2016-12-12 01:06 EST by Huijuan Zhao
Modified: 2018-10-16 09:44 EDT (History)
17 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: Network
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rule-engine: ovirt‑4.3+
ylavi: planning_ack+
danken: devel_ack+
mburman: testing_ack+


Attachments (Terms of Use)
All logs and all files in /etc/sysconfig/network-scripts (8.30 MB, application/x-gzip)
2016-12-12 01:06 EST, Huijuan Zhao
no flags Details
Screenshot of invalid bond name (139.75 KB, image/png)
2016-12-12 01:07 EST, Huijuan Zhao
no flags Details
Screenshot of networking after failed adding rhvh to rhvm (111.23 KB, image/png)
2016-12-12 01:09 EST, Huijuan Zhao
no flags Details
Comment 5: error exposed to user (256.58 KB, image/png)
2016-12-20 04:40 EST, Huijuan Zhao
no flags Details
Comment 5: engine.log in engine side (1.48 MB, application/x-gzip)
2016-12-20 04:41 EST, Huijuan Zhao
no flags Details

  None (edit)
Description Huijuan Zhao 2016-12-12 01:06:34 EST
Created attachment 1230704 [details]
All logs and all files in /etc/sysconfig/network-scripts

Description of problem:
There is not prompt message when input invalid bond name in network setting page via cockpit, and can take effect after set up invalid bond name.
In fact, we only support the bond name as bond[0-99], if we setup invalid bond name such as bd0, this will cause adding rhvh to rhvm failed.
There should be prompt message to prevent setting invalid bond name via cockpit.

Version-Release number of selected component (if applicable):
redhat-virtualization-host-4.0-20161206.0
cockpit-ws-122-3.el7.x86_64
NetworkManager-1.4.0-13.el7_3.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Install RHVH 4.0.6 via anaconda.
2. Reboot RHVH and login cockpit, enter Networking page in cockpit
3. Create bond which named bd0 over one NIC em1
4. Add rhvh to rhvm with bond bd0

Actual results:
In step3, there is not prompt message, can set bd0 successful
In step4, add rhvh to rhvm failed

Expected results:
In step3, There should be prompt message to prevent setting invalid bond name via cockpit, and should set up bond failed with invalid bond name.

Additional info:
Comment 1 Huijuan Zhao 2016-12-12 01:07 EST
Created attachment 1230705 [details]
Screenshot of invalid bond name
Comment 2 Huijuan Zhao 2016-12-12 01:09 EST
Created attachment 1230706 [details]
Screenshot of networking after failed adding rhvh to rhvm
Comment 3 Fabian Deutsch 2016-12-12 02:37:28 EST
Moving this back to RHEV, because it seems to be a limitation of vdsm.
Comment 4 Dan Kenigsberg 2016-12-20 03:39:50 EST
Both Engine and Vdsm support only bond+number bondnames. But when and how does adding a node fail? What is the error exposed to the user? Can you attach engine.log? (I could not find it in the big sosreport dump)
Comment 5 Huijuan Zhao 2016-12-20 04:38:47 EST
(In reply to Dan Kenigsberg from comment #4)
> Both Engine and Vdsm support only bond+number bondnames. But when and how
> does adding a node fail? What is the error exposed to the user? Can you
> attach engine.log? (I could not find it in the big sosreport dump)

After setup bond with name bd0 on node, when adding the node to engine with bd0, it failed when configure management network on node. Please refer to attachment for detailed error and engine.log.
Comment 6 Huijuan Zhao 2016-12-20 04:40 EST
Created attachment 1233768 [details]
Comment 5: error exposed to user
Comment 7 Huijuan Zhao 2016-12-20 04:41 EST
Created attachment 1233769 [details]
Comment 5: engine.log in engine side
Comment 8 Huijuan Zhao 2016-12-20 04:44:02 EST
(In reply to Huijuan Zhao from comment #5)

> After setup bond with name bd0 on node, when adding the node to engine with
> bd0, it failed when configure management network on node. Please refer to
> attachment for detailed error and engine.log.

Additional info:
vdsmd status on node after failed adding node to engine:
# systemctl status vdsmd
● vdsmd.service - Virtual Desktop Server Manager
   Loaded: loaded (/usr/lib/systemd/system/vdsmd.service; enabled; vendor preset: enabled)
   Active: active (running) since Tue 2016-12-20 09:28:09 GMT; 6min ago
 Main PID: 24151 (vdsm)
   CGroup: /system.slice/vdsmd.service
           └─24151 /usr/bin/python /usr/share/vdsm/vdsm

Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 ask_user_info()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 client step 1
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 ask_user_info()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 make_client_response()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 client step 2
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 parse_server_challenge()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 ask_user_info()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 make_client_response()
Dec 20 09:28:10 ibm-x3650m5-04.lab.eng.pek2.redhat.com python[24151]: DIGEST-MD5 client step 3
Dec 20 09:28:14 ibm-x3650m5-04.lab.eng.pek2.redhat.com vdsm[24151]: vdsm vds ERROR u'bd0' is not a valid bonding device name
                                                                    Traceback (most recent call last):
                                                                      File "/usr/share/vdsm/API.py", line 1473, in setupNetworks...
Hint: Some lines were ellipsized, use -l to show in full.
Comment 9 Dan Kenigsberg 2018-04-04 04:29:48 EDT
I am not sure we must support just ANY bond name; but we should allow more significant names with the `bond` prefix.
Comment 10 Dan Kenigsberg 2018-04-11 04:06:36 EDT
I'd like to support only names such as "bond20g" (bond*)
Comment 11 Michael Burman 2018-10-16 06:13:55 EDT
Note that this RFE shouldn't be backported to 4.2.z, only for 4.3

Note You need to log in before you can comment on or make changes to this bug.