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

Remove middleware reports #16712

Merged
merged 1 commit into from
Dec 22, 2017
Merged

Conversation

abonas
Copy link
Member

@abonas abonas commented Dec 21, 2017

Remove middleware reports from miq per BZ 1526791

@abonas
Copy link
Member Author

abonas commented Dec 21, 2017

@miq-bot add_label providers/middleware

@abonas
Copy link
Member Author

abonas commented Dec 21, 2017

@miq-bot add_label gaprindashvili/yes

@abonas
Copy link
Member Author

abonas commented Dec 21, 2017

@ManageIQ/core-hawkular please review

@miq-bot
Copy link
Member

miq-bot commented Dec 21, 2017

Checked commit abonas@c202bbd with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

Copy link
Member

@israel-hdez israel-hdez left a comment

Choose a reason for hiding this comment

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

All gone!

@chessbyte chessbyte self-assigned this Dec 22, 2017
@chessbyte chessbyte merged commit 031e09b into ManageIQ:master Dec 22, 2017
@chessbyte chessbyte added this to the Sprint 76 Ending Jan 1, 2018 milestone Dec 22, 2017
simaishi pushed a commit that referenced this pull request Jan 3, 2018
Remove middleware reports
(cherry picked from commit 031e09b)
@simaishi
Copy link
Contributor

simaishi commented Jan 3, 2018

Gaprindashvili backport details:

$ git log -1
commit 5feb9a329be6dd187e03615217ed2da291fcfd93
Author: Oleg Barenboim <chessbyte@gmail.com>
Date:   Thu Dec 21 22:12:24 2017 -0500

    Merge pull request #16712 from abonas/removeMiddlewareReports
    
    Remove middleware reports
    (cherry picked from commit 031e09b813af6cc04edcbb91635dccb90ada1c13)

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