Red Hat Bugzilla – Bug 1312046
Adding a duplicate Foreman provider name never redirects to the flash message
Last modified: 2017-08-29 21:36:00 EDT
New commit detected on cfme/5.5.z:
Merge: fe3d85a 0232849
Author: Dan Clarizio <email@example.com>
AuthorDate: Tue Mar 1 15:33:16 2016 -0500
Commit: Dan Clarizio <firstname.lastname@example.org>
CommitDate: Tue Mar 1 15:33:16 2016 -0500
Merge branch '5.5.z_duplicate_foreman_provider' into '5.5.z'
Move replace_right_cell out of the errors conditional
Upstream BZ https://bugzilla.redhat.com/show_bug.cgi?id=1310895
Needed to modify instance_variable, flash text for the tests to pass.
See merge request !825
app/controllers/provider_foreman_controller.rb | 2 +-
spec/controllers/provider_foreman_controller_spec.rb | 13 +++++++++++++
2 files changed, 14 insertions(+), 1 deletion(-)
Failed QA on 184.108.40.206 - 220.127.116.11.20160318152106_92d2d67.
The provider cannot be added twice anymore (this part is fixed) but it spawns 4 flash errors with sort of duplicate and incorrect text.
See attachment (left image is for configuration managers)
Also see related https://bugzilla.redhat.com/show_bug.cgi?id=1317892
Configuration Manager providers require additional field validation than infra providers, that being said, the resulting four flash errors are correct for that provider.
Seeing that the main bug (system hangs) was solved and is merged, we would like to see a new BZ created to address the UI changes.
We would have to revamp how validation errors are returned when the reported event is attempted with this provider type.
Ok, closing as verified in 18.104.22.168.
There is a separate BZ for the error mentioned above.
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.