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

Fixes unfriendly message when adding network for unavailable provider #264

Merged

Conversation

slemrmartin
Copy link
Contributor

Create/Update/Delete Cloud Network caused unfriendly message when provider unavailable (fix)

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1510769
@miq-bot add_label bug
Cc @Ladas

Steps to reproduce:

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:
Unfriendly error message. ("undefined method `response' for <...> Did you mean? respond_to?")

Expected results:
Message should be friendly.

Unavailable provider: +/- Cloud Network returns friendly error

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1510769
@miq-bot miq-bot added the bug label Apr 3, 2018
@miq-bot
Copy link
Member

miq-bot commented Apr 3, 2018

Checked commit slemrmartin@0bbc144 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 Apr 3, 2018

+1 Looks like a fine fix, but I do have to wonder why it's even possible to choose unavailable providers :)

@slemrmartin
Copy link
Contributor Author

slemrmartin commented Apr 4, 2018

That's because provider can become unavailable (maitenance, some newly installed proxy, bad configured firewall etc.). And there is no validation of all providers each time you want add network (only for making select items list)

Copy link
Contributor

@Ladas Ladas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mansam mansam merged commit e093c84 into ManageIQ:master Apr 4, 2018
@simaishi
Copy link
Contributor

@slemrmartin Can this be gaprindashvili/yes ?

@slemrmartin
Copy link
Contributor Author

can be
@miq-bot add_label gaprindashvili/yes

@slemrmartin slemrmartin deleted the unavailable-network-provider-message branch April 12, 2018 07:22
simaishi pushed a commit that referenced this pull request Apr 12, 2018
…-message

Fixes unfriendly message when adding network for unavailable provider
(cherry picked from commit e093c84)

https://bugzilla.redhat.com/show_bug.cgi?id=1566570
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 48bead6e50cf6aab1404848fd6ff33e4360c61a1
Author: Samuel Lucidi <[email protected]>
Date:   Wed Apr 4 11:05:56 2018 -0400

    Merge pull request #264 from slemrmartin/unavailable-network-provider-message
    
    Fixes unfriendly message when adding network for unavailable provider
    (cherry picked from commit e093c84b6625827f6709f7e6bff515c035b7ac48)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1566570

@aufi aufi added this to the Sprint 83 Ending Apr 9, 2018 milestone Apr 24, 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