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 API #255

Merged
merged 1 commit into from
Jan 2, 2018
Merged

Conversation

cfcosta
Copy link

@cfcosta cfcosta commented Dec 26, 2017

Middleware Manager is being phased out, so is the API.

@miq-bot add_label gaprindashvili/yes

@miq-bot
Copy link
Member

miq-bot commented Dec 26, 2017

Checked commit cfcosta@cab292b 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. 🏆

@abonas
Copy link
Member

abonas commented Dec 28, 2017

@miq-bot add_label providers/middleware

@abonas
Copy link
Member

abonas commented Dec 28, 2017

@cfcosta if this is an "as is" revert of the previous commit that added the api, it's better (and more clear) to send a regular revert

@cfcosta
Copy link
Author

cfcosta commented Dec 28, 2017

@abonas not exactly, it is a revert of the whole PR merge. It's the cleanest way of doing it, but it squashes all commits from the PR into one.

@abonas
Copy link
Member

abonas commented Dec 28, 2017

OK thanks

@abonas
Copy link
Member

abonas commented Jan 2, 2018

@abellotti @imtayadeway

@abonas
Copy link
Member

abonas commented Jan 2, 2018

test failures seem unrelated from a quick glance

@imtayadeway
Copy link
Contributor

@cfcosta agreeing with @abonas - was this done by hand? If you do:

git revert -m 1 <sha of the merge commit>

It will add useful documentation for future git historians by linking to the original merge commit, and also providing details of any conflicts that needed to be fixed by hand.

@abellotti abellotti added this to the Sprint 77 Ending Jan 15, 2018 milestone Jan 2, 2018
@abellotti
Copy link
Member

ping @cfcosta when you get a chance to push the git revert commit instead, Thanks!!

…dpoints"

This reverts commit 02f47ff, reversing
changes made to ef8a010.
@cfcosta cfcosta force-pushed the remove-middleware-api branch from cab292b to 294567f Compare January 2, 2018 19:48
@cfcosta
Copy link
Author

cfcosta commented Jan 2, 2018

@imtayadeway @abellotti it was not done by hand, I did the git revert -m 1 but reworded the commit to provide more context. Either way, done!

Copy link
Contributor

@imtayadeway imtayadeway left a comment

Choose a reason for hiding this comment

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

Thanks @cfcosta !

@imtayadeway imtayadeway self-assigned this Jan 2, 2018
@imtayadeway imtayadeway merged commit f4f2ca7 into ManageIQ:master Jan 2, 2018
simaishi pushed a commit that referenced this pull request Jan 3, 2018
Remove Middleware API
(cherry picked from commit f4f2ca7)
@simaishi
Copy link
Contributor

simaishi commented Jan 3, 2018

Gaprindashvili backport details:

$ git log -1
commit 5d89abd08f0b1a90dd109b3a1d6601ede7db8f45
Author: Tim Wade <imtayadeway@users.noreply.github.com>
Date:   Tue Jan 2 13:03:57 2018 -0800

    Merge pull request #255 from cfcosta/remove-middleware-api
    
    Remove Middleware API
    (cherry picked from commit f4f2ca76db3584f5e5a665c708c2d8bff169479c)

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