Bug 1650071 - Domain creation with wrong naming convention spinner never stop
Summary: Domain creation with wrong naming convention spinner never stop
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.11.0
Assignee: Harpreet Kataria
QA Contact: Ganesh Hubale
Red Hat CloudForms Documentation
URL:
Whiteboard:
: 1651473 (view as bug list)
Depends On:
Blocks: 1686055
TreeView+ depends on / blocked
 
Reported: 2018-11-15 09:54 UTC by Nikhil Dhandre
Modified: 2019-12-13 15:16 UTC (History)
10 users (show)

Fixed In Version: 5.11.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1686055 (view as bug list)
Environment:
Last Closed: 2019-12-13 15:16:39 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Nikhil Dhandre 2018-11-15 09:54:53 UTC
Description of problem:
-----------------------
I tried to create a domain with wrong naming convention as per domain instruction. It's showing me proper flash but getting an unbreakable spinner. 

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


How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. Navigate to Automation> Automate> Explorer
2. Try to create Domain with name `Dummy Domain` (I put space which is invalid)
3. Getting proper flash but spinner continue


Actual results:
---------------
unbreakable spinner

Expected results:
-----------------
It should stop after a proper flash message 

Additional info:

Comment 2 Nikhil Dhandre 2018-11-15 10:25:36 UTC
Same behavior for Namespace also...

Comment 4 CFME Bot 2018-11-26 22:51:33 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/1750b73d747854e0bfd3d3abda4ebaa6a32d9adb
commit 1750b73d747854e0bfd3d3abda4ebaa6a32d9adb
Author:     Harpreet Kataria <hkataria>
AuthorDate: Mon Nov 26 16:02:32 2018 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Nov 26 16:02:32 2018 -0500

    Stop spinner after rendering flash message

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

 app/controllers/miq_ae_class_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 5 CFME Bot 2018-11-27 14:52:25 UTC
New commit detected on ManageIQ/manageiq-ui-classic/hammer:

https://github.com/ManageIQ/manageiq-ui-classic/commit/9ed7c55f7da7efa40667f1ad1caac72d22f8aa31
commit 9ed7c55f7da7efa40667f1ad1caac72d22f8aa31
Author:     Dan Clarizio <dclarizi>
AuthorDate: Mon Nov 26 17:47:52 2018 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Mon Nov 26 17:47:52 2018 -0500

    Merge pull request #4990 from h-kataria/create_domain_spinner_fix

    Stop spinner after rendering flash message

    (cherry picked from commit 6b2f79a66bfada0045ebfc681db0f1e715f36605)

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

 app/controllers/miq_ae_class_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 6 Harpreet Kataria 2018-11-27 18:35:00 UTC
*** Bug 1651473 has been marked as a duplicate of this bug. ***

Comment 11 Ganesh Hubale 2019-01-03 07:13:56 UTC
Same issue occurring with namespaces while editing it's name.

Comment 13 Ganesh Hubale 2019-02-05 06:58:04 UTC
Adding this as regression because this works fine on 5.9z and also it's duplicate bug 1651473 already has 'Regression' keyword.

Comment 18 Ganesh Hubale 2019-05-03 10:57:33 UTC
Observation: No spinner observed while creating domain/namespace/class/method/instance with wrong name.

1] While editing domain/namespace etc with wrong name; it gives flash message as: "Error during 'save': Validation failed: MiqAeNamespace: Name may contain only alphanumeric and _ . - $ characters"
2] While adding class/method/instance with wrong name; it gives flash message as: "Error during 'add': Validation failed: MiqAeClass: Name may contain only alphanumeric and _ . - characters"


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