Created attachment 1276402 [details] screen showing error Description of problem:bad error message when adding playbook catalog item while embedded ansible is disabled Version-Release number of selected component (if applicable):5.8.0.13 How reproducible:100% Steps to Reproduce: 1.provision appliance 2.enable embedded ansible 3.add repo 4.disable embedded ansible 5.try to add a catalog item with playbook from repo Actual results: Could not create Service Template - <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"> <html><head> <title>503 Service Unavailable</title> </head><body> <h1>Service Unavailable</h1> <p>The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later.</p> </body></html> Expected results: Error message such as 'Embedded ansible is disabled' Additional info:
Drew, I am thinking of a few ways we could approach this since it could be the result more than just the role disabled. Suggestions: 1) Override the with_provider_connection method for the embedded provider to check if the role is enabled. Raise error if it is not. This would help in many areas that communicate with the provider, includes authoring and ordering of the service. 2) Look to detect and raise a cleaner error message if the gem if the Tower server is unavailable. Suggest work with Brandon if needed.
https://github.com/ManageIQ/manageiq/pull/15045
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/6b99c98811f1723fc0424ca7dd0dc81ee45d3b64 commit 6b99c98811f1723fc0424ca7dd0dc81ee45d3b64 Author: Drew Bomhof <dbomhof> AuthorDate: Wed May 10 14:05:07 2017 -0400 Commit: Drew Bomhof <dbomhof> CommitDate: Wed May 10 15:00:30 2017 -0400 Modify specs to set the embedded role https://bugzilla.redhat.com/show_bug.cgi?id=1448186 app/models/manageiq/providers/embedded_ansible/provider.rb | 4 ++-- .../automation_manager/configuration_script_spec.rb | 4 ++++ .../embedded_ansible/automation_manager/credential_spec.rb | 4 ++++ .../refresher_configuration_script_source_spec.rb | 4 ++++ .../providers/embedded_ansible/automation_manager/refresher_spec.rb | 4 ++++ .../embedded_ansible/automation_manager/refresher_v2_spec.rb | 4 ++++ spec/models/manageiq/providers/embedded_ansible/provider_spec.rb | 4 +--- spec/support/evm_spec_helper.rb | 6 ++++++ 8 files changed, 29 insertions(+), 5 deletions(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/27dc7f85949836e9f774be4a37a913e22314f409 commit 27dc7f85949836e9f774be4a37a913e22314f409 Author: Drew Bomhof <dbomhof> AuthorDate: Tue May 9 15:20:49 2017 -0400 Commit: Drew Bomhof <dbomhof> CommitDate: Wed May 10 13:33:53 2017 -0400 Check the Embedded Ansible role is on 1. Raise an error if an attempt at raw_connect is made if the role is not enabled 2. Continue on as normal with the connect if the role is enabled https://bugzilla.redhat.com/show_bug.cgi?id=1448186 .../manageiq/providers/embedded_ansible/provider.rb | 10 ++++++++++ .../providers/embedded_ansible/provider_spec.rb | 19 +++++++++++++++++++ 2 files changed, 29 insertions(+)
Verified in 5.9.0.2