Bug 1443520

Summary: Error Message when adding a Containers Provider with SSL/Wrong Port
Product: Red Hat CloudForms Management Engine Reporter: Pavel Zagalsky <pzagalsk>
Component: UI - OPSAssignee: Yaacov Zamir <yzamir>
Status: CLOSED CURRENTRELEASE QA Contact: Shalom Naim <snaim>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 5.8.0CC: cpelland, dclarizi, fsimonce, hkataria, jhardy, juwatts, mpovolny, obarenbo
Target Milestone: GAKeywords: TestOnly
Target Release: 5.9.0Flags: snaim: automate_bug+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: container
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1445002 (view as bug list) Environment:
Last Closed: 2018-03-06 15:13:19 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Container Management Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1445002    
Attachments:
Description Flags
ValidationMessage
none
AddProviderError
none
validation_js_error
none
validation_log
none
add_log none

Description Pavel Zagalsky 2017-04-19 12:16:08 UTC
Created attachment 1272587 [details]
ValidationMessage

Description of problem:
When attempting to add a Containers Provider with an incorrect port the app crashes

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

How reproducible:
Every time

Steps to Reproduce:
1. Go to Compute --> Containers --> Providers
2. Attempt to add a Container Provider with SSL as the Security Protocol and intentionally incorrect SSL Port ie 13000
3.

Actual results:
1. There's a broken message when attempting to validate the provider
2. The Add button is available for pressing and pressing it results in an Error message

Expected results:
When a user attempts to add a Provider with an incorrect port he should receieve a better error message when validating and the Add button should remain disabled

Additional info:
Logs and screenshots atttached

Comment 2 Pavel Zagalsky 2017-04-19 12:16:40 UTC
Created attachment 1272588 [details]
AddProviderError

Comment 3 Pavel Zagalsky 2017-04-19 12:17:14 UTC
Created attachment 1272589 [details]
validation_js_error

Comment 4 Pavel Zagalsky 2017-04-19 12:17:41 UTC
Created attachment 1272590 [details]
validation_log

Comment 5 Pavel Zagalsky 2017-04-19 12:18:07 UTC
Created attachment 1272591 [details]
add_log

Comment 6 Yaacov Zamir 2017-04-20 12:56:16 UTC
Pavel, the real bug here is that this pass validation, so we will have to fix both things :-)

The first thing will be stopping validation with wrong port, this will also fix this bug ... and then fix this one too, just in case.

Comment 7 Yaacov Zamir 2017-04-24 11:13:03 UTC
Submited upstream:
https://github.com/ManageIQ/manageiq-ui-classic/pull/1126

Comment 8 Yaacov Zamir 2017-04-24 14:41:51 UTC
Merged upstream:
https://github.com/ManageIQ/manageiq-ui-classic/pull/1126

Comment 10 Pavel Zagalsky 2017-10-29 16:06:33 UTC
Verified