New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=cf17bb340fe922fe7259e629a239b80ccbb1e480 commit cf17bb340fe922fe7259e629a239b80ccbb1e480 Merge: fe3d85a 0232849 Author: Dan Clarizio <dclarizi> AuthorDate: Tue Mar 1 15:33:16 2016 -0500 Commit: Dan Clarizio <dclarizi> 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 PR https://github.com/ManageIQ/manageiq/pull/7015 https://bugzilla.redhat.com/show_bug.cgi?id=1312046 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 5.5.3.2 - 5.5.3.2.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
Hi Jan, 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 5.5.3.2. 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. https://access.redhat.com/errata/RHBA-2016:0616