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

Raise a notification when registration fails #17037

Merged

Conversation

carbonin
Copy link
Member

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

Requires ManageIQ/linux_admin#199

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
@carbonin carbonin force-pushed the add_notification_on_failed_registration branch from f4ffcbe to 7d7840e Compare February 23, 2018 14:55
@carbonin carbonin changed the title [WIP] Raise a notification when registration fails Raise a notification when registration fails Feb 23, 2018
@carbonin carbonin removed the wip label Feb 23, 2018
@carbonin carbonin force-pushed the add_notification_on_failed_registration branch from 0ca77a8 to 7d7840e Compare February 23, 2018 16:12
@miq-bot
Copy link
Member

miq-bot commented Feb 23, 2018

Checked commits carbonin/manageiq@7d7840e~...0ca77a8 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 0 offenses detected
Everything looks fine. 🏆

@bdunne bdunne merged commit d4e6e61 into ManageIQ:master Feb 26, 2018
@bdunne bdunne added this to the Sprint 80 Ending Feb 26, 2018 milestone Feb 26, 2018
simaishi pushed a commit that referenced this pull request Mar 7, 2018
…gistration

Raise a notification when registration fails
(cherry picked from commit d4e6e61)

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

simaishi commented Mar 7, 2018

Gaprindashvili backport details:

$ git log -1
commit ab47e64064320f5dcae3ce962def82d2da6f11ce
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Mon Feb 26 18:27:18 2018 -0500

    Merge pull request #17037 from carbonin/add_notification_on_failed_registration
    
    Raise a notification when registration fails
    (cherry picked from commit d4e6e61438bb844906b8a9be6a58e4b8219e7d89)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1552802

@carbonin carbonin deleted the add_notification_on_failed_registration branch August 16, 2019 17:27
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