Bug 1195637

Summary: UI should report errors to the user when External AD UserGroup addition fails
Product: Red Hat Satellite Reporter: Kedar Bidarkar <kbidarka>
Component: WebUIAssignee: Marek Hulan <mhulan>
WebUI sub component: Foreman QA Contact: Kedar Bidarkar <kbidarka>
Status: CLOSED ERRATA Docs Contact:
Severity: high    
Priority: high CC: bkearney, jmontleo, omaciel
Version: 6.1.0Keywords: Triaged
Target Milestone: Unspecified   
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/9592
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 05:27:14 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Kedar Bidarkar 2015-02-24 08:36:58 UTC
Description of problem:

UI should report errors to the user when External AD UserGroup addition fails.


Processing by UsergroupsController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"w4GDcBCBEdfi+TG0Lg7cPljNH4/96go+SS0giOfYNdY=", "usergroup"=>{"name"=>"foreman", "user_ids"=>["", "5", "", "4", "", "6", ""], "admin"=>"0", "role_ids"=>["", "11"], "external_usergroups_attributes"=>{"0"=>{"_destroy"=>"false", "name"=>"kedargrp", "auth_source_id"=>"4", "id"=>"8"}, "1"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3", "id"=>"7"}, "new_external_usergroups"=>{"_destroy"=>"false", "name"=>"", "auth_source_id"=>"3"}, "new_1424766851040"=>{"_destroy"=>"false", "name"=>"hdhdhd", "auth_source_id"=>"3"}}}, "id"=>"3-foreman"}
Failed to save: Name is not found in the authentication source

OR

Processing by UsergroupsController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"w4GDcBCBEdfi+TG0Lg7cPljNH4/96go+SS0giOfYNdY=", "usergroup"=>{"name"=>"foreman", "user_ids"=>["", "5", "", "4", "", "6", ""], "admin"=>"0", "role_ids"=>["", "11"], "external_usergroups_attributes"=>{"0"=>{"_destroy"=>"false", "name"=>"kedargrp", "auth_source_id"=>"4", "id"=>"8"}, "1"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3", "id"=>"7"}, "new_external_usergroups"=>{"_destroy"=>"false", "name"=>"", "auth_source_id"=>"3"}, "new_1424767137133"=>{"_destroy"=>"false", "name"=>"foobargroup", "auth_source_id"=>"3"}}}, "id"=>"3-foreman"}
Failed to save: Name has already been taken



Version-Release number of selected component (if applicable):
sat6.1 Beta snap3

How reproducible:


Steps to Reproduce:
1. create usergroup 
2. add an invalid external AD UserGroup.
3.

Actual results:
No error message seen on the UI.

foreman/production.log shows 

a) "Failed to save: Name is not found in the authentication source"
b) Failed to save: Name has already been taken

Expected results:
Shouls display the actual error on the UI, 
a) "Name is not found in the authentication source"
b) Name has already been taken

Additional info:

Comment 2 Marek Hulan 2015-03-01 14:24:10 UTC
Created redmine issue http://projects.theforeman.org/issues/9592 from this bug

Comment 3 Marek Hulan 2015-03-01 14:38:34 UTC
The expected behaviour here is the following:
if you're adding new external user group it's validated (e.g. it's name is not taken, it's reachable in external auth source)
if you're saving user group which has some existing external user groups, they are not validated, as this could prevent you from saving user group changes just because external service is temporary down
if you're saving user group and add new external user group, then it's also supposed to run all validations on this new external user group

It already adds error messages and it also marks the field red, but the issue was that the list of external user groups fields was folded, so the fix is to unfold the form in case there are any errors.

Comment 4 Bryan Kearney 2015-03-01 19:05:15 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9592 has been closed
-------------
Marek Hulán
Applied in changeset commit:8e170d2186edc805e440e587438a4cf8207facd9.

Comment 7 Kedar Bidarkar 2015-03-09 11:53:26 UTC
now we get the below error message from the UI

a) has already been taken
b) is not found in the authentication source

VERIFIED with sat6.1 beta snap5

Comment 8 Bryan Kearney 2015-08-11 13:19:33 UTC
This bug is slated to be released with Satellite 6.1.

Comment 9 errata-xmlrpc 2015-08-12 05:27:14 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:1592