Bug 1580346 - Cluster properties 'Enable to set host/VM...' should be set by default (and hidden?)
Summary: Cluster properties 'Enable to set host/VM...' should be set by default (and h...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.3.0
: ---
Assignee: Dana
QA Contact: Eli Mesika
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-05-21 10:34 UTC by Yaniv Kaul
Modified: 2019-10-22 05:16 UTC (History)
3 users (show)

Fixed In Version: ovirt-engine-4.3.0_rc2
Doc Type: Enhancement
Doc Text:
In this release, the cluster property "set maintenance reason" is enabled by default.
Clone Of:
Environment:
Last Closed: 2019-02-13 07:45:09 UTC
oVirt Team: Infra
Embargoed:
rule-engine: ovirt-4.3+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 96040 0 master MERGED engine: Enable to set maintenance reason is set to ON 2018-12-20 08:55:41 UTC
oVirt gerrit 96371 0 master MERGED core: Fix upgrade script issue 2018-12-21 09:10:36 UTC
oVirt gerrit 96598 0 master MERGED 'maintenance_reason' and 'optional_reason' fields are deprecated 2019-01-14 12:36:28 UTC
oVirt gerrit 96948 0 master MERGED restapi: Update to model 4.3.21 2019-01-17 11:20:24 UTC

Description Yaniv Kaul 2018-05-21 10:34:24 UTC 
Description of problem:
We have 2 cluster properties:
- Enable to set VM maintenance reason
- Enable to set Host maintenance reason

1. Should be set to ON by default (checked)
2. Do not even need to appear in the UI - I don't see the point in disabling them. If you don't want to input a reason, don't.

I prefer removing them altogether, having them set to ON by default. It'll simplify a bit the UI and ensure these useful features are enabled by default.
Comment 1 Martin Perina 2018-11-30 15:59:13 UTC 
Let's remove those settings from webadmin and by default enable to set maintenance reason in both UI and RESTAPI
Comment 2 Martin Perina 2019-01-16 11:56:51 UTC 
Moving back to MODIFIED as RESTAPI verification requires new model as mentioned in https://gerrit.ovirt.org/96948
Comment 3 Sandro Bonazzola 2019-02-13 07:45:09 UTC 
This bugzilla is included in oVirt 4.3.0 release, published on February 4th 2019.

Since the problem described in this bug report should be
resolved in oVirt 4.3.0 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.
Note You need to log in before you can comment on or make changes to this bug.