Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Friendly error message for HTTP 503 #293

Merged
merged 1 commit into from
May 21, 2018

Conversation

slemrmartin
Copy link
Contributor

@slemrmartin slemrmartin commented May 21, 2018

Handling HTML error responses from undercloud

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1566570
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1510769


Steps to reproduce:

In our Setup we usually use a deployment of 3 controllers and 2 compute nodes.
You need to login from undercloud to each controller and stop below services:

neutron-dhcp-agent.service                                           
neutron-openvswitch-agent.service                                               
neutron-server.service

In Networks -> Networks:

  1. Click Configure / Add a new Cloud Network
  2. Choose Network manager of unavailable provider (OpenStack / Ovirt only)
  3. Fill required fields and click Add

Actual results:
unexpected token at '<html><body><h1>503 Service Unavailable</h1> No server is available to handle this request. </body></html> '

Expected results:
503 Service Unavailable No server is available to handle this request.


@miq-bot add_label bug
@miq-bot add_label gaprindashvili/yes

Cc @Ladas, @aufi

@slemrmartin slemrmartin force-pushed the neutron-svc-off-message branch from 14dc88d to 0c34c37 Compare May 21, 2018 13:32
@miq-bot
Copy link
Member

miq-bot commented May 21, 2018

Checked commit slemrmartin@0c34c37 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@mansam
Copy link
Contributor

mansam commented May 21, 2018

Looks pretty good to me. Have you tested it to ensure it works?

@JPrause
Copy link
Member

JPrause commented May 21, 2018

@miq-bot add_label documentation

@JPrause
Copy link
Member

JPrause commented May 21, 2018

@miq-bot remove_label documentation

@JPrause
Copy link
Member

JPrause commented May 21, 2018

@miq-bot add_label blocker

@slemrmartin
Copy link
Contributor Author

@mansam Yes, I have. It was the hardest part to get credentials to this environment :)

@mansam
Copy link
Contributor

mansam commented May 21, 2018

@slemrmartin thanks!

@mansam mansam merged commit 1afb38f into ManageIQ:master May 21, 2018
simaishi pushed a commit that referenced this pull request May 21, 2018
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit a8b9d0b4cdacd837b03dc3589abcade2dfd5def3
Author: Samuel Lucidi <[email protected]>
Date:   Mon May 21 16:28:40 2018 -0400

    Merge pull request #293 from slemrmartin/neutron-svc-off-message
    
    Friendly error message for HTTP 503
    (cherry picked from commit 1afb38fd3b275826d3d88a16796c70b951b9f5c6)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1566570

@slemrmartin slemrmartin deleted the neutron-svc-off-message branch May 22, 2018 05:31
@aufi aufi added this to the Sprint 86 Ending May 21, 2018 milestone May 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants