Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
MIME-Version: 1.0
References: <CADOTMGGwi2xmJb2Y8GprC3Dra6B3KwPpSU-joa6+W4hp0SwbMw@mail.gmail.com>
 <CADOTMGERd2mjsrJ-QSVhJr4o=JE9TnNtRAxNmTftNu1KnJ4_RA@mail.gmail.com>
 <BY2PR02MB200760284BEE82F5A9774A63A9210@BY2PR02MB2007.namprd02.prod.outlook.com>
 <CAGQtxvYaQ0Y4oEVzqwwJa5TP62e-FQhV7QP8rU5SOLV41hSsDg@mail.gmail.com>
 <CAG97raexZ5-KNZban2zLF+2G=VS+zt5Se-cYcZLefHX20-wxCg@mail.gmail.com>
 <CAGQtxvZXQkVewbyRhnmy5Kbbu12fjAbhUFxQVa8HafvYcfwucQ@mail.gmail.com>
 <CAG97rachLzsBFkwbra=fXrVoqb-XFyxoG7muSpn2=cKaaS78kA@mail.gmail.com>
 <CAC2panqb1RaFW4VwguPv0VmAKY-1P_O1oP1BZ7MEiYM2miLTmA@mail.gmail.com>
 <CAG97radsvMtqp3J_8Krg9anuf3-k31BTesG+xrTSSJ2VYeZ_0w@mail.gmail.com>
 <CAC2panpaZAz8yjsP_9VTQ9R3Mx9e1MmS9sjaVcu01jbKsdCjFw@mail.gmail.com>
 <CAG97racY8xj8=ehh-uRWMg0RjYN-ZdKTXPUsia+ksuRjHxaYdQ@mail.gmail.com>
 <CAC2pano_CfgOriLOyrMJ5kxz+K9fQpSSV0-mypp-WMnEhVZMOg@mail.gmail.com> <CADMzvUrN5-fP9v65kT4KPQ_kvk4RiX=vwnN0E5EJ7WyYUbO3aw@mail.gmail.com>
In-Reply-To: <CADMzvUrN5-fP9v65kT4KPQ_kvk4RiX=vwnN0E5EJ7WyYUbO3aw@mail.gmail.com>
From: Syed Ahmed <sahmed@cloudops.com>
Date: Fri, 10 Mar 2017 01:55:22 +0000
Message-ID: <CADMzvUoCvYairDzwGQ-H1cmXOhwmbfCRnZze28A_Af53Zes32A@mail.gmail.com>
Subject: Re: midonet-client and Guava dependency conflict
To: dev@cloudstack.apache.org
Content-Type: multipart/alternative; boundary=001a113c1bec2ffb09054a56a8eb
archived-at: Fri, 10 Mar 2017 01:55:45 -0000

--001a113c1bec2ffb09054a56a8eb
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

*git history

Sorry got autocorrected
On Thu, Mar 9, 2017 at 20:54 Syed Ahmed <sahmed@cloudops.com> wrote:

> We can look at the gut history and try to contact the guy who wrote the
> plugin to get more info about who's using it
> On Thu, Mar 9, 2017 at 20:01 Will Stevens <williamstevens@gmail.com>
> wrote:
>
> 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=C3=A4rtner" <rafaelweingartner@gmai=
l.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=C3=A4rtner" <
> > 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 trac=
k
> > > 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=C3=A4rtner <
> > > > > 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 us=
e.
> > > > > > >
> > > > > > > On Thu, Mar 9, 2017 at 7:11 PM, Rafael Weing=C3=A4rtner
> > > > > > > <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 point=
s
> > 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 conflic=
t
> > > > > > > >> >
> > > > > > > >> > 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 updat=
ed
> > 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=3D3&s=3D400
> > > > > > ]<https://
> > > > > > > >> github.com/openwide-java/tomcat-classloader-ordered>
> > > > > > > >> >
> > > > > > > >> > GitHub - openwide-java/tomcat-classloader-ordered: A ...=
<
> > > > > > > >> https://github.com/openwide-java/tomcat-classloader-ordere=
d
> >
> > > > > > > >> > 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 OutOfBandServiceManagerImp=
l.
> > > 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. Th=
e
> > > > > > Equivalence
> > > > > > > >> >> class is being loaded from the midonet jar for some
> reason,
> > > and
> > > > > > that
> > > > > > > >> >> version does not have the method needed. Thus, the erro=
r.
> > > > > > > >> >>
> > > > > > > >> >> This is because Tomcat apparently does not load jars in
> > > > > > alphabetical
> > > > > > > >> order
> > > > > > > >> >> anymore, starting with version 8. An open ticket for th=
em
> > to
> > > > fix
> > > > > > > this is
> > > > > > > >> >> here:
> https://bz.apache.org/bugzilla/show_bug.cgi?id=3D57129
> > > > > > > >> > 57129 =E2=80=93 Regression. Load WEB-INF/lib jarfiles in=
 ...<
> > > > > > > >> https://bz.apache.org/bugzilla/show_bug.cgi?id=3D57129>
> > > > > > > >> > bz.apache.org
> > > > > > > >> > ASF Bugzilla =E2=80=93 Bug 57129 Regression. Load WEB-IN=
F/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 righ=
t
> > > > 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=C3=A4rtner
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Daan
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Rafael Weing=C3=A4rtner
> > > > > >
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Rafael Weing=C3=A4rtner
> > > >
> > >
> >
> >
> >
> > --
> > Rafael Weing=C3=A4rtner
> >
>
>

--001a113c1bec2ffb09054a56a8eb--