Bug 1643458

Summary:	unable to edit tags on an infrastructure host
Product:	Red Hat CloudForms Management Engine	Reporter:	Felix Dewaleyne <fdewaley>
Component:	UI - OPS	Assignee:	Harpreet Kataria <hkataria>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Nandini Chandra <nachandr>
Severity:	medium	Docs Contact:	Red Hat CloudForms Documentation <cloudforms-docs>
Priority:	medium
Version:	5.9.3	CC:	dmetzger, fdewaley, hkataria, lavenel, mpovolny, nachandr, obarenbo, simaishi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.10.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	5.10.0.23	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1647188 (view as bug list)		Environment:
Last Closed:	2019-02-12 16:50:30 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1647188

Description Felix Dewaleyne 2018-10-26 10:02:51 UTC

Description of problem:
on this customer's system, trying to edit tags for any infrastructure host will result in various different errors and never be possible.

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

How reproducible:
all the time on this one region. not reproduced on others.

Steps to Reproduce:
1.log in as admin
2.go to compute > infrastructure > host
3.select any host
4. go to policy > edit tags
5. try to set any tag of any category

Actual results:
selecting a different category can result in an error. selecting a tag from the pre-selected category in another.

Expected results:
no error

Additional info:
only happens in that environment for the customer.

Comment 2 Dave Johnson 2018-10-26 10:05:06 UTC

Please assess the impact of this issue and update the severity accordingly.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#bug_severity for a reminder on each severity's definition.

If it's something like a tracker bug where it doesn't matter, please set the severity to Low.

Comment 10 Harpreet Kataria 2018-10-30 21:47:51 UTC

https://github.com/ManageIQ/manageiq-ui-classic/pull/4856

Comment 11 CFME Bot 2018-11-02 14:15:08 UTC

New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/36227b14d477cee5d48f1d59e772c38070e74895
commit 36227b14d477cee5d48f1d59e772c38070e74895
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue Oct 30 17:09:12 2018 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Oct 30 17:09:12 2018 -0400

    No need to load default search when user is on tagging screen.

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1643458

 app/controllers/application_controller.rb | 2 +-
 spec/controllers/application_controller_spec.rb | 18 +
 2 files changed, 19 insertions(+), 1 deletion(-)

Comment 12 CFME Bot 2018-11-02 14:15:32 UTC

New commit detected on ManageIQ/manageiq-ui-classic/hammer:

https://github.com/ManageIQ/manageiq-ui-classic/commit/7170c37638c87918998f3eb0a70cbca739477515
commit 7170c37638c87918998f3eb0a70cbca739477515
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Nov  1 06:55:42 2018 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Nov  1 06:55:42 2018 -0400

    Merge pull request #4856 from h-kataria/dont_load_default_seacrh_on_tagging_form

    No need to load default search when user is on tagging screen.

    (cherry picked from commit 80c1c665f2579a1feb5ac53a14a5b5ca5d7c0bb2)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1643458

 app/controllers/application_controller.rb | 2 +-
 spec/controllers/application_controller_spec.rb | 18 +
 2 files changed, 19 insertions(+), 1 deletion(-)

Comment 13 CFME Bot 2018-11-02 14:15:50 UTC

New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/36227b14d477cee5d48f1d59e772c38070e74895
commit 36227b14d477cee5d48f1d59e772c38070e74895
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue Oct 30 17:09:12 2018 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Oct 30 17:09:12 2018 -0400

    No need to load default search when user is on tagging screen.

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1643458

 app/controllers/application_controller.rb | 2 +-
 spec/controllers/application_controller_spec.rb | 18 +
 2 files changed, 19 insertions(+), 1 deletion(-)

Comment 14 CFME Bot 2018-11-02 14:16:25 UTC

New commit detected on ManageIQ/manageiq-ui-classic/hammer:

https://github.com/ManageIQ/manageiq-ui-classic/commit/7170c37638c87918998f3eb0a70cbca739477515
commit 7170c37638c87918998f3eb0a70cbca739477515
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Nov  1 06:55:42 2018 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Nov  1 06:55:42 2018 -0400

    Merge pull request #4856 from h-kataria/dont_load_default_seacrh_on_tagging_form

    No need to load default search when user is on tagging screen.

    (cherry picked from commit 80c1c665f2579a1feb5ac53a14a5b5ca5d7c0bb2)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1643458

 app/controllers/application_controller.rb | 2 +-
 spec/controllers/application_controller_spec.rb | 18 +
 2 files changed, 19 insertions(+), 1 deletion(-)

Comment 15 Harpreet Kataria 2018-11-05 21:46:59 UTC

*** Bug 1646712 has been marked as a duplicate of this bug. ***

Comment 17 Nandini Chandra 2018-11-07 21:36:23 UTC

Verified in 5.10.0.23