OpenDaylight JIRA

OpenDaylight JIRA https://jira.opendaylight.org This file is an XML representation of an issue en-us

8.20.10

820010

22-06-2022

[ALTO-7] simple-impl failing build unable to find controler "sal" artifact https://jira.opendaylight.org/browse/ALTO-7 alto <p>The POM for org.opendaylight.controller:sal:jar:0.10.0-Daily-v201507071642 is missing, no dependency information available</p> <p>The above message came from the autorelease build of master:</p> <p><a href="https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/81/org.opendaylight.alto$simple-impl/console" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/81/org.opendaylight.alto$simple-impl/console</a></p> <p>Operating System: All<br/> Platform: All</p> ALTO-7

simple-impl failing build unable to find controler "sal" artifact

Bug Resolved Done Unassigned Thanh Ha (zxiiro) Tue, 7 Jul 2015 19:58:32 +0000 Thu, 19 Oct 2017 21:25:13 +0000 Mon, 20 Jul 2015 17:04:35 +0000 unspecified General 0 2 <p>This bug is cause by a AD-SAL dependency in alto-northbound and simple-alto. The dependency on AD-SAL is planned to be removed in next release, so we will just modify the pom file to depend on a stable release version of AD-SAL package to solve this problem for now.</p> <p><a href="https://git.opendaylight.org/gerrit/24152" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/24152</a></p> <p>According to Kai's comment, if the autorelease is using the snapshot repository, maven would not be able to find the packages with the 'lithium' release. I'm not sure if it is the case but if the fix doesn't work out I will find a better way to solve this problem.</p> <p>(In reply to Shu Dong from comment #3)<br/> > According to Kai's comment, if the autorelease is using the snapshot<br/> > repository, maven would not be able to find the packages with the 'lithium'<br/> > release. I'm not sure if it is the case but if the fix doesn't work out I<br/> > will find a better way to solve this problem.</p> <p>That's right, autorelease builds perform release type builds and thus does not depend on any opendaylight artifacts from the Internet or Nexus. All ODL artifacts must be resolveable and buildable from within the autorelease build itself. So I am -1 on the proposed change.</p> <p>Since AD-SAL is being removed from Beryllium you must adapt alto to not use this outdated artifact.</p> <p>(In reply to Thanh Ha from comment #4)<br/> > (In reply to Shu Dong from comment #3)<br/> > > According to Kai's comment, if the autorelease is using the snapshot<br/> > > repository, maven would not be able to find the packages with the 'lithium'<br/> > > release. I'm not sure if it is the case but if the fix doesn't work out I<br/> > > will find a better way to solve this problem.<br/> > <br/> > That's right, autorelease builds perform release type builds and thus does<br/> > not depend on any opendaylight artifacts from the Internet or Nexus. All ODL<br/> > artifacts must be resolveable and buildable from within the autorelease<br/> > build itself. So I am -1 on the proposed change.<br/> > <br/> > Since AD-SAL is being removed from Beryllium you must adapt alto to not use<br/> > this outdated artifact.</p> <p>Please review the latest fix: <a href="https://git.opendaylight.org/gerrit/24189" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/24189</a></p> <p>(In reply to Thanh Ha from comment #4)<br/> > (In reply to Shu Dong from comment #3)<br/> > > According to Kai's comment, if the autorelease is using the snapshot<br/> > > repository, maven would not be able to find the packages with the 'lithium'<br/> > > release. I'm not sure if it is the case but if the fix doesn't work out I<br/> > > will find a better way to solve this problem.<br/> > <br/> > That's right, autorelease builds perform release type builds and thus does<br/> > not depend on any opendaylight artifacts from the Internet or Nexus. All ODL<br/> > artifacts must be resolveable and buildable from within the autorelease<br/> > build itself. So I am -1 on the proposed change.<br/> > <br/> > Since AD-SAL is being removed from Beryllium you must adapt alto to not use<br/> > this outdated artifact.</p> <p>I noticed that you voted +1 on my latest fix. Thanks very much for your code review! So does it mean next we should merge the patch to master by ourselves to see if it works?</p> <p>(In reply to Shu Dong from comment #6)<br/> > I noticed that you voted +1 on my latest fix. Thanks very much for your code<br/> > review! So does it mean next we should merge the patch to master by<br/> > ourselves to see if it works?</p> <p>Yes please do, I'm not a Java expert so I can't say if the fix is good but once you merge it we can kick off an autorelease build to see if it passes.</p> <p>(In reply to Thanh Ha from comment #7)<br/> > (In reply to Shu Dong from comment #6)<br/> > > I noticed that you voted +1 on my latest fix. Thanks very much for your code<br/> > > review! So does it mean next we should merge the patch to master by<br/> > > ourselves to see if it works?<br/> > <br/> > Yes please do, I'm not a Java expert so I can't say if the fix is good but<br/> > once you merge it we can kick off an autorelease build to see if it passes.</p> <p>Thanks for your explanation. I have merged the fix into master. Please feel free to kick off a new autorelease build to check if it solves the problem. Thanks a lot!</p> <p>(In reply to Thanh Ha from comment #7)<br/> > (In reply to Shu Dong from comment #6)<br/> > > I noticed that you voted +1 on my latest fix. Thanks very much for your code<br/> > > review! So does it mean next we should merge the patch to master by<br/> > > ourselves to see if it works?<br/> > <br/> > Yes please do, I'm not a Java expert so I can't say if the fix is good but<br/> > once you merge it we can kick off an autorelease build to see if it passes.</p> <p>I noticed that the ALTO was not enabled in the latest Jenkins autorelease build <a href="https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/94/" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/94/</a>, do you know how to enable it so we could see if the fix works out? Thanks a lot!</p> <p>(In reply to Shu Dong from comment #9)<br/> > I noticed that the ALTO was not enabled in the latest Jenkins autorelease<br/> > build<br/> > <a href="https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/94/" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/job/autorelease-daily-master/94/</a>, do<br/> > you know how to enable it so we could see if the fix works out? Thanks a lot!</p> <p>Sorry I didn't re-enable alto for the build before taking off for the weekend. I kicked off build #96 which should include alto this time.</p> <p>simple-impl successfully built:</p> <p><a href="https://jenkins.opendaylight.org/releng/view/autorelease/job/autorelease-daily-master/96/org.opendaylight.alto$simple-impl/console" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/view/autorelease/job/autorelease-daily-master/96/org.opendaylight.alto$simple-impl/console</a></p> <p>We're now failing on alto-northbuild. I'll open a separate bug for that.</p> Development External issue ID 3955 External issue URL Rank 0|i02swv: