Isaku Yamahata <yamahata@...>
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@...> 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@...> 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/production/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@...
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
_______________________________________________ release mailing list release@... https://lists.opendaylight.org/mailman/listinfo/release
_______________________________________________ neutron-dev mailing list neutron-dev@... https://lists.opendaylight.org/mailman/listinfo/neutron-dev -- Isaku Yamahata <isaku.yamahata@...>
|
Vishal Thapar <vishal.thapar@...>
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.
Regards, Vishal.
toggle quoted message
Show quoted text
-----Original Message----- From: neutron-dev-bounces@... [mailto:neutron-dev-bounces@...] On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil Belur <abelur@...> Cc: yamahata@...; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>; Release <release@...>; neutron-dev@...; 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@...> 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@...> 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@...
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
_______________________________________________ release mailing list release@... https://lists.opendaylight.org/mailman/listinfo/release
_______________________________________________ neutron-dev mailing list neutron-dev@... https://lists.opendaylight.org/mailman/listinfo/neutron-dev -- Isaku Yamahata <isaku.yamahata@...> _______________________________________________ neutron-dev mailing list neutron-dev@... https://lists.opendaylight.org/mailman/listinfo/neutron-dev
|
|
Vishal Thapar <vishal.thapar@...>
Hi Anil,
I am working on a patch that I think might address it. Will be pushing it soon.
Regards,
Vishal.
toggle quoted message
Show quoted text
From: Anil Belur [mailto:abelur@...]
Sent: 31 July 2017 13:33
To: Vishal Thapar <vishal.thapar@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>; Release <release@...>; neutron-dev@...; 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@...> 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.
-----Original Message-----
From: neutron-dev-bounces@... [mailto:neutron-dev-bounces@...] On Behalf Of Isaku Yamahata
Sent: 29 July 2017 01:09
To: Anil Belur <abelur@...>
Cc: yamahata@...; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...>
> 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@...> 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@...
> > 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
> >
> >
> > _______________________________________________
> > release mailing list
> > release@...
> >
https://lists.opendaylight.org/mailman/listinfo/release
> >
> >
> _______________________________________________
> neutron-dev mailing list
> neutron-dev@...
>
https://lists.opendaylight.org/mailman/listinfo/neutron-dev
--
Isaku Yamahata <isaku.yamahata@...> _______________________________________________
|
Vishal Thapar <vishal.thapar@...>
toggle quoted message
Show quoted text
From: neutron-dev-bounces@... [mailto:neutron-dev-bounces@...]
On Behalf Of Vishal Thapar
Sent: 31 July 2017 13:55
To: Anil Belur <abelur@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>; Release <release@...>; neutron-dev@...; 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@...]
Sent: 31 July 2017 13:33
To: Vishal Thapar <vishal.thapar@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...> 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.
-----Original Message-----
From: neutron-dev-bounces@... [mailto:neutron-dev-bounces@...] On Behalf Of Isaku Yamahata
Sent: 29 July 2017 01:09
To: Anil Belur <abelur@...>
Cc: yamahata@...; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...>
> 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@...> 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@...
> > 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
> >
> >
> > _______________________________________________
> > release mailing list
> > release@...
> >
https://lists.opendaylight.org/mailman/listinfo/release
> >
> >
> _______________________________________________
> neutron-dev mailing list
> neutron-dev@...
>
https://lists.opendaylight.org/mailman/listinfo/neutron-dev
--
Isaku Yamahata <isaku.yamahata@...> _______________________________________________
|
Vishal Thapar <vishal.thapar@...>
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.
toggle quoted message
Show quoted text
From: Vishal Thapar
Sent: 31 July 2017 14:19
To: Vishal Thapar <vishal.thapar@...>; Anil Belur <abelur@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>; Release <release@...>; neutron-dev@...; 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.
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@...]
Sent: 31 July 2017 13:33
To: Vishal Thapar <vishal.thapar@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...> 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.
-----Original Message-----
From: neutron-dev-bounces@... [mailto:neutron-dev-bounces@...] On Behalf Of Isaku Yamahata
Sent: 29 July 2017 01:09
To: Anil Belur <abelur@...>
Cc: yamahata@...; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...>
> 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@...> 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@...
> > 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
> >
> >
> > _______________________________________________
> > release mailing list
> > release@...
> >
https://lists.opendaylight.org/mailman/listinfo/release
> >
> >
> _______________________________________________
> neutron-dev mailing list
> neutron-dev@...
>
https://lists.opendaylight.org/mailman/listinfo/neutron-dev
--
Isaku Yamahata <isaku.yamahata@...> _______________________________________________
|
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.
toggle quoted message
Show quoted text
On Jul 31, 2017 5:12 AM, "Vishal Thapar" < 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@...>; Anil Belur <abelur@...> 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.
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]
Sent: 31 July 2017 13:33
To: Vishal Thapar <vishal.thapar@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@opendaylight.org>; odl ovsdb dev <ovsdb-dev@....org>;
Release <release@....org>;
neutron-dev@lists.opendaylight.org;
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@...> 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.
-----Original Message-----
From: 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@...>
Cc: yamahata@...; Jenkins <jenkins-dontreply@opendaylight.org>; odl ovsdb dev <ovsdb-dev@....org>;
Release <release@....org>;
neutron-dev@lists.opendaylight.org;
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@...>
> 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@...> 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
> > 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
> >
> >
> > _______________________________________________
> > release mailing list
> > release@...
> >
https://lists.opendaylight.org/mailman/listinfo/release
> >
> >
> _______________________________________________
> neutron-dev mailing list
> neutron-dev@lists.opendaylight.org
>
https://lists.opendaylight.org/mailman/listinfo/neutron-dev
--
Isaku Yamahata <isaku.yamahata@...> _______________________________________________
|
Vishal Thapar <vishal.thapar@...>
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.
toggle quoted message
Show quoted text
From: Sam Hague [mailto:shague@...]
Sent: 31 July 2017 16:40
To: Vishal Thapar <vishal.thapar@...>
Cc: <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; Isaku Yamahata <yamahata@...>; isaku.yamahata@...; Stephen Kitt <skitt@...>;
neutron-dev@...; Anil Belur <abelur@...>; Jenkins <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@...> 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@...>; Anil Belur <abelur@...>
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.
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@...]
Sent: 31 July 2017 13:33
To: Vishal Thapar <vishal.thapar@...>
Cc: Isaku Yamahata <yamahata@...>; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...> 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.
-----Original Message-----
From:
neutron-dev-bounces@... [mailto:neutron-dev-bounces@...] On Behalf Of Isaku Yamahata
Sent: 29 July 2017 01:09
To: Anil Belur <abelur@...>
Cc: yamahata@...; Jenkins <jenkins-dontreply@...>; odl ovsdb dev <ovsdb-dev@...>;
Release <release@...>;
neutron-dev@...;
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@...>
> 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@...> 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@...
> > 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
> >
> >
> > _______________________________________________
> > release mailing list
> > release@...
> >
https://lists.opendaylight.org/mailman/listinfo/release
> >
> >
> _______________________________________________
> neutron-dev mailing list
> neutron-dev@...
>
https://lists.opendaylight.org/mailman/listinfo/neutron-dev
--
Isaku Yamahata <isaku.yamahata@...> _______________________________________________
|
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@...> 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@...] Sent: 31 July 2017 16:40 To: Vishal Thapar <vishal.thapar@...> Cc: <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; Isaku Yamahata <yamahata@...>; isaku.yamahata@...; Stephen Kitt <skitt@...>; neutron-dev@...; Anil Belur <abelur@...>; Jenkins <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@...>> 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@...>>; Anil Belur <abelur@...<mailto:abelur@...>>
Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] Sent: 31 July 2017 13:33 To: Vishal Thapar <vishal.thapar@...<mailto:vishal.thapar@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>] On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: yamahata@...<mailto:yamahata@...>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>> 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@...>
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
|
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. https://git.opendaylight.org/gerrit/60936Regards, Stephen On Mon, 31 Jul 2017 15:55:17 +0200 Stephen Kitt <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@...> 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@...] Sent: 31 July 2017 16:40 To: Vishal Thapar <vishal.thapar@...> Cc: <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; Isaku Yamahata <yamahata@...>; isaku.yamahata@...; Stephen Kitt <skitt@...>; neutron-dev@...; Anil Belur <abelur@...>; Jenkins <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@...>> 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@...>>; Anil Belur <abelur@...<mailto:abelur@...>>
Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] Sent: 31 July 2017 13:33 To: Vishal Thapar <vishal.thapar@...<mailto:vishal.thapar@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>] On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: yamahata@...<mailto:yamahata@...>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>> 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@...>
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
|
|
Vishal Thapar <vishal.thapar@...>
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.
toggle quoted message
Show quoted text
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@...> 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@...> 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@...> 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@...]
> > Sent: 31 July 2017 16:40
> > To: Vishal Thapar <vishal.thapar@...>
> > Cc: <ovsdb-dev@...>
> > <ovsdb-dev@...>; Release
> > (release@...) <release@...>;
> > Isaku Yamahata <yamahata@...>;
> > isaku.yamahata@...; Stephen Kitt <skitt@...>;
> > neutron-dev@...; Anil Belur
> > <abelur@...>; Jenkins
> > <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@...>>
> > 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@...>>;
> > Anil Belur
> > <abelur@...<mailto:abelur@...>>
> >
> > Cc: Isaku Yamahata
> > <yamahata@...<mailto:yamahata@...>>;
> > Jenkins
> > <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>;
> > odl ovsdb dev
> > <ovsdb-dev@...<mailto:ovsdb-dev@...>>;
> > Release
> > <release@...<mailto:release@...>>;
> > neutron-dev@...<mailto:neutron-dev@...>;
> > 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@...] On Behalf Of
> > Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur
> > <abelur@...<mailto:abelur@...>> Cc:
> > Isaku Yamahata
> > <yamahata@...<mailto:yamahata@...>>;
> > Jenkins
> > <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>;
> > odl ovsdb dev
> > <ovsdb-dev@...<mailto:ovsdb-dev@...>>;
> > Release
> > <release@...<mailto:release@...>>;
> > neutron-dev@...<mailto:neutron-dev@...>;
> > 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@...]
> > Sent: 31 July 2017 13:33
> > To: Vishal Thapar
> > <vishal.thapar@...<mailto:vishal.thapar@...>> Cc:
> > Isaku Yamahata
> > <yamahata@...<mailto:yamahata@...>>;
> > Jenkins
> > <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>;
> > odl ovsdb dev
> > <ovsdb-dev@...<mailto:ovsdb-dev@...>>;
> > Release
> > <release@...<mailto:release@...>>;
> > neutron-dev@...<mailto:neutron-dev@...>;
> > 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@...>>
> > 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@...>]
> > On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil Belur
> > <abelur@...<mailto:abelur@...>> Cc:
> > yamahata@...<mailto:yamahata@...>;
> > Jenkins
> > <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>;
> > odl ovsdb dev
> > <ovsdb-dev@...<mailto:ovsdb-dev@...>>;
> > Release
> > <release@...<mailto:release@...>>;
> > neutron-dev@...<mailto:neutron-dev@...>;
> > 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@...>> 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@...>>
> > 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@...>
> > > > 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
|
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
|
Isaku Yamahata <yamahata@...>
Wow, thank you for tracking it down to cook a patch. On Mon, Jul 31, 2017 at 04:22:27PM +0200, Stephen Kitt <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.
https://git.opendaylight.org/gerrit/60936
Regards,
Stephen
On Mon, 31 Jul 2017 15:55:17 +0200 Stephen Kitt <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@...> 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@...] Sent: 31 July 2017 16:40 To: Vishal Thapar <vishal.thapar@...> Cc: <ovsdb-dev@...> <ovsdb-dev@...>; Release (release@...) <release@...>; Isaku Yamahata <yamahata@...>; isaku.yamahata@...; Stephen Kitt <skitt@...>; neutron-dev@...; Anil Belur <abelur@...>; Jenkins <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@...>> 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@...>>; Anil Belur <abelur@...<mailto:abelur@...>>
Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] On Behalf Of Vishal Thapar Sent: 31 July 2017 13:55 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...] Sent: 31 July 2017 13:33 To: Vishal Thapar <vishal.thapar@...<mailto:vishal.thapar@...>> Cc: Isaku Yamahata <yamahata@...<mailto:yamahata@...>>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>] On Behalf Of Isaku Yamahata Sent: 29 July 2017 01:09 To: Anil Belur <abelur@...<mailto:abelur@...>> Cc: yamahata@...<mailto:yamahata@...>; Jenkins <jenkins-dontreply@...<mailto:jenkins-dontreply@...>>; odl ovsdb dev <ovsdb-dev@...<mailto:ovsdb-dev@...>>; Release <release@...<mailto:release@...>>; neutron-dev@...<mailto:neutron-dev@...>; 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@...>> 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@...>> 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@...>
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 -- Isaku Yamahata <isaku.yamahata@...>
|
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.
toggle quoted message
Show quoted text
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
|
Vishal Thapar <vishal.thapar@...>
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:
-
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.
-
Optionally do the change only on Carbon.
-
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.
-
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.
toggle quoted message
Show quoted text
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
--
|