Bug 1615632

Summary: [v2v] Transformation host tagging behaves erratic: blocks the UI
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Kedar Kulkarni <kkulkarn>
Severity: medium Docs Contact:
Priority: high    
Version: 5.9.4CC: agrare, bthurber, dclarizi, dmetzger, hkataria, kkulkarn, lavenel, mpovolny, obarenbo, smallamp
Target Milestone: GAKeywords: ZStream
Target Release: 5.9.4   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: v2v
Fixed In Version: 5.9.4.6 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1614437 Environment:
Last Closed: 2018-09-05 12:56:01 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1614437    
Bug Blocks:    
Attachments:
Description Flags
ScreenGIF none

Comment 1 CFME Bot 2018-08-13 22:47:22 UTC
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/5387925e242a735839eda4ce257647c46fdb1566
commit 5387925e242a735839eda4ce257647c46fdb1566
Author:     Dan Clarizio <dclarizi>
AuthorDate: Fri Aug 10 14:02:40 2018 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Fri Aug 10 14:02:40 2018 -0400

    Merge pull request #4452 from h-kataria/tagging_second_drop_down_fix

    Added spinner after Category is selected on tagging screen
    (cherry picked from commit e2bd0a9843d0e51a8fe0a6af5c47745394b3572d)

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

 app/controllers/application_controller/tags.rb | 52 +-
 app/views/layouts/_tag_edit.html.haml | 2 +-
 2 files changed, 31 insertions(+), 23 deletions(-)

Comment 2 Kedar Kulkarni 2018-08-17 15:04:20 UTC
On 5.9.4.4 this is still an issue. It keeps freezing. I will attach a GIF.

Comment 3 Kedar Kulkarni 2018-08-17 15:04:53 UTC
Created attachment 1476640 [details]
ScreenGIF

Comment 8 Sudhir Mallamprabhakara 2018-08-20 20:48:07 UTC
KK - Can you provide a reproducer??

Comment 9 Sudhir Mallamprabhakara 2018-08-20 20:59:12 UTC
maybe a browser issue.. I am not seeing it on Safari either.

Comment 10 Kedar Kulkarni 2018-08-20 22:29:24 UTC
Sudhir, it is same as shown in attached video. I do get it in Chrome and Firefox as well sometimes.

Comment 11 CFME Bot 2018-08-21 18:34:04 UTC
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/91398ffcea19f105b2e91a5ba5cbc404ec94c259
commit 91398ffcea19f105b2e91a5ba5cbc404ec94c259
Author:     Dan Clarizio <dclarizi>
AuthorDate: Tue Aug 21 11:34:04 2018 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Tue Aug 21 11:34:04 2018 -0400

    Merge pull request #4499 from h-kataria/tagging_edit_double_observer_transaction_fix

    Removed `data-miq_observe` from drop down
    (cherry picked from commit a46f0712cc8b8ca5ca950b3ee9bdb8a827a215c1)

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

 app/views/layouts/_tag_edit.html.haml | 1 -
 1 file changed, 1 deletion(-)

Comment 12 Kedar Kulkarni 2018-08-22 17:38:22 UTC
On CFME Build version 5.9.4.5 I tried to tag hosts on 2 different appliances and they did work properly, did not cause the blocking of UI as it used to do. So marking this verified.

Comment 13 CFME Bot 2018-08-23 19:42:42 UTC
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/ffe20d38326d94136ee5ce687a385fb0c1c12aeb
commit ffe20d38326d94136ee5ce687a385fb0c1c12aeb
Author:     Dan Clarizio <dclarizi>
AuthorDate: Thu Aug 23 14:58:12 2018 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Thu Aug 23 14:58:12 2018 -0400

    Merge pull request #4533 from h-kataria/fixed_tag_drop_down_transactions

    Added spinner after Value is selected on tagging screen
    (cherry picked from commit 390dc915991559d1fb177b814991e066fd9f15a9)

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

 app/controllers/application_controller/tags.rb | 2 +-
 app/views/layouts/_tag_edit_cat_tags.html.haml | 3 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

Comment 15 Kedar Kulkarni 2018-08-24 20:17:42 UTC
On 5.9.4.6 UI behaves normal, no more blocking issues while tagging.  Also spinner is present as mentioned in comment 13.