Description of problem: When testing registration with Cloudforms on 5.9 this should fail due to non-existant repos. Process is in a stuck state without any notification that the registration has failed Version-Release number of selected component (if applicable): 5.9.0.11 How reproducible: Always Steps to Reproduce: 1. Register CloudForms 5.9 Actual results: Stuck at enabling repos Expected results: Error message or Error out canceling the registration Additional info:
https://github.com/ManageIQ/linux_admin/pull/199
https://github.com/ManageIQ/manageiq/pull/17037
New commit detected on ManageIQ/linux_admin/master: https://github.com/ManageIQ/linux_admin/commit/b709532c0f1a019aa05265c1bc4dcc794790a5c2 commit b709532c0f1a019aa05265c1bc4dcc794790a5c2 Author: Nick Carboni <ncarboni> AuthorDate: Thu Feb 22 13:58:32 2018 -0500 Commit: Nick Carboni <ncarboni> CommitDate: Thu Feb 22 17:55:11 2018 -0500 Raise a SubscriptionManagerError from the SubscriptionManager class https://bugzilla.redhat.com/show_bug.cgi?id=1518801 lib/linux_admin/exceptions.rb | 2 ++ .../registration_system/subscription_manager.rb | 2 +- spec/subscription_manager_spec.rb | 18 ++++++++++++++++++ 3 files changed, 21 insertions(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/7d7840ec3e8568dd9d521519f8e2d82afb2bffa8 commit 7d7840ec3e8568dd9d521519f8e2d82afb2bffa8 Author: Nick Carboni <ncarboni> AuthorDate: Thu Feb 22 15:10:28 2018 -0500 Commit: Nick Carboni <ncarboni> CommitDate: Fri Feb 23 09:36:48 2018 -0500 Raise a notification when registration fails This adds a new notification type and creates a notification of this type when we fail to execute any of the steps of the Subscription Manager registration process. https://bugzilla.redhat.com/show_bug.cgi?id=1518801 Gemfile | 2 +- app/models/miq_server/update_management.rb | 4 ++++ db/fixtures/notification_types.yml | 5 +++++ spec/models/miq_server/update_management_spec.rb | 11 +++++++++++ 4 files changed, 21 insertions(+), 1 deletion(-)
Created attachment 1513684 [details] picture
https://github.com/ManageIQ/manageiq/pull/18290
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/c25631a85aca6cb0513415444e71697e3bb2d832 commit c25631a85aca6cb0513415444e71697e3bb2d832 Author: Nick Carboni <ncarboni> AuthorDate: Wed Dec 12 14:56:53 2018 -0500 Commit: Nick Carboni <ncarboni> CommitDate: Wed Dec 12 14:56:53 2018 -0500 Raise a notification for each repo that fails to enable Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1518801 app/models/miq_server/update_management.rb | 1 + db/fixtures/notification_types.yml | 5 + spec/models/miq_server/update_management_spec.rb | 23 +- 3 files changed, 25 insertions(+), 4 deletions(-)
New commit detected on ManageIQ/manageiq/hammer: https://github.com/ManageIQ/manageiq/commit/e5a0252945a13907ec76552daf1be194cb1f3759 commit e5a0252945a13907ec76552daf1be194cb1f3759 Author: Joe Rafaniello <jrafanie.github.com> AuthorDate: Wed Dec 12 15:57:02 2018 -0500 Commit: Joe Rafaniello <jrafanie.github.com> CommitDate: Wed Dec 12 15:57:02 2018 -0500 Merge pull request #18290 from carbonin/repo_enable_failure_notification Raise a notification for each repo that fails to enable (cherry picked from commit 1eb52eab66f61db2b207bbba60d19eca0826f0cd) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1518801 app/models/miq_server/update_management.rb | 1 + db/fixtures/notification_types.yml | 5 + spec/models/miq_server/update_management_spec.rb | 23 +- 3 files changed, 25 insertions(+), 4 deletions(-)
Version 5.10.0.31.20190108221820_a0968c8 When I changed two of the repos to invalid name and then clicked to register, I have got a flash message saying that one of the repos failed to register. I am satisfied with this result.