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

[CLOUDSTACK-9858] Retirement of midonet plugin (build disabling) #2036

Merged
merged 1 commit into from
Apr 17, 2017

Conversation

rafaelweingartner
Copy link
Member

Recently there have been two threads asking and discussing the “midonet” integration with Apache CloudStack (ACS) [1-2].

After quite some discussions, we noticed that despite having some people willing to use it, the plugin has never been fully developed by its vendor (Midokura). Further, nobody else has put the effort on fully testing and finishing its implementation. It seems that the plugin was incorporated into our code base without being fully finished. Moreover, I have asked around at the Midonet community, and the java client they use has changed quite a bit from the one we use.

It begs the question if it does not work, why do we advertise such integration? [3].
Following the component retirement process defined in [4], a vote thread was started in [5]. The community decided to retire this Midonet plugin. This task represents the first step of the retirement, which is the disabling of the plugin in CloudStack`s build process.

[1] http://cloudstack.markmail.org/thread/qyedle5jb2c34gsc#query:+page:1+mid:xn2zq2v3eim5vl2q+state:results
[2] http://cloudstack.markmail.org/message/rewzk4v7dgzpsxkm?q=midonet+order:date-backward&page=1#query:midonet%20order%3Adate-backward+page:1+mid:i563khxlginf6smg+state:results
[3] http://docs.cloudstack.apache.org/en/latest/networking/midonet.html
[4] https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=68720798
[5] http://markmail.org/message/qigrtfirwnmct4hr

@wido
Copy link
Contributor

wido commented Apr 12, 2017

LGTM

@rafaelweingartner
Copy link
Member Author

I tried to access the Jenkins details to see the checking problems, but I am getting a 404.
Am I missing something?

@DaanHoogland
Copy link
Contributor

LGTM

@wido wido merged commit b8dd08c into apache:master Apr 17, 2017
@rohityadavcloud
Copy link
Member

-1
This was merged without packaging/regression testing.
We've previously agreed with a regression test run, this has broken master and packaging.

rohityadavcloud added a commit to shapeblue/cloudstack that referenced this pull request Apr 18, 2017
This fixes packaging/build regression from apache#2036

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
abhinandanprateek pushed a commit to shapeblue/cloudstack that referenced this pull request Apr 26, 2017
This fixes packaging/build regression from apache#2036

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
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