Bug 1530583

Summary: [OVS] - Disable/gray out the option to change the switch type on existing cluster
Product: [oVirt] ovirt-engine Reporter: Michael Burman <mburman>
Component: BLL.NetworkAssignee: Ales Musil <amusil>
Status: CLOSED CURRENTRELEASE QA Contact: Michael Burman <mburman>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.2.0CC: amusil, bugs, edwardh, lveyde, ylavi
Target Milestone: ovirt-4.2.2Flags: rule-engine: ovirt-4.2+
ylavi: exception+
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: ovirt-engine-4.2.2.4 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-03-29 11:10:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Michael Burman 2018-01-03 12:11:00 UTC
Description of problem:
[OVS] - Disable/gray out the option to change the switch type on existing cluster

We decided to disable the option to change the cluster's type ovs<>legacy and it's now should be grayed out for existing clusters.
The only option to choose the cluster's type will be on new cluster flow.

Version-Release number of selected component (if applicable):
4.2.0.2-0.1.el7

Comment 1 Edward Haas 2018-02-20 13:00:29 UTC
There may be an agreement, but there is no consensus here.
IMO this is wrong.

We are breaking users who used it until now (for a year+ it was out there) and they will not be able to change their cluster back (from OVS to legacy).

I guess this came up from a reasoning that VM/s cannot be migrated to a new network type, which IMO should be a blocker to any feature that seeks users.
New capabilities will probably be added to OVN/OVS based clusters, leaving existing users behind, this makes little sense to me.

Comment 2 Michael Burman 2018-03-18 08:12:21 UTC
Hello Ales,

I'm going to verify this, but wondering should we add a tooltip on the grayed out 'Switch Type' layout explaining why it is disabled or something?

Comment 3 Michael Burman 2018-03-18 08:16:20 UTC
Verified on - 4.2.2.4-0.1.el7

Comment 4 Sandro Bonazzola 2018-03-29 11:10:09 UTC
This bugzilla is included in oVirt 4.2.2 release, published on March 28th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.2 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.

Comment 5 Ales Musil 2018-04-17 06:05:17 UTC
Sorry for the late response. We can add it later if it is needed.