cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Will Stevens <williamstev...@gmail.com>
Subject Re: midonet-client and Guava dependency conflict
Date Fri, 10 Mar 2017 01:01:24 GMT
I will review. Thanks for digging that up. Haha. You know every issue now
because of your cleanup. :)

On Mar 9, 2017 7:51 PM, "Rafael Weingärtner" <rafaelweingartner@gmail.com>
wrote:

> https://issues.apache.org/jira/browse/CLOUDSTACK-8677
>
> On Thu, Mar 9, 2017 at 6:48 PM, Will Stevens <williamstevens@gmail.com>
> wrote:
>
> > I don't remember that conversation, but I will try to track it down.
> Thx...
> >
> > On Mar 9, 2017 6:39 PM, "Rafael Weingärtner" <
> rafaelweingartner@gmail.com>
> > wrote:
> >
> > > I recall a thread discussing some call home functionalities a long time
> > > ago; so, we could get some information about ACS usage. Does anyone
> know
> > > how that thread ended?
> > >
> > > The use o ACS plugin could be something trackable by a call home
> feature.
> > >
> > > On Thu, Mar 9, 2017 at 5:55 PM, Will Stevens <wstevens@cloudops.com>
> > > wrote:
> > >
> > > > We have had similar conversations many times recently.  Unfortunately
> > we
> > > > have no way to track IF people are using different plugins, so it
> makes
> > > it
> > > > really hard to know if people will expect it to be there if they
> > > upgrade...
> > > >
> > > > If anyone has ideas for how we can potentially find a way to track
> > that,
> > > we
> > > > should probably start a thread around that.
> > > >
> > > > *Will STEVENS*
> > > > Lead Developer
> > > >
> > > > <https://goo.gl/NYZ8KK>
> > > >
> > > > On Thu, Mar 9, 2017 at 2:36 PM, Rafael Weingärtner <
> > > > rafaelweingartner@gmail.com> wrote:
> > > >
> > > > > Agree with you, but I think a nice talk with people using it to ask
> > for
> > > > > help might be a good idea.
> > > > >
> > > > > Sometimes they are not aware of these situations.
> > > > >
> > > > > On Thu, Mar 9, 2017 at 2:23 PM, Daan Hoogland <
> > daan.hoogland@gmail.com
> > > >
> > > > > wrote:
> > > > >
> > > > > > If we have to lay a burdon of fixing before release on people
> that
> > > > > > don't use it, i would say no. You use it, you maintain it. I
am
> > happy
> > > > > > to charge money to whoever wants to not maintain what they use.
> > > > > >
> > > > > > On Thu, Mar 9, 2017 at 7:11 PM, Rafael Weingärtner
> > > > > > <rafaelweingartner@gmail.com> wrote:
> > > > > > > Daan, before removing anything, I think we should check
if
> there
> > > are
> > > > > > people
> > > > > > > using it, right?
> > > > > > >
> > > > > > > On Thu, Mar 9, 2017 at 11:08 AM, Daan Hoogland <
> > > > > daan.hoogland@gmail.com>
> > > > > > > wrote:
> > > > > > >
> > > > > > >> I was about to reply along those lines. As you brought
it up,
> we
> > > are
> > > > > > >> now considering it. If the fix is easy I'd say let
it stay
> till
> > > the
> > > > > > >> next problem but it is ot the first time mido bugs
us.
> > > > > > >>
> > > > > > >> On Thu, Mar 9, 2017 at 5:07 PM, Simon Weller <sweller@ena.com
> >
> > > > wrote:
> > > > > > >> > So this brings up a good discussion point. As
Jeff points
> out,
> > > the
> > > > > > >> Midonet plugin hasn't been actively supported for almost
5
> > years.
> > > At
> > > > > > what
> > > > > > >> point do we consider retiring unsupported plugins?
> > > > > > >> >
> > > > > > >> >
> > > > > > >> > - Si
> > > > > > >> >
> > > > > > >> >
> > > > > > >> > ________________________________
> > > > > > >> > From: Jeff Hair <jeff@greenqloud.com>
> > > > > > >> > Sent: Thursday, March 9, 2017 9:43 AM
> > > > > > >> > To: dev@cloudstack.apache.org
> > > > > > >> > Subject: Re: midonet-client and Guava dependency
conflict
> > > > > > >> >
> > > > > > >> > After doing some more digging, I have confirmed
the
> following:
> > > > > > >> >
> > > > > > >> >    - The midonet plugin is using the Maven Shade
plugin to
> > put a
> > > > > > bunch of
> > > > > > >> >    dependencies into itself.
> > > > > > >> >    - The plugin hosted in this repository was
last updated
> in
> > > > 2013.
> > > > > > >> >    - Most importantly: removing all the guava
stuff out of
> the
> > > > > midonet
> > > > > > >> >    plugin fixes this issue.
> > > > > > >> >
> > > > > > >> > I have not had any success in applying
> > > > > > >> > https://github.com/openwide-java/tomcat-classloader-ordered
> > to
> > > > get
> > > > > > >> Tomcat
> > > > > > >> > [https://avatars1.githubusercontent.com/u/1385131?v=3&s=400
> > > > > ]<https://
> > > > > > >> github.com/openwide-java/tomcat-classloader-ordered>
> > > > > > >> >
> > > > > > >> > GitHub - openwide-java/tomcat-classloader-ordered:
A ...<
> > > > > > >> https://github.com/openwide-java/tomcat-classloader-ordered>
> > > > > > >> > github.com
> > > > > > >> > README.md tomcat-classloader-ordered. A classloader
for
> Apache
> > > > > Tomcat
> > > > > > 8
> > > > > > >> which loads the jars of WEB-INF lib in alphabetical
order.
> Prior
> > > to
> > > > > > version
> > > > > > >> 8, Apache Tomcat ...
> > > > > > >> >
> > > > > > >> >
> > > > > > >> >
> > > > > > >> > to load its jars in alphabetical order, for whatever
> reason. I
> > > > tried
> > > > > > >> > putting the Loader in various context definition
locations,
> > but
> > > it
> > > > > > >> refuses
> > > > > > >> > to work. Any ideas?
> > > > > > >> >
> > > > > > >> > Jeff
> > > > > > >> >
> > > > > > >> >
> > > > > > >> > On Thu, Mar 9, 2017 at 1:43 PM, Jeff Hair <
> > jeff@greenqloud.com>
> > > > > > wrote:
> > > > > > >> >
> > > > > > >> >> Hi,
> > > > > > >> >>
> > > > > > >> >> I'm deploying 4.9.2.0 (not the vanilla version,
but rather
> an
> > > > > > upgraded
> > > > > > >> >> version of our fork) on Tomcat 8. Management
server startup
> > > fails
> > > > > > with
> > > > > > >> the
> > > > > > >> >> following error:
> > > > > > >> >>
> > > > > > >> >> java.lang.IncompatibleClassChangeError: Found
interface
> > > > > > >> >> com.google.common.base.Equivalence, but class
was expected
> > > > > > >> >>
> > > > > > >> >> I've traced this down to the OutOfBandServiceManagerImpl.
> > More
> > > > > > >> >> specifically, when it tries to build the hostAlertCache
> using
> > > > > Guava's
> > > > > > >> >> CacheBuilder. Deep in Guava, it's calling
an "identity()"
> > > method
> > > > on
> > > > > > the
> > > > > > >> >> Equivalence class.  All of the Guava classes
are coming
> from
> > > > > > guava-19.0
> > > > > > >> >> except for com/google/common/base/Equivalence.class.
The
> > > > > Equivalence
> > > > > > >> >> class is being loaded from the midonet jar
for some reason,
> > and
> > > > > that
> > > > > > >> >> version does not have the method needed. Thus,
the error.
> > > > > > >> >>
> > > > > > >> >> This is because Tomcat apparently does not
load jars in
> > > > > alphabetical
> > > > > > >> order
> > > > > > >> >> anymore, starting with version 8. An open
ticket for them
> to
> > > fix
> > > > > > this is
> > > > > > >> >> here: https://bz.apache.org/bugzilla/show_bug.cgi?id=57129
> > > > > > >> > 57129 – Regression. Load WEB-INF/lib jarfiles
in ...<
> > > > > > >> https://bz.apache.org/bugzilla/show_bug.cgi?id=57129>
> > > > > > >> > bz.apache.org
> > > > > > >> > ASF Bugzilla – Bug 57129 Regression. Load WEB-INF/lib
> jarfiles
> > > in
> > > > > > >> alphabetical order Last modified: 2016-03-17 09:59:50
UTC
> > > > > > >> >
> > > > > > >> >
> > > > > > >> >
> > > > > > >> >>
> > > > > > >> >> It could be possible to "fix" this by using
a custom
> > > ClassLoader
> > > > to
> > > > > > >> force
> > > > > > >> >> Tomcat to load things alphabetically (testing
that right
> > > now--and
> > > > > not
> > > > > > >> >> really succeeding), but the proper fix is
to have the
> midonet
> > > > > client
> > > > > > >> not be
> > > > > > >> >> packaging guava with itself. Does anyone know
why this is?
> > > > > > >> >>
> > > > > > >> >> Jeff
> > > > > > >> >>
> > > > > > >>
> > > > > > >>
> > > > > > >>
> > > > > > >> --
> > > > > > >> Daan
> > > > > > >>
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Rafael Weingärtner
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Daan
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Rafael Weingärtner
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Rafael Weingärtner
> > >
> >
>
>
>
> --
> Rafael Weingärtner
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message