Bug 1600059
| Summary: | [RFE] Add by default a storage lease to HA VMs | |||
|---|---|---|---|---|
| Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Andrea Perotti <aperotti> | |
| Component: | ovirt-engine | Assignee: | Eyal Shenitzky <eshenitz> | |
| Status: | CLOSED ERRATA | QA Contact: | meital avital <mavital> | |
| Severity: | high | Docs Contact: | ||
| Priority: | unspecified | |||
| Version: | 4.2.7 | CC: | aefrat, emarcus, eshenitz, lsurette, michal.skrivanek, mkalinin, rdlugyhe, sfishbai, srevivo, tnisan | |
| Target Milestone: | ovirt-4.4.0 | Keywords: | FutureFeature, ZStream | |
| Target Release: | --- | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | ||||
| Fixed In Version: | Doc Type: | Enhancement | ||
| Doc Text: |
Previously, when High Availability was selected for a new virtual machine, the Lease Storage Domain was set to a bootable Storage Domain automatically if the user did not already select one.
In this release, a bootable Storage Domain is set as the lease Storage Domain for new High Availability virtual machines.
|
Story Points: | --- | |
| Clone Of: | ||||
| : | 1719735 (view as bug list) | Environment: | ||
| Last Closed: | 2020-08-04 13:16:11 UTC | Type: | Bug | |
| Regression: | --- | Mount Type: | --- | |
| Documentation: | --- | CRM: | ||
| Verified Versions: | Category: | --- | ||
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | ||
| Cloudforms Team: | --- | Target Upstream Version: | ||
| Embargoed: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1719735 | |||
|
Description
Andrea Perotti
2018-07-11 10:45:22 UTC
to clarify the bug description - the consideration is only for HA VMs. Regular VMs will still not use the lease mechanism The risk that described above is also relevant for an existing VM that is updated to be HA VM. The fix for this RFE should solve this option too. After giving it another thought, I believe that this default behavior should take place only when the user creating/updating a VM to HA via the UI. We should avoid this behavior when using the REST and keep it as is (we count that the user is knowing what he is doing when using the REST-API). So I think the solution will be much simpler, just changing the default value of the lease SD in the UI from 'No VM lease' to one of the VM's disks storage domains. people will eventually complain, they always do when we do something in UI only. The patch comments are relatively small, and the correct handling on update VM needs to be happen anyway for UI too. We can change behavior in new cluster level so it wouldn't affect existing setups. OTOH it's not a big deal if there's indeed some complication with that. Maybe with a clear note in REST API docs it's ok... Creating or updating a VM to HA will create a lease as default behavior from the UI, for REST we will leave it as is now as the change is not visible as in the UI and will practically force the user to have a VM lease when creating an HA VM (In reply to Tal Nisan from comment #6) > Creating or updating a VM to HA will create a lease as default behavior from > the UI, for REST we will leave it as is now as the change is not visible as > in the UI and will practically force the user to have a VM lease when > creating an HA VM Setting a lease by default when updating a VM to be HA is problematic. We cannot distinguish between the two options: 1) The VM set as HA and doesn't have a lease 2) The user removed the lease from the HA VM Therefore the solution will be setting a VM lease by default only for new HA VMs. WARN: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops
WARN: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops
WARN: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops
WARN: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops
WARN: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops: Bug status (ON_QA) wasn't changed but the folowing should be fixed:
[Found non-acked flags: '{}', ]
For more info please contact: rhv-devops
Verified When setting a VM lease on the storage domain that contains the VM bootable disk, the HA sets by default. ovirt-engine-4.4.0-0.19.master.el7.noarch vdsm-4.40.2-1.el8ev.x86_64 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: RHV Manager (ovirt-engine) 4.4 security, bug fix, and enhancement 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-2020:3247 |