[release] [neutron-dev] Autorelease carbon failed to build features-neutron from neutron


Thanh Ha <thanh.ha@...>
 

Any updates here?

We're still failing and Carbon code freeze is soon I think. We need a solution asap.

Regards,
Thanh

On Mon, Jul 31, 2017 at 10:49 PM, Vishal Thapar <vishal.thapar@...> wrote:

So, any agreement on a) How to address this issue in short term? And b) Proper way to do it long term?

 

Both can be the same, but we need to agree soon. Options I’ve seen discussed are:

  1. Make utils their own feature and have consumers explicitly use them. This will require changes to all dependent projects and will also require changes on Nitrogen.
    1. Optionally do the change only on Carbon.
  2. Add explicit dependency in broken projects [Neutron] to depend on library and only on Carbon. Karaf4 takes care of transitive dependencies for us on Nitrogen.
  3. Add library-features and utils to southbound-api on Carbon. This will isolate the fix to OVSDB itself and not require changes in any downstream project. Option to do the same on Nitrogen too, though not required fo now due to karaf4

 

Did I miss anything?

 

Regards,

Vishal.

 

From: Anil Vishnoi [mailto:vishnoianil@...]
Sent: 01 August 2017 00:14
To: Stephen Kitt <skitt@...>
Cc: Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@opendaylight.org>; <ovsdb-dev@....org> <ovsdb-dev@....org>; Release (release@....org) <release@....org>; Anil Belur <abelur@...>; neutron-dev@lists.opendaylight.org; isaku.yamahata@...
Subject: Re: [ovsdb-dev] [neutron-dev] [release] Autorelease carbon failed to build features-neutron from neutron

 

I think we had a long discussion on these utilities classes, but it was kind of inconclusive. Making utils code as a feature is probably right thing to do, but with that we are officially exposing all the utility methods as a Project API's. So we need to be bit careful on what API we are exposing to downstream consumer, because any changes in this will have to go through API deprecation process.

 

On Mon, Jul 31, 2017 at 7:45 AM, Stephen Kitt <skitt@...> wrote:

Ideally, changes made to a stable release shouldn’t involve changes in
feature dependencies...

But yes, we could add a utils feature (in Nitrogen and Carbon, why not
since we’ve already painted ourselves into a corner); that would allow
a bunch of cleanup in places pulling in the utils bundles.

Regards,

Stephen


On Mon, 31 Jul 2017 14:37:09 +0000
Vishal Thapar <vishal.thapar@...> wrote:

> Hi Sam,
>
> Yep, that is exactly what I had in mind too and left as comment on
> Stephen’s patch. This will likely require dependent projects to pull
> in utils feature. Alternative, for carbon at least, could be to
> append utils and library to southbound-api.
>
> Regards,
> Vishal.
>
> From: Sam Hague [mailto:shague@...]
> Sent: 31 July 2017 20:04
> To: Stephen Kitt <skitt@...>
> Cc: Vishal Thapar <vishal.thapar@...>;
> <ovsdb-dev@....org>
> <ovsdb-dev@....org>; Release
> (release@....org) <release@....org>;
> Isaku Yamahata <yamahata@...>;
> isaku.yamahata@...; neutron-dev@lists.opendaylight.org; Anil
> Belur <abelur@...>; Jenkins
> <jenkins-dontreply@opendaylight.org> Subject: Re: [neutron-dev]
> [release] Autorelease carbon failed to build features-neutron from
> neutron
>
>
>
> On Mon, Jul 31, 2017 at 10:22 AM, Stephen Kitt
> <skitt@...<mailto:skitt@redhat.com>> wrote: Actually no,
> that’s got nothing to do with it; the problem is that
> southbound-utils now needs the ovsdb library, and since Karaf 3
> feature dependencies are manual, we need to add that explicitly.
>
> The Karaf 4 feature generator discovers this on its own, which is why
> this “just works” in Nitrogen.
> And does this work for netvirt and genius because they pull in the
> odl-ovsdb-southbound-impl feature which pulls in the
> odl-ovsdb-library?
>
> Seems like the original patch to add the dependency on
> southbound-utils to include the lib.notation may not have been best.
> it si hard for dependent projects to bring in transitive libs. Maybe
> the southbound-utils should be a feature and includes everything it
> needs so projects can simply include the feature?
>
> https://git.opendaylight.org/gerrit/60936
>
> Regards,
>
> Stephen
>
>
> On Mon, 31 Jul 2017 15:55:17 +0200
> Stephen Kitt <skitt@...<mailto:skitt@redhat.com>> wrote:
>
> > In Carbon, the odl-neutron-hostconfig-ovs feature is being generated
> > incorrectly, with Nitrogen dependencies. I’m trying to figure out
> > why that is...
> >
> > Regards,
> >
> > Stephen
> >
> >
> > On Mon, 31 Jul 2017 11:15:01 +0000
> > Vishal Thapar
> > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com>>
> > wrote:
> > > Hi Sam,
> > >
> > > Yeah, this fails on Carbon but Nitrogen is working fine. One key
> > > difference between Neutron and Genius/Netvirt is that Neutron
> > > doesn’t require southbound-impl. I suspect it is because library
> > > gets pulled in with southbound-impl but not API. Note that library
> > > has its own artifacts/features.
> > >
> > > If it were failing on Nitrogen too, would’ve been easier to figure
> > > out. I am guessing the way karaf4 changes work it is able to pull
> > > in library on Nitrogen, but not on Carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Sam Hague
> > > [mailto:shague@...<mailto:shague@...>] Sent: 31
> > > July 2017 16:40 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com>>
> > > Cc:
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org>>
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org>>;
> > > Release
> > > (release@....org<mailto:release@lists.opendaylight.org>)
> > > <release@....org<mailto:release@lists.opendaylight.org>>;
> > > Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@gmail.com>;
> > > Stephen Kitt <skitt@...<mailto:skitt@redhat.com>>;
> > > neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@linuxfoundation.org>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > So this fails on carbon or is that backwards? Genius and netvirt
> > > both pull these in so it should work fine for neutron. Netvirt
> > > includes the feature as well as the artifact so maybe that is why
> > > it works.
> > >
> > > I will take a look today also.
> > >
> > > On Jul 31, 2017 5:12 AM, "Vishal Thapar"
> > > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@...>>>
> > > wrote: Okay, it didn’t work.
> > >
> > > Sam, Stephen,
> > >
> > > Any ideas? I can’t figure out how ovsdb.utils.* are being
> > > published as I don’t see any features for it. Probably want to ad
> > > mor to features/pom and/or features.xml on carbon. Just wondering
> > > why it works on Nitrogen but not on carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Vishal Thapar
> > > Sent: 31 July 2017 14:19
> > > To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@...>>>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@linuxfoundation.org><mailto:abelur@linuxfoundation.org<mailto:abelur@...>>>
> > >
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@lists.opendaylight.org>>>;
> > > Release
> > > <release@....org<mailto:release@lists.opendaylight.org><mailto:release@...<mailto:release@lists.opendaylight.org>>>;
> > > neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...><mailto:neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@gmail.com><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@...>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Patch: https://git.opendaylight.org/gerrit/#/c/60930/
> > >
> > > It passed the autorelease verification, so assume it means good to
> > > go?
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From:
> > > neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org><mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org>]
> > > On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
> > > <abelur@...<mailto:abelur@linuxfoundation.org><mailto:abelur@linuxfoundation.org<mailto:abelur@...>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@lists.opendaylight.org>>>;
> > > Release
> > > <release@....org<mailto:release@lists.opendaylight.org><mailto:release@...<mailto:release@lists.opendaylight.org>>>;
> > > neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...><mailto:neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@gmail.com><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hi Anil,
> > >
> > > I am working on a patch that I think might address it. Will be
> > > pushing it soon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Anil Belur
> > > [mailto:abelur@linuxfoundation.org<mailto:abelur@...>]
> > > Sent: 31 July 2017 13:33 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@...>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@lists.opendaylight.org>>>;
> > > Release
> > > <release@....org<mailto:release@lists.opendaylight.org><mailto:release@...<mailto:release@lists.opendaylight.org>>>;
> > > neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...><mailto:neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@gmail.com><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hello Isaku, Vishal:
> > >
> > > On Sat, Jul 29, 2017 at 3:35 PM, Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@ericsson.com><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@...>>>
> > > wrote: This is cherry pick of patch from Nitrogen. We haven't seen
> > > such failures on Nitrogen, have we? What needs to be done may vary
> > > based on that. Besides netvirt also uses southbound utils, so
> > > wondering why didn't fail for Netvirt. Could this be a transient
> > > issue where autorelease ran while Carbon patch's merge job was in
> > > flight?
> > >
> > > Other question, shouldn't this have been caught by distribution
> > > check job? Looking into it, just trying to figure out how and why
> > > we didn't catch it sooner.
> > >
> > >
> > > Not sure if this is transient, since the same failure is seen
> > > across the last three AR builds. Could we get this resolved
> > > sooner, we like to see AR unblocked.
> > >
> > > Thanks,
> > > Anil
> > >
> > >
> > >
> > >
> > > -----Original Message-----
> > > From:
> > > neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org><mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org><mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@lists.opendaylight.org>>]
> > > On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil
> > > Belur
> > > <abelur@...<mailto:abelur@linuxfoundation.org><mailto:abelur@linuxfoundation.org<mailto:abelur@...>>>
> > > Cc:
> > > yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@...>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@lists.opendaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@lists.opendaylight.org>>>;
> > > Release
> > > <release@....org<mailto:release@lists.opendaylight.org><mailto:release@...<mailto:release@lists.opendaylight.org>>>;
> > > neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...><mailto:neutron-dev@lists.opendaylight.org<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@gmail.com><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > >
> > > It looks like ovsdb issue. Cc'ed to ovsdb-dev.
> > > ovsdb folks, can you please take a look?
> > >
> > >
> > > > Tests in error:
> > > >   Can't install feature odl-neutron-hostconfig-ovs/0.8.2-Carbon:
> > > >   Could not start bundle
> > > > mvn:org.opendaylight.ovsdb/utils.southbound-utils/1.4.2-Carbon
> > > > in feature(s) odl-ovsdb-southbound-test-1.4.2-Carbon,
> > > > odl-neutron-hostconfig-ovs-0.8.2-Carbon: The bundle
> > > > "org.opendaylight.ovsdb.utils.southbound-utils_1.4.2.Carbon
> > > > [293]" could not be resolved. Reason: Missing Constraint:
> > > > Import-Package: org.opendaylight.ovsdb.lib.notation;
> > > > version="[1.4.0,2.0.0)"
> > >
> > > The following changeset of ovsdb updates
> > > utils/southbound-utils/pom.xml which might be a culprit. I'm
> > > guessing other pom.xml or features.xml also needs update.
> > >
> > > > commit c9bbb9b3f71b867e0ebf5dc56ab6f143c0f5a8bd
> > > > Author: Victor Pickard
> > > > <vpickard@...<mailto:vpickard@...><mailto:vpickard@...<mailto:vpickard@...>>>
> > > > Date:   Tue Jul 25 17:20:19 2017 -0400
> > > >
> > > >     Bug 8476 Add support for logical switch replication mode
> > > >
> > > >     Add helper method to SouthboundUtils to check database
> > > >         version.
> > > ...
> > > > diff --git a/utils/southbound-utils/pom.xml
> > > > b/utils/southbound-utils/pom.xml index a16a419..6a3234f 100644
> > > > --- a/utils/southbound-utils/pom.xml
> > > > +++ b/utils/southbound-utils/pom.xml
> > > > @@ -50,6 +50,11 @@ and is available at
> > > > http://www.eclipse.org/legal/epl-v10.html
> > > > <groupId>org.opendaylight.mdsal.model</groupId>
> > > > <artifactId>ietf-topology</artifactId> </dependency>
> > > > +    <dependency>
> > > > +      <groupId>org.opendaylight.ovsdb</groupId>
> > > > +      <artifactId>library</artifactId>
> > > > +      <version>${project.version}</version>
> > > > +    </dependency>
> > > >      <!-- testing dependencies -->
> > > >      <dependency>
> > > >        <groupId>junit</groupId>
> > >
> > > thanks,
> > >
> > > On Fri, Jul 28, 2017 at 07:32:16PM +1000, Anil Belur
> > > <abelur@...<mailto:abelur@linuxfoundation.org><mailto:abelur@linuxfoundation.org<mailto:abelur@...>>>
> > > wrote:
> > >
> > > Hello neutron-dev,
> > >
> > > Autorelease Carbon has failed on SFT timeouts. Please fix them.
> > >
> > > > Running org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
> > > > 130.387 sec <<< FAILURE! - in
> > > > org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > installFeature(org.opendaylight.odlparent.featuretest.SingleFeatureTest)[repoUrl:
> > > > file:/w/workspace/autorelease-release-carbon/neutron/features/producti
> > > > on/features-neutron/target/classes/features.xml,
> > > > Feature: odl-neutron-hostconfig-ovs 0.8.2-Carbon]  Time elapsed:
> > > > 16.327 sec  <<< ERROR!
> > > >
> > > > Thanks,
> > > > Anil
> > > >
> > > > On Fri, Jul 28, 2017 at 1:46 PM, Jenkins
> > > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>

> > > > > wrote:
> > > >
> > > > > Attention neutron-devs,
> > > > >
> > > > > Autorelease carbon failed to build features-neutron from
> > > > > neutron in build 401. Attached is a snippet of the error
> > > > > message related to the failure that we were able to
> > > > > automatically parse as well as console logs.
> > > > >
> > > > >
> > > > > Console Logs:
> > > > > https://logs.opendaylight.org/releng/jenkins092/autorelease-
> > > > > release-carbon/401
> > > > >
> > > > > Jenkins Build:
> > > > > https://jenkins.opendaylight.org/releng/job/autorelease-
> > > > > release-carbon/401/
> > > > >
> > > > > Please review and provide an ETA on when a fix will be
> > > > > available.
> > > > >
> > > > > Thanks,
> > > > > ODL releng/autorelease team


_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@....org
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



 

--

Thanks

Anil


_______________________________________________
release mailing list
release@...
https://lists.opendaylight.org/mailman/listinfo/release



Sam Hague
 

Thanh,

we can just merge [1] which has the fix for neutron. The path those is failing sft on the AAA stuff though. I thought that was fixed though, so not sure what will happen if we merge [1].

We can work on a prettier fix later.

Sam


On Thu, Aug 3, 2017 at 12:06 PM, Thanh Ha <thanh.ha@...> wrote:
Any updates here?

We're still failing and Carbon code freeze is soon I think. We need a solution asap.

Regards,
Thanh

On Mon, Jul 31, 2017 at 10:49 PM, Vishal Thapar <vishal.thapar@...> wrote:

So, any agreement on a) How to address this issue in short term? And b) Proper way to do it long term?

 

Both can be the same, but we need to agree soon. Options I’ve seen discussed are:

  1. Make utils their own feature and have consumers explicitly use them. This will require changes to all dependent projects and will also require changes on Nitrogen.
    1. Optionally do the change only on Carbon.
  2. Add explicit dependency in broken projects [Neutron] to depend on library and only on Carbon. Karaf4 takes care of transitive dependencies for us on Nitrogen.
  3. Add library-features and utils to southbound-api on Carbon. This will isolate the fix to OVSDB itself and not require changes in any downstream project. Option to do the same on Nitrogen too, though not required fo now due to karaf4

 

Did I miss anything?

 

Regards,

Vishal.

 

From: Anil Vishnoi [mailto:vishnoianil@...]
Sent: 01 August 2017 00:14
To: Stephen Kitt <skitt@...>
Cc: Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@opendaylight.org>; <ovsdb-dev@....org> <ovsdb-dev@....org>; Release (release@...g) <release@...g>; Anil Belur <abelur@...>; neutron-dev@....org; isaku.yamahata@...
Subject: Re: [ovsdb-dev] [neutron-dev] [release] Autorelease carbon failed to build features-neutron from neutron

 

I think we had a long discussion on these utilities classes, but it was kind of inconclusive. Making utils code as a feature is probably right thing to do, but with that we are officially exposing all the utility methods as a Project API's. So we need to be bit careful on what API we are exposing to downstream consumer, because any changes in this will have to go through API deprecation process.

 

On Mon, Jul 31, 2017 at 7:45 AM, Stephen Kitt <skitt@...> wrote:

Ideally, changes made to a stable release shouldn’t involve changes in
feature dependencies...

But yes, we could add a utils feature (in Nitrogen and Carbon, why not
since we’ve already painted ourselves into a corner); that would allow
a bunch of cleanup in places pulling in the utils bundles.

Regards,

Stephen


On Mon, 31 Jul 2017 14:37:09 +0000
Vishal Thapar <vishal.thapar@...> wrote:

> Hi Sam,
>
> Yep, that is exactly what I had in mind too and left as comment on
> Stephen’s patch. This will likely require dependent projects to pull
> in utils feature. Alternative, for carbon at least, could be to
> append utils and library to southbound-api.
>
> Regards,
> Vishal.
>
> From: Sam Hague [mailto:shague@...]
> Sent: 31 July 2017 20:04
> To: Stephen Kitt <skitt@...>
> Cc: Vishal Thapar <vishal.thapar@...>;
> <ovsdb-dev@....org>
> <ovsdb-dev@....org>; Release
> (release@...g) <release@...g>;
> Isaku Yamahata <yamahata@...>;
> isaku.yamahata@...; neutron-dev@....org; Anil
> Belur <abelur@...>; Jenkins
> <jenkins-dontreply@opendaylight.org> Subject: Re: [neutron-dev]
> [release] Autorelease carbon failed to build features-neutron from
> neutron
>
>
>
> On Mon, Jul 31, 2017 at 10:22 AM, Stephen Kitt
> <skitt@...<mailto:skitt@redhat.com>> wrote: Actually no,
> that’s got nothing to do with it; the problem is that
> southbound-utils now needs the ovsdb library, and since Karaf 3
> feature dependencies are manual, we need to add that explicitly.
>
> The Karaf 4 feature generator discovers this on its own, which is why
> this “just works” in Nitrogen.
> And does this work for netvirt and genius because they pull in the
> odl-ovsdb-southbound-impl feature which pulls in the
> odl-ovsdb-library?
>
> Seems like the original patch to add the dependency on
> southbound-utils to include the lib.notation may not have been best.
> it si hard for dependent projects to bring in transitive libs. Maybe
> the southbound-utils should be a feature and includes everything it
> needs so projects can simply include the feature?
>
> https://git.opendaylight.org/gerrit/60936
>
> Regards,
>
> Stephen
>
>
> On Mon, 31 Jul 2017 15:55:17 +0200
> Stephen Kitt <skitt@...<mailto:skitt@redhat.com>> wrote:
>
> > In Carbon, the odl-neutron-hostconfig-ovs feature is being generated
> > incorrectly, with Nitrogen dependencies. I’m trying to figure out
> > why that is...
> >
> > Regards,
> >
> > Stephen
> >
> >
> > On Mon, 31 Jul 2017 11:15:01 +0000
> > Vishal Thapar
> > <vishal.thapar@...<mailto:vishal.thapar@...m>>
> > wrote:
> > > Hi Sam,
> > >
> > > Yeah, this fails on Carbon but Nitrogen is working fine. One key
> > > difference between Neutron and Genius/Netvirt is that Neutron
> > > doesn’t require southbound-impl. I suspect it is because library
> > > gets pulled in with southbound-impl but not API. Note that library
> > > has its own artifacts/features.
> > >
> > > If it were failing on Nitrogen too, would’ve been easier to figure
> > > out. I am guessing the way karaf4 changes work it is able to pull
> > > in library on Nitrogen, but not on Carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Sam Hague
> > > [mailto:shague@...<mailto:shague@...>] Sent: 31
> > > July 2017 16:40 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m>>
> > > Cc:
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org>>
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org>>;
> > > Release
> > > (release@...g<mailto:release@...light.org>)
> > > <release@...g<mailto:release@...light.org>>;
> > > Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...>;
> > > Stephen Kitt <skitt@...<mailto:skitt@redhat.com>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...g>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > So this fails on carbon or is that backwards? Genius and netvirt
> > > both pull these in so it should work fine for neutron. Netvirt
> > > includes the feature as well as the artifact so maybe that is why
> > > it works.
> > >
> > > I will take a look today also.
> > >
> > > On Jul 31, 2017 5:12 AM, "Vishal Thapar"
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > wrote: Okay, it didn’t work.
> > >
> > > Sam, Stephen,
> > >
> > > Any ideas? I can’t figure out how ovsdb.utils.* are being
> > > published as I don’t see any features for it. Probably want to ad
> > > mor to features/pom and/or features.xml on carbon. Just wondering
> > > why it works on Nitrogen but not on carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Vishal Thapar
> > > Sent: 31 July 2017 14:19
> > > To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > >
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...ht.org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@lists.opendaylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@gmail.com>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Patch: https://git.opendaylight.org/gerrit/#/c/60930/
> > >
> > > It passed the autorelease verification, so assume it means good to
> > > go?
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From:
> > > neutron-dev-bounces@...daylight.org<mailto:neutron-dev-bounces@....org><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@...ylight.org>]
> > > On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...ht.org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@lists.opendaylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hi Anil,
> > >
> > > I am working on a patch that I think might address it. Will be
> > > pushing it soon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Anil Belur
> > > [mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>]
> > > Sent: 31 July 2017 13:33 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...ht.org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@lists.opendaylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hello Isaku, Vishal:
> > >
> > > On Sat, Jul 29, 2017 at 3:35 PM, Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > wrote: This is cherry pick of patch from Nitrogen. We haven't seen
> > > such failures on Nitrogen, have we? What needs to be done may vary
> > > based on that. Besides netvirt also uses southbound utils, so
> > > wondering why didn't fail for Netvirt. Could this be a transient
> > > issue where autorelease ran while Carbon patch's merge job was in
> > > flight?
> > >
> > > Other question, shouldn't this have been caught by distribution
> > > check job? Looking into it, just trying to figure out how and why
> > > we didn't catch it sooner.
> > >
> > >
> > > Not sure if this is transient, since the same failure is seen
> > > across the last three AR builds. Could we get this resolved
> > > sooner, we like to see AR unblocked.
> > >
> > > Thanks,
> > > Anil
> > >
> > >
> > >
> > >
> > > -----Original Message-----
> > > From:
> > > neutron-dev-bounces@...daylight.org<mailto:neutron-dev-bounces@....org><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@...ylight.org><mailto:neutron-dev-bounces@....org<mailto:neutron-dev-bounces@...>>]
> > > On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil
> > > Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > Cc:
> > > yamahata@...<mailto:yamahata@...e.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...ht.org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@lists.opendaylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@gmail.com<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > >
> > > It looks like ovsdb issue. Cc'ed to ovsdb-dev.
> > > ovsdb folks, can you please take a look?
> > >
> > >
> > > > Tests in error:
> > > >   Can't install feature odl-neutron-hostconfig-ovs/0.8.2-Carbon:
> > > >   Could not start bundle
> > > > mvn:org.opendaylight.ovsdb/utils.southbound-utils/1.4.2-Carbon
> > > > in feature(s) odl-ovsdb-southbound-test-1.4.2-Carbon,
> > > > odl-neutron-hostconfig-ovs-0.8.2-Carbon: The bundle
> > > > "org.opendaylight.ovsdb.utils.southbound-utils_1.4.2.Carbon
> > > > [293]" could not be resolved. Reason: Missing Constraint:
> > > > Import-Package: org.opendaylight.ovsdb.lib.notation;
> > > > version="[1.4.0,2.0.0)"
> > >
> > > The following changeset of ovsdb updates
> > > utils/southbound-utils/pom.xml which might be a culprit. I'm
> > > guessing other pom.xml or features.xml also needs update.
> > >
> > > > commit c9bbb9b3f71b867e0ebf5dc56ab6f143c0f5a8bd
> > > > Author: Victor Pickard
> > > > <vpickard@...<mailto:vpickard@...><mailto:vpickard@...<mailto:vpickard@...>>>
> > > > Date:   Tue Jul 25 17:20:19 2017 -0400
> > > >
> > > >     Bug 8476 Add support for logical switch replication mode
> > > >
> > > >     Add helper method to SouthboundUtils to check database
> > > >         version.
> > > ...
> > > > diff --git a/utils/southbound-utils/pom.xml
> > > > b/utils/southbound-utils/pom.xml index a16a419..6a3234f 100644
> > > > --- a/utils/southbound-utils/pom.xml
> > > > +++ b/utils/southbound-utils/pom.xml
> > > > @@ -50,6 +50,11 @@ and is available at
> > > > http://www.eclipse.org/legal/epl-v10.html
> > > > <groupId>org.opendaylight.mdsal.model</groupId>
> > > > <artifactId>ietf-topology</artifactId> </dependency>
> > > > +    <dependency>
> > > > +      <groupId>org.opendaylight.ovsdb</groupId>
> > > > +      <artifactId>library</artifactId>
> > > > +      <version>${project.version}</version>
> > > > +    </dependency>
> > > >      <!-- testing dependencies -->
> > > >      <dependency>
> > > >        <groupId>junit</groupId>
> > >
> > > thanks,
> > >
> > > On Fri, Jul 28, 2017 at 07:32:16PM +1000, Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > wrote:
> > >
> > > Hello neutron-dev,
> > >
> > > Autorelease Carbon has failed on SFT timeouts. Please fix them.
> > >
> > > > Running org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
> > > > 130.387 sec <<< FAILURE! - in
> > > > org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > installFeature(org.opendaylight.odlparent.featuretest.SingleFeatureTest)[repoUrl:
> > > > file:/w/workspace/autorelease-release-carbon/neutron/features/producti
> > > > on/features-neutron/target/classes/features.xml,
> > > > Feature: odl-neutron-hostconfig-ovs 0.8.2-Carbon]  Time elapsed:
> > > > 16.327 sec  <<< ERROR!
> > > >
> > > > Thanks,
> > > > Anil
> > > >
> > > > On Fri, Jul 28, 2017 at 1:46 PM, Jenkins
> > > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@...>>

> > > > > wrote:
> > > >
> > > > > Attention neutron-devs,
> > > > >
> > > > > Autorelease carbon failed to build features-neutron from
> > > > > neutron in build 401. Attached is a snippet of the error
> > > > > message related to the failure that we were able to
> > > > > automatically parse as well as console logs.
> > > > >
> > > > >
> > > > > Console Logs:
> > > > > https://logs.opendaylight.org/releng/jenkins092/autorelease-
> > > > > release-carbon/401
> > > > >
> > > > > Jenkins Build:
> > > > > https://jenkins.opendaylight.org/releng/job/autorelease-
> > > > > release-carbon/401/
> > > > >
> > > > > Please review and provide an ETA on when a fix will be
> > > > > available.
> > > > >
> > > > > Thanks,
> > > > > ODL releng/autorelease team


_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...rg
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



 

--

Thanks

Anil


_______________________________________________
release mailing list
release@...
https://lists.opendaylight.org/mailman/listinfo/release



_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@....org
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



Thanh Ha <thanh.ha@...>
 

Agreed. With code freeze soon I think we should have a working Carbon release asap.

Regards,
Thanh


On Thu, Aug 3, 2017 at 1:08 PM, Sam Hague <shague@...> wrote:
Thanh,

we can just merge [1] which has the fix for neutron. The path those is failing sft on the AAA stuff though. I thought that was fixed though, so not sure what will happen if we merge [1].

We can work on a prettier fix later.

Sam


On Thu, Aug 3, 2017 at 12:06 PM, Thanh Ha <thanh.ha@...> wrote:
Any updates here?

We're still failing and Carbon code freeze is soon I think. We need a solution asap.

Regards,
Thanh

On Mon, Jul 31, 2017 at 10:49 PM, Vishal Thapar <vishal.thapar@...> wrote:

So, any agreement on a) How to address this issue in short term? And b) Proper way to do it long term?

 

Both can be the same, but we need to agree soon. Options I’ve seen discussed are:

  1. Make utils their own feature and have consumers explicitly use them. This will require changes to all dependent projects and will also require changes on Nitrogen.
    1. Optionally do the change only on Carbon.
  2. Add explicit dependency in broken projects [Neutron] to depend on library and only on Carbon. Karaf4 takes care of transitive dependencies for us on Nitrogen.
  3. Add library-features and utils to southbound-api on Carbon. This will isolate the fix to OVSDB itself and not require changes in any downstream project. Option to do the same on Nitrogen too, though not required fo now due to karaf4

 

Did I miss anything?

 

Regards,

Vishal.

 

From: Anil Vishnoi [mailto:vishnoianil@...]
Sent: 01 August 2017 00:14
To: Stephen Kitt <skitt@...>
Cc: Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@opendaylight.org>; <ovsdb-dev@....org> <ovsdb-dev@....org>; Release (release@...g) <release@...g>; Anil Belur <abelur@...>; neutron-dev@....org; isaku.yamahata@...
Subject: Re: [ovsdb-dev] [neutron-dev] [release] Autorelease carbon failed to build features-neutron from neutron

 

I think we had a long discussion on these utilities classes, but it was kind of inconclusive. Making utils code as a feature is probably right thing to do, but with that we are officially exposing all the utility methods as a Project API's. So we need to be bit careful on what API we are exposing to downstream consumer, because any changes in this will have to go through API deprecation process.

 

On Mon, Jul 31, 2017 at 7:45 AM, Stephen Kitt <skitt@...> wrote:

Ideally, changes made to a stable release shouldn’t involve changes in
feature dependencies...

But yes, we could add a utils feature (in Nitrogen and Carbon, why not
since we’ve already painted ourselves into a corner); that would allow
a bunch of cleanup in places pulling in the utils bundles.

Regards,

Stephen


On Mon, 31 Jul 2017 14:37:09 +0000
Vishal Thapar <vishal.thapar@...> wrote:

> Hi Sam,
>
> Yep, that is exactly what I had in mind too and left as comment on
> Stephen’s patch. This will likely require dependent projects to pull
> in utils feature. Alternative, for carbon at least, could be to
> append utils and library to southbound-api.
>
> Regards,
> Vishal.
>
> From: Sam Hague [mailto:shague@...]
> Sent: 31 July 2017 20:04
> To: Stephen Kitt <skitt@...>
> Cc: Vishal Thapar <vishal.thapar@...>;
> <ovsdb-dev@....org>
> <ovsdb-dev@....org>; Release
> (release@...g) <release@...g>;
> Isaku Yamahata <yamahata@...>;
> isaku.yamahata@...; neutron-dev@....org; Anil
> Belur <abelur@...>; Jenkins
> <jenkins-dontreply@opendaylight.org> Subject: Re: [neutron-dev]
> [release] Autorelease carbon failed to build features-neutron from
> neutron
>
>
>
> On Mon, Jul 31, 2017 at 10:22 AM, Stephen Kitt
> <skitt@...<mailto:skitt@redhat.com>> wrote: Actually no,
> that’s got nothing to do with it; the problem is that
> southbound-utils now needs the ovsdb library, and since Karaf 3
> feature dependencies are manual, we need to add that explicitly.
>
> The Karaf 4 feature generator discovers this on its own, which is why
> this “just works” in Nitrogen.
> And does this work for netvirt and genius because they pull in the
> odl-ovsdb-southbound-impl feature which pulls in the
> odl-ovsdb-library?
>
> Seems like the original patch to add the dependency on
> southbound-utils to include the lib.notation may not have been best.
> it si hard for dependent projects to bring in transitive libs. Maybe
> the southbound-utils should be a feature and includes everything it
> needs so projects can simply include the feature?
>
> https://git.opendaylight.org/gerrit/60936
>
> Regards,
>
> Stephen
>
>
> On Mon, 31 Jul 2017 15:55:17 +0200
> Stephen Kitt <skitt@...<mailto:skitt@redhat.com>> wrote:
>
> > In Carbon, the odl-neutron-hostconfig-ovs feature is being generated
> > incorrectly, with Nitrogen dependencies. I’m trying to figure out
> > why that is...
> >
> > Regards,
> >
> > Stephen
> >
> >
> > On Mon, 31 Jul 2017 11:15:01 +0000
> > Vishal Thapar
> > <vishal.thapar@...<mailto:vishal.thapar@...m>>
> > wrote:
> > > Hi Sam,
> > >
> > > Yeah, this fails on Carbon but Nitrogen is working fine. One key
> > > difference between Neutron and Genius/Netvirt is that Neutron
> > > doesn’t require southbound-impl. I suspect it is because library
> > > gets pulled in with southbound-impl but not API. Note that library
> > > has its own artifacts/features.
> > >
> > > If it were failing on Nitrogen too, would’ve been easier to figure
> > > out. I am guessing the way karaf4 changes work it is able to pull
> > > in library on Nitrogen, but not on Carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Sam Hague
> > > [mailto:shague@...<mailto:shague@...>] Sent: 31
> > > July 2017 16:40 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m>>
> > > Cc:
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org>>
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org>>;
> > > Release
> > > (release@...g<mailto:release@...light.org>)
> > > <release@...g<mailto:release@...light.org>>;
> > > Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...>;
> > > Stephen Kitt <skitt@...<mailto:skitt@redhat.com>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...g>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > So this fails on carbon or is that backwards? Genius and netvirt
> > > both pull these in so it should work fine for neutron. Netvirt
> > > includes the feature as well as the artifact so maybe that is why
> > > it works.
> > >
> > > I will take a look today also.
> > >
> > > On Jul 31, 2017 5:12 AM, "Vishal Thapar"
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > wrote: Okay, it didn’t work.
> > >
> > > Sam, Stephen,
> > >
> > > Any ideas? I can’t figure out how ovsdb.utils.* are being
> > > published as I don’t see any features for it. Probably want to ad
> > > mor to features/pom and/or features.xml on carbon. Just wondering
> > > why it works on Nitrogen but not on carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Vishal Thapar
> > > Sent: 31 July 2017 14:19
> > > To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > >
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@opendaylight.org>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@....org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@...aylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@gmail.com>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Patch: https://git.opendaylight.org/gerrit/#/c/60930/
> > >
> > > It passed the autorelease verification, so assume it means good to
> > > go?
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From:
> > > neutron-dev-bounces@...daylight.org<mailto:neutron-dev-bounces@....org><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@...ylight.org>]
> > > On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@opendaylight.org>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@....org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@...aylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hi Anil,
> > >
> > > I am working on a patch that I think might address it. Will be
> > > pushing it soon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Anil Belur
> > > [mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>]
> > > Sent: 31 July 2017 13:33 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@....ne.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@opendaylight.org>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@....org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@...aylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hello Isaku, Vishal:
> > >
> > > On Sat, Jul 29, 2017 at 3:35 PM, Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...m><mailto:vishal.thapar@ericsson.com<mailto:vishal.thapar@ericsson.com>>>
> > > wrote: This is cherry pick of patch from Nitrogen. We haven't seen
> > > such failures on Nitrogen, have we? What needs to be done may vary
> > > based on that. Besides netvirt also uses southbound utils, so
> > > wondering why didn't fail for Netvirt. Could this be a transient
> > > issue where autorelease ran while Carbon patch's merge job was in
> > > flight?
> > >
> > > Other question, shouldn't this have been caught by distribution
> > > check job? Looking into it, just trying to figure out how and why
> > > we didn't catch it sooner.
> > >
> > >
> > > Not sure if this is transient, since the same failure is seen
> > > across the last three AR builds. Could we get this resolved
> > > sooner, we like to see AR unblocked.
> > >
> > > Thanks,
> > > Anil
> > >
> > >
> > >
> > >
> > > -----Original Message-----
> > > From:
> > > neutron-dev-bounces@...daylight.org<mailto:neutron-dev-bounces@....org><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@lists.opendaylight.org>>
> > > [mailto:neutron-dev-bounces@lists.opendaylight.org<mailto:neutron-dev-bounces@...ylight.org><mailto:neutron-dev-bounces@....org<mailto:neutron-dev-bounces@lists.opendaylight.org>>]
> > > On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil
> > > Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > Cc:
> > > yamahata@...<mailto:yamahata@...e.jp><mailto:yamahata@private.email.ne.jp<mailto:yamahata@private.email.ne.jp>>;
> > > Jenkins
> > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@opendaylight.org>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@....org<mailto:ovsdb-dev@...ndaylight.org><mailto:ovsdb-dev@...<mailto:ovsdb-dev@....org>>>;
> > > Release
> > > <release@...g<mailto:release@...light.org><mailto:release@lists.opendaylight.org<mailto:release@...>>>;
> > > neutron-dev@....org<mailto:neutron-dev@lists.opendaylight.org><mailto:neutron-dev@...<mailto:neutron-dev@...aylight.org>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@gmail.com>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > >
> > > It looks like ovsdb issue. Cc'ed to ovsdb-dev.
> > > ovsdb folks, can you please take a look?
> > >
> > >
> > > > Tests in error:
> > > >   Can't install feature odl-neutron-hostconfig-ovs/0.8.2-Carbon:
> > > >   Could not start bundle
> > > > mvn:org.opendaylight.ovsdb/utils.southbound-utils/1.4.2-Carbon
> > > > in feature(s) odl-ovsdb-southbound-test-1.4.2-Carbon,
> > > > odl-neutron-hostconfig-ovs-0.8.2-Carbon: The bundle
> > > > "org.opendaylight.ovsdb.utils.southbound-utils_1.4.2.Carbon
> > > > [293]" could not be resolved. Reason: Missing Constraint:
> > > > Import-Package: org.opendaylight.ovsdb.lib.notation;
> > > > version="[1.4.0,2.0.0)"
> > >
> > > The following changeset of ovsdb updates
> > > utils/southbound-utils/pom.xml which might be a culprit. I'm
> > > guessing other pom.xml or features.xml also needs update.
> > >
> > > > commit c9bbb9b3f71b867e0ebf5dc56ab6f143c0f5a8bd
> > > > Author: Victor Pickard
> > > > <vpickard@...<mailto:vpickard@...><mailto:vpickard@...<mailto:vpickard@...>>>
> > > > Date:   Tue Jul 25 17:20:19 2017 -0400
> > > >
> > > >     Bug 8476 Add support for logical switch replication mode
> > > >
> > > >     Add helper method to SouthboundUtils to check database
> > > >         version.
> > > ...
> > > > diff --git a/utils/southbound-utils/pom.xml
> > > > b/utils/southbound-utils/pom.xml index a16a419..6a3234f 100644
> > > > --- a/utils/southbound-utils/pom.xml
> > > > +++ b/utils/southbound-utils/pom.xml
> > > > @@ -50,6 +50,11 @@ and is available at
> > > > http://www.eclipse.org/legal/epl-v10.html
> > > > <groupId>org.opendaylight.mdsal.model</groupId>
> > > > <artifactId>ietf-topology</artifactId> </dependency>
> > > > +    <dependency>
> > > > +      <groupId>org.opendaylight.ovsdb</groupId>
> > > > +      <artifactId>library</artifactId>
> > > > +      <version>${project.version}</version>
> > > > +    </dependency>
> > > >      <!-- testing dependencies -->
> > > >      <dependency>
> > > >        <groupId>junit</groupId>
> > >
> > > thanks,
> > >
> > > On Fri, Jul 28, 2017 at 07:32:16PM +1000, Anil Belur
> > > <abelur@...<mailto:abelur@...g><mailto:abelur@linuxfoundation.org<mailto:abelur@linuxfoundation.org>>>
> > > wrote:
> > >
> > > Hello neutron-dev,
> > >
> > > Autorelease Carbon has failed on SFT timeouts. Please fix them.
> > >
> > > > Running org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
> > > > 130.387 sec <<< FAILURE! - in
> > > > org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > installFeature(org.opendaylight.odlparent.featuretest.SingleFeatureTest)[repoUrl:
> > > > file:/w/workspace/autorelease-release-carbon/neutron/features/producti
> > > > on/features-neutron/target/classes/features.xml,
> > > > Feature: odl-neutron-hostconfig-ovs 0.8.2-Carbon]  Time elapsed:
> > > > 16.327 sec  <<< ERROR!
> > > >
> > > > Thanks,
> > > > Anil
> > > >
> > > > On Fri, Jul 28, 2017 at 1:46 PM, Jenkins
> > > > <jenkins-dontreply@opendaylight.org<mailto:jenkins-dontreply@opendaylight.org><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@opendaylight.org>>

> > > > > wrote:
> > > >
> > > > > Attention neutron-devs,
> > > > >
> > > > > Autorelease carbon failed to build features-neutron from
> > > > > neutron in build 401. Attached is a snippet of the error
> > > > > message related to the failure that we were able to
> > > > > automatically parse as well as console logs.
> > > > >
> > > > >
> > > > > Console Logs:
> > > > > https://logs.opendaylight.org/releng/jenkins092/autorelease-
> > > > > release-carbon/401
> > > > >
> > > > > Jenkins Build:
> > > > > https://jenkins.opendaylight.org/releng/job/autorelease-
> > > > > release-carbon/401/
> > > > >
> > > > > Please review and provide an ETA on when a fix will be
> > > > > available.
> > > > >
> > > > > Thanks,
> > > > > ODL releng/autorelease team


_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...rg
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



 

--

Thanks

Anil


_______________________________________________
release mailing list
release@...
https://lists.opendaylight.org/mailman/listinfo/release



_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...rg
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev




Vishal Thapar <vishal.thapar@...>
 

+1.

 

I think AAA issue might also be related to ovsdb.library as library depends on AAA for TLS. Let us try meerging [1], if it works, good. In the meantime I’ll try to get a cleaner patch ready.

 

Regards,

Vishal.

 

From: Thanh Ha [mailto:thanh.ha@...]
Sent: 03 August 2017 22:41
To: Sam Hague <shague@...>
Cc: Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; neutron-dev@...; Stephen Kitt <skitt@...>; isaku.yamahata@...
Subject: Re: [ovsdb-dev] [release] [neutron-dev] Autorelease carbon failed to build features-neutron from neutron

 

Agreed. With code freeze soon I think we should have a working Carbon release asap.

 

Regards,

Thanh

 

 

On Thu, Aug 3, 2017 at 1:08 PM, Sam Hague <shague@...> wrote:

Thanh,

 

we can just merge [1] which has the fix for neutron. The path those is failing sft on the AAA stuff though. I thought that was fixed though, so not sure what will happen if we merge [1].

 

We can work on a prettier fix later.

 

Sam

 

 

On Thu, Aug 3, 2017 at 12:06 PM, Thanh Ha <thanh.ha@...> wrote:

Any updates here?


We're still failing and Carbon code freeze is soon I think. We need a solution asap.

 

Regards,

Thanh

 

On Mon, Jul 31, 2017 at 10:49 PM, Vishal Thapar <vishal.thapar@...> wrote:

So, any agreement on a) How to address this issue in short term? And b) Proper way to do it long term?

 

Both can be the same, but we need to agree soon. Options I’ve seen discussed are:

  1. Make utils their own feature and have consumers explicitly use them. This will require changes to all dependent projects and will also require changes on Nitrogen.
    1. Optionally do the change only on Carbon.
  1. Add explicit dependency in broken projects [Neutron] to depend on library and only on Carbon. Karaf4 takes care of transitive dependencies for us on Nitrogen.
  2. Add library-features and utils to southbound-api on Carbon. This will isolate the fix to OVSDB itself and not require changes in any downstream project. Option to do the same on Nitrogen too, though not required fo now due to karaf4

 

Did I miss anything?

 

Regards,

Vishal.

 

From: Anil Vishnoi [mailto:vishnoianil@...]
Sent: 01 August 2017 00:14
To: Stephen Kitt <skitt@...>
Cc: Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; Anil Belur <abelur@...>; neutron-dev@...; isaku.yamahata@...
Subject: Re: [ovsdb-dev] [neutron-dev] [release] Autorelease carbon failed to build features-neutron from neutron

 

I think we had a long discussion on these utilities classes, but it was kind of inconclusive. Making utils code as a feature is probably right thing to do, but with that we are officially exposing all the utility methods as a Project API's. So we need to be bit careful on what API we are exposing to downstream consumer, because any changes in this will have to go through API deprecation process.

 

On Mon, Jul 31, 2017 at 7:45 AM, Stephen Kitt <skitt@...> wrote:

Ideally, changes made to a stable release shouldn’t involve changes in
feature dependencies...

But yes, we could add a utils feature (in Nitrogen and Carbon, why not
since we’ve already painted ourselves into a corner); that would allow
a bunch of cleanup in places pulling in the utils bundles.

Regards,

Stephen


On Mon, 31 Jul 2017 14:37:09 +0000
Vishal Thapar <vishal.thapar@...> wrote:

> Hi Sam,
>
> Yep, that is exactly what I had in mind too and left as comment on
> Stephen’s patch. This will likely require dependent projects to pull
> in utils feature. Alternative, for carbon at least, could be to
> append utils and library to southbound-api.
>
> Regards,
> Vishal.
>
> From: Sam Hague [mailto:shague@...]
> Sent: 31 July 2017 20:04
> To: Stephen Kitt <skitt@...>
> Cc: Vishal Thapar <vishal.thapar@...>;
> <ovsdb-dev@...>
> <ovsdb-dev@...>; Release
> (release@...) <release@...>;
> Isaku Yamahata <yamahata@...>;
> isaku.yamahata@...; neutron-dev@...; Anil
> Belur <abelur@...>; Jenkins
> <jenkins-dontreply@...> Subject: Re: [neutron-dev]
> [release] Autorelease carbon failed to build features-neutron from
> neutron
>
>
>
> On Mon, Jul 31, 2017 at 10:22 AM, Stephen Kitt
> <skitt@...<mailto:skitt@...>> wrote: Actually no,
> that’s got nothing to do with it; the problem is that
> southbound-utils now needs the ovsdb library, and since Karaf 3
> feature dependencies are manual, we need to add that explicitly.
>
> The Karaf 4 feature generator discovers this on its own, which is why
> this “just works” in Nitrogen.
> And does this work for netvirt and genius because they pull in the
> odl-ovsdb-southbound-impl feature which pulls in the
> odl-ovsdb-library?
>
> Seems like the original patch to add the dependency on
> southbound-utils to include the lib.notation may not have been best.
> it si hard for dependent projects to bring in transitive libs. Maybe
> the southbound-utils should be a feature and includes everything it
> needs so projects can simply include the feature?
>
> https://git.opendaylight.org/gerrit/60936
>
> Regards,
>
> Stephen
>
>
> On Mon, 31 Jul 2017 15:55:17 +0200
> Stephen Kitt <skitt@...<mailto:skitt@...>> wrote:
>
> > In Carbon, the odl-neutron-hostconfig-ovs feature is being generated
> > incorrectly, with Nitrogen dependencies. I’m trying to figure out
> > why that is...
> >
> > Regards,
> >
> > Stephen
> >
> >
> > On Mon, 31 Jul 2017 11:15:01 +0000
> > Vishal Thapar
> > <vishal.thapar@...<mailto:vishal.thapar@...>>
> > wrote:
> > > Hi Sam,
> > >
> > > Yeah, this fails on Carbon but Nitrogen is working fine. One key
> > > difference between Neutron and Genius/Netvirt is that Neutron
> > > doesn’t require southbound-impl. I suspect it is because library
> > > gets pulled in with southbound-impl but not API. Note that library
> > > has its own artifacts/features.
> > >
> > > If it were failing on Nitrogen too, would’ve been easier to figure
> > > out. I am guessing the way karaf4 changes work it is able to pull
> > > in library on Nitrogen, but not on Carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Sam Hague
> > > [mailto:shague@...<mailto:shague@...>] Sent: 31
> > > July 2017 16:40 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...>>
> > > Cc:
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...>>
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...>>;
> > > Release
> > > (release@...<mailto:release@...>)
> > > <release@...<mailto:release@...>>;
> > > Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...>;
> > > Stephen Kitt <skitt@...<mailto:skitt@...>>;
> > > neutron-dev@...<mailto:neutron-dev@...>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...>>;
> > > Jenkins
> > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > So this fails on carbon or is that backwards? Genius and netvirt
> > > both pull these in so it should work fine for neutron. Netvirt
> > > includes the feature as well as the artifact so maybe that is why
> > > it works.
> > >
> > > I will take a look today also.
> > >
> > > On Jul 31, 2017 5:12 AM, "Vishal Thapar"
> > > <vishal.thapar@...<mailto:vishal.thapar@...><mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
> > > wrote: Okay, it didn’t work.
> > >
> > > Sam, Stephen,
> > >
> > > Any ideas? I can’t figure out how ovsdb.utils.* are being
> > > published as I don’t see any features for it. Probably want to ad
> > > mor to features/pom and/or features.xml on carbon. Just wondering
> > > why it works on Nitrogen but not on carbon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Vishal Thapar
> > > Sent: 31 July 2017 14:19
> > > To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...><mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>;
> > > Anil Belur
> > > <abelur@...<mailto:abelur@...><mailto:abelur@...<mailto:abelur@...>>>
> > >
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@...><mailto:yamahata@...<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...>>>;
> > > Release
> > > <release@...<mailto:release@...><mailto:release@...<mailto:release@...>>>;
> > > neutron-dev@...<mailto:neutron-dev@...><mailto:neutron-dev@...<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@...>>
> > > Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Patch: https://git.opendaylight.org/gerrit/#/c/60930/
> > >
> > > It passed the autorelease verification, so assume it means good to
> > > go?
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From:
> > > neutron-dev-bounces@...<mailto:neutron-dev-bounces@...><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@...>>
> > > [mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@...>]
> > > On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
> > > <abelur@...<mailto:abelur@...><mailto:abelur@...<mailto:abelur@...>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@...><mailto:yamahata@...<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...>>>;
> > > Release
> > > <release@...<mailto:release@...><mailto:release@...<mailto:release@...>>>;
> > > neutron-dev@...<mailto:neutron-dev@...><mailto:neutron-dev@...<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hi Anil,
> > >
> > > I am working on a patch that I think might address it. Will be
> > > pushing it soon.
> > >
> > > Regards,
> > > Vishal.
> > >
> > > From: Anil Belur
> > > [mailto:abelur@...<mailto:abelur@...>]
> > > Sent: 31 July 2017 13:33 To: Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...><mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
> > > Cc: Isaku Yamahata
> > > <yamahata@...<mailto:yamahata@...><mailto:yamahata@...<mailto:yamahata@...>>>;
> > > Jenkins
> > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...>>>;
> > > Release
> > > <release@...<mailto:release@...><mailto:release@...<mailto:release@...>>>;
> > > neutron-dev@...<mailto:neutron-dev@...><mailto:neutron-dev@...<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > > Hello Isaku, Vishal:
> > >
> > > On Sat, Jul 29, 2017 at 3:35 PM, Vishal Thapar
> > > <vishal.thapar@...<mailto:vishal.thapar@...><mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
> > > wrote: This is cherry pick of patch from Nitrogen. We haven't seen
> > > such failures on Nitrogen, have we? What needs to be done may vary
> > > based on that. Besides netvirt also uses southbound utils, so
> > > wondering why didn't fail for Netvirt. Could this be a transient
> > > issue where autorelease ran while Carbon patch's merge job was in
> > > flight?
> > >
> > > Other question, shouldn't this have been caught by distribution
> > > check job? Looking into it, just trying to figure out how and why
> > > we didn't catch it sooner.
> > >
> > >
> > > Not sure if this is transient, since the same failure is seen
> > > across the last three AR builds. Could we get this resolved
> > > sooner, we like to see AR unblocked.
> > >
> > > Thanks,
> > > Anil
> > >
> > >
> > >
> > >
> > > -----Original Message-----
> > > From:
> > > neutron-dev-bounces@...<mailto:neutron-dev-bounces@...><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@...>>
> > > [mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@...><mailto:neutron-dev-bounces@...<mailto:neutron-dev-bounces@...>>]
> > > On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil
> > > Belur
> > > <abelur@...<mailto:abelur@...><mailto:abelur@...<mailto:abelur@...>>>
> > > Cc:
> > > yamahata@...<mailto:yamahata@...><mailto:yamahata@...<mailto:yamahata@...>>;
> > > Jenkins
> > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@...>>>;
> > > odl ovsdb dev
> > > <ovsdb-dev@...<mailto:ovsdb-dev@...><mailto:ovsdb-dev@...<mailto:ovsdb-dev@...>>>;
> > > Release
> > > <release@...<mailto:release@...><mailto:release@...<mailto:release@...>>>;
> > > neutron-dev@...<mailto:neutron-dev@...><mailto:neutron-dev@...<mailto:neutron-dev@...>>;
> > > isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:isaku.yamahata@...<mailto:isaku.yamahata@...>>
> > > Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
> > > build features-neutron from neutron
> > >
> > >
> > > It looks like ovsdb issue. Cc'ed to ovsdb-dev.
> > > ovsdb folks, can you please take a look?
> > >
> > >
> > > > Tests in error:
> > > >   Can't install feature odl-neutron-hostconfig-ovs/0.8.2-Carbon:
> > > >   Could not start bundle
> > > > mvn:org.opendaylight.ovsdb/utils.southbound-utils/1.4.2-Carbon
> > > > in feature(s) odl-ovsdb-southbound-test-1.4.2-Carbon,
> > > > odl-neutron-hostconfig-ovs-0.8.2-Carbon: The bundle
> > > > "org.opendaylight.ovsdb.utils.southbound-utils_1.4.2.Carbon
> > > > [293]" could not be resolved. Reason: Missing Constraint:
> > > > Import-Package: org.opendaylight.ovsdb.lib.notation;
> > > > version="[1.4.0,2.0.0)"
> > >
> > > The following changeset of ovsdb updates
> > > utils/southbound-utils/pom.xml which might be a culprit. I'm
> > > guessing other pom.xml or features.xml also needs update.
> > >
> > > > commit c9bbb9b3f71b867e0ebf5dc56ab6f143c0f5a8bd
> > > > Author: Victor Pickard
> > > > <vpickard@...<mailto:vpickard@...><mailto:vpickard@...<mailto:vpickard@...>>>
> > > > Date:   Tue Jul 25 17:20:19 2017 -0400
> > > >
> > > >     Bug 8476 Add support for logical switch replication mode
> > > >
> > > >     Add helper method to SouthboundUtils to check database
> > > >         version.
> > > ...
> > > > diff --git a/utils/southbound-utils/pom.xml
> > > > b/utils/southbound-utils/pom.xml index a16a419..6a3234f 100644
> > > > --- a/utils/southbound-utils/pom.xml
> > > > +++ b/utils/southbound-utils/pom.xml
> > > > @@ -50,6 +50,11 @@ and is available at
> > > > http://www.eclipse.org/legal/epl-v10.html
> > > > <groupId>org.opendaylight.mdsal.model</groupId>
> > > > <artifactId>ietf-topology</artifactId> </dependency>
> > > > +    <dependency>
> > > > +      <groupId>org.opendaylight.ovsdb</groupId>
> > > > +      <artifactId>library</artifactId>
> > > > +      <version>${project.version}</version>
> > > > +    </dependency>
> > > >      <!-- testing dependencies -->
> > > >      <dependency>
> > > >        <groupId>junit</groupId>
> > >
> > > thanks,
> > >
> > > On Fri, Jul 28, 2017 at 07:32:16PM +1000, Anil Belur
> > > <abelur@...<mailto:abelur@...><mailto:abelur@...<mailto:abelur@...>>>
> > > wrote:
> > >
> > > Hello neutron-dev,
> > >
> > > Autorelease Carbon has failed on SFT timeouts. Please fix them.
> > >
> > > > Running org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
> > > > 130.387 sec <<< FAILURE! - in
> > > > org.opendaylight.odlparent.featuretest.SingleFeatureTest
> > > > installFeature(org.opendaylight.odlparent.featuretest.SingleFeatureTest)[repoUrl:
> > > > file:/w/workspace/autorelease-release-carbon/neutron/features/producti
> > > > on/features-neutron/target/classes/features.xml,
> > > > Feature: odl-neutron-hostconfig-ovs 0.8.2-Carbon]  Time elapsed:
> > > > 16.327 sec  <<< ERROR!
> > > >
> > > > Thanks,
> > > > Anil
> > > >
> > > > On Fri, Jul 28, 2017 at 1:46 PM, Jenkins
> > > > <jenkins-dontreply@...<mailto:jenkins-dontreply@...><mailto:jenkins-dontreply@...<mailto:jenkins-dontreply@...>>

> > > > > wrote:
> > > >
> > > > > Attention neutron-devs,
> > > > >
> > > > > Autorelease carbon failed to build features-neutron from
> > > > > neutron in build 401. Attached is a snippet of the error
> > > > > message related to the failure that we were able to
> > > > > automatically parse as well as console logs.
> > > > >
> > > > >
> > > > > Console Logs:
> > > > > https://logs.opendaylight.org/releng/jenkins092/autorelease-
> > > > > release-carbon/401
> > > > >
> > > > > Jenkins Build:
> > > > > https://jenkins.opendaylight.org/releng/job/autorelease-
> > > > > release-carbon/401/
> > > > >
> > > > > Please review and provide an ETA on when a fix will be
> > > > > available.
> > > > >
> > > > > Thanks,
> > > > > ODL releng/autorelease team


_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



 

--

Thanks

Anil

 

_______________________________________________
release mailing list
release@...
https://lists.opendaylight.org/mailman/listinfo/release

 


_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev

 

 


Isaku Yamahata
 

Nitrogen branch also is hitting the similar issue.


On Thu, Aug 03, 2017 at 01:08:35PM -0400,
Sam Hague <shague@...> wrote:

Thanh,

we can just merge [1] which has the fix for neutron. The path those is
failing sft on the AAA stuff though. I thought that was fixed though, so
not sure what will happen if we merge [1].

We can work on a prettier fix later.

Sam

[1] https://git.opendaylight.org/gerrit/#/c/60936/

On Thu, Aug 3, 2017 at 12:06 PM, Thanh Ha <thanh.ha@...>
wrote:

Any updates here?

We're still failing and Carbon code freeze is soon I think. We need a
solution asap.

Regards,
Thanh

On Mon, Jul 31, 2017 at 10:49 PM, Vishal Thapar <
vishal.thapar@...> wrote:

So, any agreement on a) How to address this issue in short term? And b)
Proper way to do it long term?



Both can be the same, but we need to agree soon. Options I’ve seen
discussed are:

1. Make utils their own feature and have consumers explicitly use
them. This will require changes to all dependent projects and will also
require changes on Nitrogen.
1. Optionally do the change only on Carbon.
2. Add explicit dependency in broken projects [Neutron] to depend on
library and only on Carbon. Karaf4 takes care of transitive dependencies
for us on Nitrogen.
3. Add library-features and utils to southbound-api on Carbon. This
will isolate the fix to OVSDB itself and not require changes in any
downstream project. Option to do the same on Nitrogen too, though not
required fo now due to karaf4



Did I miss anything?



Regards,

Vishal.



*From:* Anil Vishnoi [mailto:vishnoianil@...]
*Sent:* 01 August 2017 00:14
*To:* Stephen Kitt <skitt@...>
*Cc:* Vishal Thapar <vishal.thapar@...>; Isaku Yamahata <
yamahata@...>; Jenkins <jenkins-dontreply@opendayligh
t.org>; <ovsdb-dev@...> <ovsdb-dev@....
org>; Release (release@...) <
release@...>; Anil Belur <abelur@...>;
neutron-dev@...; isaku.yamahata@...
*Subject:* Re: [ovsdb-dev] [neutron-dev] [release] Autorelease carbon
failed to build features-neutron from neutron



I think we had a long discussion on these utilities classes, but it was
kind of inconclusive. Making utils code as a feature is probably right
thing to do, but with that we are officially exposing all the utility
methods as a Project API's. So we need to be bit careful on what API we are
exposing to downstream consumer, because any changes in this will have to
go through API deprecation process.



On Mon, Jul 31, 2017 at 7:45 AM, Stephen Kitt <skitt@...> wrote:

Ideally, changes made to a stable release shouldn’t involve changes in
feature dependencies...

But yes, we could add a utils feature (in Nitrogen and Carbon, why not
since we’ve already painted ourselves into a corner); that would allow
a bunch of cleanup in places pulling in the utils bundles.

Regards,

Stephen


On Mon, 31 Jul 2017 14:37:09 +0000
Vishal Thapar <vishal.thapar@...> wrote:

Hi Sam,

Yep, that is exactly what I had in mind too and left as comment on
Stephen’s patch. This will likely require dependent projects to pull
in utils feature. Alternative, for carbon at least, could be to
append utils and library to southbound-api.

Regards,
Vishal.

From: Sam Hague [mailto:shague@...]
Sent: 31 July 2017 20:04
To: Stephen Kitt <skitt@...>
Cc: Vishal Thapar <vishal.thapar@...>;
<ovsdb-dev@...>
<ovsdb-dev@...>; Release
(release@...) <release@...>;
Isaku Yamahata <yamahata@...>;
isaku.yamahata@...; neutron-dev@...; Anil
Belur <abelur@...>; Jenkins
<jenkins-dontreply@...> Subject: Re: [neutron-dev]
[release] Autorelease carbon failed to build features-neutron from
neutron



On Mon, Jul 31, 2017 at 10:22 AM, Stephen Kitt
<skitt@...<mailto:skitt@...>> wrote: Actually no,
that’s got nothing to do with it; the problem is that
southbound-utils now needs the ovsdb library, and since Karaf 3
feature dependencies are manual, we need to add that explicitly.

The Karaf 4 feature generator discovers this on its own, which is why
this “just works” in Nitrogen.
And does this work for netvirt and genius because they pull in the
odl-ovsdb-southbound-impl feature which pulls in the
odl-ovsdb-library?

Seems like the original patch to add the dependency on
southbound-utils to include the lib.notation may not have been best.
it si hard for dependent projects to bring in transitive libs. Maybe
the southbound-utils should be a feature and includes everything it
needs so projects can simply include the feature?

https://git.opendaylight.org/gerrit/60936

Regards,

Stephen


On Mon, 31 Jul 2017 15:55:17 +0200
Stephen Kitt <skitt@...<mailto:skitt@...>> wrote:

In Carbon, the odl-neutron-hostconfig-ovs feature is being generated
incorrectly, with Nitrogen dependencies. I’m trying to figure out
why that is...

Regards,

Stephen


On Mon, 31 Jul 2017 11:15:01 +0000
Vishal Thapar
<vishal.thapar@...<mailto:vishal.thapar@...>>
wrote:
Hi Sam,

Yeah, this fails on Carbon but Nitrogen is working fine. One key
difference between Neutron and Genius/Netvirt is that Neutron
doesn’t require southbound-impl. I suspect it is because library
gets pulled in with southbound-impl but not API. Note that library
has its own artifacts/features.

If it were failing on Nitrogen too, would’ve been easier to figure
out. I am guessing the way karaf4 changes work it is able to pull
in library on Nitrogen, but not on Carbon.

Regards,
Vishal.

From: Sam Hague
[mailto:shague@...<mailto:shague@...>] Sent: 31
July 2017 16:40 To: Vishal Thapar
<vishal.thapar@...<mailto:vishal.thapar@...>>
Cc:
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org>>
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org>>;
Release
(release@...<mailto:release@...
light.org>)
<release@...<mailto:release@...
light.org>>;
Isaku Yamahata
<yamahata@...<mailto:yamahata@...
;
isaku.yamahata@...<mailto:isaku.yamahata@...>;
Stephen Kitt <skitt@...<mailto:skitt@...>>;
neutron-dev@...<mailto:neutron-dev@lists.
opendaylight.org>;
Anil Belur
<abelur@...<mailto:abelur@...>>;
Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...>>
Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
build features-neutron from neutron

So this fails on carbon or is that backwards? Genius and netvirt
both pull these in so it should work fine for neutron. Netvirt
includes the feature as well as the artifact so maybe that is why
it works.

I will take a look today also.

On Jul 31, 2017 5:12 AM, "Vishal Thapar"
<vishal.thapar@...<mailto:vishal.thapar@...
<mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
wrote: Okay, it didn’t work.

Sam, Stephen,

Any ideas? I can’t figure out how ovsdb.utils.* are being
published as I don’t see any features for it. Probably want to ad
mor to features/pom and/or features.xml on carbon. Just wondering
why it works on Nitrogen but not on carbon.

Regards,
Vishal.

From: Vishal Thapar
Sent: 31 July 2017 14:19
To: Vishal Thapar
<vishal.thapar@...<mailto:vishal.thapar@...
<mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>;
Anil Belur
<abelur@...<mailto:abelur@...
<mailto:abelur@...<mailto:abelur@...>>>

Cc: Isaku Yamahata
<yamahata@...<mailto:yamahata@...
<mailto:yamahata@...<mailto:yamahata@...
;
Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...><mailto:jenkins-dontreply@...
<mailto:jenkins-dontreply@...>>>;
odl ovsdb dev
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org><mailto:ovsdb-dev@...<mailto:
ovsdb-dev@...>>>;
Release
<release@...<mailto:release@...
light.org><mailto:release@...<mailto:
release@...>>>;
neutron-dev@...<mailto:neutron-dev@lists.
opendaylight.org><mailto:neutron-dev@...<mailto:
neutron-dev@...>>;
isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:
isaku.yamahata@...<mailto:isaku.yamahata@...>>
Subject: RE: [neutron-dev] [release] Autorelease carbon failed to
build features-neutron from neutron

Patch: https://git.opendaylight.org/gerrit/#/c/60930/

It passed the autorelease verification, so assume it means good to
go?

Regards,
Vishal.

From:
neutron-dev-bounces@...<mailto:neutron-
dev-bounces@...><mailto:neutron-dev-
bounces@...<mailto:neutron-dev-bounces@
lists.opendaylight.org>>
[mailto:neutron-dev-bounces@...<mailto:ne
utron-dev-bounces@...>]
On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
<abelur@...<mailto:abelur@...
<mailto:abelur@...<mailto:abelur@...>>>
Cc: Isaku Yamahata
<yamahata@...<mailto:yamahata@...
<mailto:yamahata@...<mailto:yamahata@...
;
Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...><mailto:jenkins-dontreply@...
<mailto:jenkins-dontreply@...>>>;
odl ovsdb dev
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org><mailto:ovsdb-dev@...<mailto:
ovsdb-dev@...>>>;
Release
<release@...<mailto:release@...
light.org><mailto:release@...<mailto:
release@...>>>;
neutron-dev@...<mailto:neutron-dev@lists.
opendaylight.org><mailto:neutron-dev@...<mailto:
neutron-dev@...>>;
isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:
isaku.yamahata@...<mailto:isaku.yamahata@...>>
Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
build features-neutron from neutron

Hi Anil,

I am working on a patch that I think might address it. Will be
pushing it soon.

Regards,
Vishal.

From: Anil Belur
[mailto:abelur@...<mailto:abelur@
linuxfoundation.org>]
Sent: 31 July 2017 13:33 To: Vishal Thapar
<vishal.thapar@...<mailto:vishal.thapar@...
<mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
Cc: Isaku Yamahata
<yamahata@...<mailto:yamahata@...
<mailto:yamahata@...<mailto:yamahata@...
;
Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...><mailto:jenkins-dontreply@...
<mailto:jenkins-dontreply@...>>>;
odl ovsdb dev
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org><mailto:ovsdb-dev@...<mailto:
ovsdb-dev@...>>>;
Release
<release@...<mailto:release@...
light.org><mailto:release@...<mailto:
release@...>>>;
neutron-dev@...<mailto:neutron-dev@lists.
opendaylight.org><mailto:neutron-dev@...<mailto:
neutron-dev@...>>;
isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:
isaku.yamahata@...<mailto:isaku.yamahata@...>>
Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
build features-neutron from neutron

Hello Isaku, Vishal:

On Sat, Jul 29, 2017 at 3:35 PM, Vishal Thapar
<vishal.thapar@...<mailto:vishal.thapar@...
<mailto:vishal.thapar@...<mailto:vishal.thapar@...>>>
wrote: This is cherry pick of patch from Nitrogen. We haven't seen
such failures on Nitrogen, have we? What needs to be done may vary
based on that. Besides netvirt also uses southbound utils, so
wondering why didn't fail for Netvirt. Could this be a transient
issue where autorelease ran while Carbon patch's merge job was in
flight?

Other question, shouldn't this have been caught by distribution
check job? Looking into it, just trying to figure out how and why
we didn't catch it sooner.


Not sure if this is transient, since the same failure is seen
across the last three AR builds. Could we get this resolved
sooner, we like to see AR unblocked.

Thanks,
Anil




-----Original Message-----
From:
neutron-dev-bounces@...<mailto:neutron-
dev-bounces@...><mailto:neutron-dev-
bounces@...<mailto:neutron-dev-bounces@
lists.opendaylight.org>>
[mailto:neutron-dev-bounces@...<mailto:ne
utron-dev-bounces@...><mailto:neutron-
dev-bounces@...<mailto:neutron-dev-
bounces@...>>]
On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil
Belur
<abelur@...<mailto:abelur@...
<mailto:abelur@...<mailto:abelur@...>>>
Cc:
yamahata@...<mailto:yamahata@...
<mailto:yamahata@...<mailto:yamahata@...
;
Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...><mailto:jenkins-dontreply@...
<mailto:jenkins-dontreply@...>>>;
odl ovsdb dev
<ovsdb-dev@...<mailto:ovsdb-dev@...
ndaylight.org><mailto:ovsdb-dev@...<mailto:
ovsdb-dev@...>>>;
Release
<release@...<mailto:release@...
light.org><mailto:release@...<mailto:
release@...>>>;
neutron-dev@...<mailto:neutron-dev@lists.
opendaylight.org><mailto:neutron-dev@...<mailto:
neutron-dev@...>>;
isaku.yamahata@...<mailto:isaku.yamahata@...><mailto:
isaku.yamahata@...<mailto:isaku.yamahata@...>>
Subject: Re: [neutron-dev] [release] Autorelease carbon failed to
build features-neutron from neutron


It looks like ovsdb issue. Cc'ed to ovsdb-dev.
ovsdb folks, can you please take a look?


Tests in error:
Can't install feature odl-neutron-hostconfig-ovs/0.8.2-Carbon:
Could not start bundle
mvn:org.opendaylight.ovsdb/utils.southbound-utils/1.4.2-Carbon
in feature(s) odl-ovsdb-southbound-test-1.4.2-Carbon,
odl-neutron-hostconfig-ovs-0.8.2-Carbon: The bundle
"org.opendaylight.ovsdb.utils.southbound-utils_1.4.2.Carbon
[293]" could not be resolved. Reason: Missing Constraint:
Import-Package: org.opendaylight.ovsdb.lib.notation;
version="[1.4.0,2.0.0)"
The following changeset of ovsdb updates
utils/southbound-utils/pom.xml which might be a culprit. I'm
guessing other pom.xml or features.xml also needs update.

commit c9bbb9b3f71b867e0ebf5dc56ab6f143c0f5a8bd
Author: Victor Pickard
<vpickard@...<mailto:vpickard@...><mailto:vpic
kard@...<mailto:vpickard@...>>>
Date: Tue Jul 25 17:20:19 2017 -0400

Bug 8476 Add support for logical switch replication mode

Add helper method to SouthboundUtils to check database
version.
...
diff --git a/utils/southbound-utils/pom.xml
b/utils/southbound-utils/pom.xml index a16a419..6a3234f 100644
--- a/utils/southbound-utils/pom.xml
+++ b/utils/southbound-utils/pom.xml
@@ -50,6 +50,11 @@ and is available at
http://www.eclipse.org/legal/epl-v10.html
<groupId>org.opendaylight.mdsal.model</groupId>
<artifactId>ietf-topology</artifactId> </dependency>
+ <dependency>
+ <groupId>org.opendaylight.ovsdb</groupId>
+ <artifactId>library</artifactId>
+ <version>${project.version}</version>
+ </dependency>
<!-- testing dependencies -->
<dependency>
<groupId>junit</groupId>
thanks,

On Fri, Jul 28, 2017 at 07:32:16PM +1000, Anil Belur
<abelur@...<mailto:abelur@...
<mailto:abelur@...<mailto:abelur@...>>>
wrote:

Hello neutron-dev,

Autorelease Carbon has failed on SFT timeouts. Please fix them.

Running org.opendaylight.odlparent.featuretest.SingleFeatureTest
Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
130.387 sec <<< FAILURE! - in
org.opendaylight.odlparent.featuretest.SingleFeatureTest
installFeature(org.opendaylight.odlparent.featuretest.
SingleFeatureTest)[repoUrl:
file:/w/workspace/autorelease-release-carbon/neutron/feature
s/producti
on/features-neutron/target/classes/features.xml,
Feature: odl-neutron-hostconfig-ovs 0.8.2-Carbon] Time elapsed:
16.327 sec <<< ERROR!

Thanks,
Anil

On Fri, Jul 28, 2017 at 1:46 PM, Jenkins
<jenkins-dontreply@...<mailto:jenkins-
dontreply@...><mailto:jenkins-dontreply@...
<mailto:jenkins-dontreply@...>>

wrote:
Attention neutron-devs,

Autorelease carbon failed to build features-neutron from
neutron in build 401. Attached is a snippet of the error
message related to the failure that we were able to
automatically parse as well as console logs.


Console Logs:
https://logs.opendaylight.org/releng/jenkins092/autorelease-
release-carbon/401

Jenkins Build:
https://jenkins.opendaylight.org/releng/job/autorelease-
release-carbon/401/

Please review and provide an ETA on when a fix will be
available.

Thanks,
ODL releng/autorelease team

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev





--

Thanks

Anil

_______________________________________________
release mailing list
release@...
https://lists.opendaylight.org/mailman/listinfo/release

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev

--
Isaku Yamahata <isaku.yamahata@...>