Bug 1991622 - Creating conflicting Affinity Groups brings a message to user with VM Ids instead of names
Summary: Creating conflicting Affinity Groups brings a message to user with VM Ids ins...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Virt
Version: 4.4.7
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ovirt-4.5.2
: 4.5.2
Assignee: Milan Zamazal
QA Contact: Nikolai Sednev
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-08-09 14:31 UTC by Polina
Modified: 2022-08-30 08:47 UTC (History)
6 users (show)

Fixed In Version: ovirt-engine-4.5.2
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-08-30 08:47:42 UTC
oVirt Team: Virt
Embargoed:
nsednev: needinfo-
sbonazzo: ovirt-4.5+


Attachments (Terms of Use)
AffinityOperationCanceled.png (197.15 KB, image/png)
2021-08-09 14:31 UTC, Polina
no flags Details
Screen shot of reproduced issue from 11.08.22. (121.08 KB, image/png)
2022-08-11 17:45 UTC, Nikolai Sednev
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github oVirt ovirt-engine pull 542 0 None open core: Report names rather than ids in affinity conflicts 2022-07-20 18:38:37 UTC
Red Hat Issue Tracker RHV-42992 0 None None None 2021-08-09 14:35:34 UTC

Description Polina 2021-08-09 14:31:53 UTC
Created attachment 1812455 [details]
AffinityOperationCanceled.png

Description of problem:
While adding a new affinity group conflicting with another existing group the user gets an error window talking about VM Ids which looks hard readable. It would be more user-friendly to have VM names in this window.

Version-Release number of selected component (if applicable):


How reproducible:100%


Steps to Reproduce:
1. Create affinity group A with VM1 and VM2 positive VM Affinity Rule.
2. Create affinity group B with VM1 and VM2 negative VM Affinity Rule.


Actual results:
Error while executing action: Affinity Group collision detected in unified affinity group of VMs:
e8b47916-4cb9-41f6-b114-00dc3e2d05b8,6d5009b3-e731-42b4-a6ad-8927a6d61238
and negative affinity group:
B
with VMs:
e8b47916-4cb9-41f6-b114-00dc3e2d05b8,6d5009b3-e731-42b4-a6ad-8927a6d61238

Expected results: VM names instead of IDs in the Error message


Additional info:

Comment 1 Michal Skrivanek 2022-04-11 09:34:45 UTC
trivial? please fix or close if no capacity

Comment 2 Polina 2022-08-02 18:54:34 UTC
the bug is on_qa , but still happens on the last available build . tested on ovirt-engine-4.5.2-0.3.el8ev.noarch

Comment 3 Nikolai Sednev 2022-08-11 17:44:17 UTC
Tested on Software Version:4.5.0.7-0.9.el8ev and successfully reproduced with attached print screen.
Will wait to resolve the engine upgrade issue of "nothing provides sos >= 4.2-20 needed by ovirt-log-collector-4.4.7-2.el8ev.noarch" to be able to upgrade the engine to latest ovirt-engine-setup-4.5.2.2-0.1.el8ev.noarch.

Comment 4 Nikolai Sednev 2022-08-11 17:45:29 UTC
Created attachment 1905022 [details]
Screen shot of reproduced issue from 11.08.22.

Comment 6 Nikolai Sednev 2022-08-15 10:26:37 UTC
Works fine on:
ovirt-engine-setup-4.5.2.2-0.1.el8ev.noarch
ovirt-hosted-engine-ha-2.5.0-1.el8ev.noarch
ovirt-hosted-engine-setup-2.6.5-1.1.el8ev.noarch
Linux 4.18.0-372.19.1.el8_6.x86_64 #1 SMP Mon Jul 18 11:14:02 EDT 2022 x86_64 x86_64 x86_64 GNU/Linux

Screenshot have been attached.

Comment 7 Nikolai Sednev 2022-08-15 10:27:42 UTC
Created attachment 1905545 [details]
Screenshot of properly working names for VMs and also for hosts.

Comment 8 Nikolai Sednev 2022-08-15 10:31:04 UTC
Created attachment 1905546 [details]
Only VMs without hosts conflicting affinity group fixed names

Comment 9 Sandro Bonazzola 2022-08-30 08:47:42 UTC
This bugzilla is included in oVirt 4.5.2 release, published on August 10th 2022.
Since the problem described in this bug report should be resolved in oVirt 4.5.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.


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