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

Rescue error messages when executing retire action #4874

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Nov 5, 2018

Issue:
There is not rescued exception for error raised on backend during execution of retire operation.

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1579017

@miq-bot add-label gaprindashvili/yes, hammer/yes, services, bug

BEFORE: there is no any confirmation or error messages when saving new retirement date

AFTER: (if remote server is down)
before1

@miq-bot
Copy link
Member

miq-bot commented Nov 5, 2018

Checked commit yrudman@8ec0dd5 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. 🍪

@yrudman
Copy link
Contributor Author

yrudman commented Nov 15, 2018

Depends on: ManageIQ/manageiq#18210

@yrudman yrudman changed the title [WIP] Show confirmation/error messages when changing retirement date for Service via Centralized Administration Show confirmation/error messages when changing retirement date for Service via Centralized Administration Nov 16, 2018
@miq-bot miq-bot removed the wip label Nov 16, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Nov 16, 2018

@miq-bot add-label blocker

@yrudman yrudman changed the title Show confirmation/error messages when changing retirement date for Service via Centralized Administration Rescue error messages when executing retire action Nov 19, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Nov 19, 2018

\cc @h-kataria

@mzazrivec mzazrivec self-assigned this Nov 19, 2018
@mzazrivec mzazrivec added this to the Sprint 99 Ending Nov 19, 2018 milestone Nov 19, 2018
@mzazrivec mzazrivec merged commit 5518e01 into ManageIQ:master Nov 19, 2018
simaishi pushed a commit that referenced this pull request Nov 19, 2018
…-centralized-administration

Rescue error messages when executing retire action

(cherry picked from commit 5518e01)

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

Hammer backport details:

$ git log -1
commit a1a211f5c545b4b5c62e8d1c105135cac0dab5ab
Author: Milan Zázrivec <[email protected]>
Date:   Mon Nov 19 15:17:15 2018 +0100

    Merge pull request #4874 from yrudman/support-service-retierement-via-centralized-administration
    
    Rescue error messages when executing retire action
    
    (cherry picked from commit 5518e013fadd629e871acc4e8b213eb5f5fee8b0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1579017

simaishi pushed a commit that referenced this pull request Nov 19, 2018
…-centralized-administration

Rescue error messages when executing retire action

(cherry picked from commit 5518e01)

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

Gaprindashvili backport details:

$ git log -1
commit d2e317a60cf572ebe4b0c1bedc02e6a9a1d5c67c
Author: Milan Zázrivec <[email protected]>
Date:   Mon Nov 19 15:17:15 2018 +0100

    Merge pull request #4874 from yrudman/support-service-retierement-via-centralized-administration
    
    Rescue error messages when executing retire action
    
    (cherry picked from commit 5518e013fadd629e871acc4e8b213eb5f5fee8b0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1648991

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.

4 participants