[SNMP4SDN-7] Lithium snmp4sdn depends on Helium version of dependencies Created: 03/Jun/15  Updated: 19/Oct/17  Resolved: 08/Jun/15

Status: Verified
Project: snmp4sdn
Component/s: General
Affects Version/s: unspecified
Fix Version/s: None

Type: Bug
Reporter: Thanh Ha (zxiiro) Assignee: Unassigned
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 3572

 Description   

Autorelease is unable to build snmp4sdn because it incorrectly depends on the Helium version of a few dependencies:

  • commons.opendaylight 1.4.4 should be updated to 1.5.0
  • sal 0.8.1 should be updated to 0.9.0
  • mdsal 1.1 should be updated to 1.2.0
  • nsf 0.4.2 should be updated to 0.5.0
  • yangtools 0.6.2 should be updated to 0.7.0


 Comments   
Comment by Thanh Ha (zxiiro) [ 03/Jun/15 ]

We need to bump all ODL dependency versions in snmp4sdn back up to the correct Lithium release versions. In my opinion this is a regression from before the code dump.

Comment by Thanh Ha (zxiiro) [ 03/Jun/15 ]

My initial attempt at getting this going https://git.opendaylight.org/gerrit/21824

Hopefully we can work off of this to get this fixed to the end.

Comment by Thanh Ha (zxiiro) [ 03/Jun/15 ]

Looks like my initial attempt fails with "method does not override or implement a method from a supertype". I'm not sure how to resolve this.

Christine, can someone on SNMP4SDN project take a look and update the patch as necessary?

Comment by Christine Hsieh [ 04/Jun/15 ]

Thanh,

With this patch, jenkins build succesfully now:
the patch set 2 in https://git.opendaylight.org/gerrit/#/c/21809/

In this patch, besides the dependencies you suggest to update, guava's version should not be placed in snmp4sdn's local pom so I removed it, then success.

Comment by Thanh Ha (zxiiro) [ 04/Jun/15 ]

(In reply to Yi-Ling Hsieh from comment #4)
> Thanh,
>
> With this patch, jenkins build succesfully now:
> the patch set 2 in https://git.opendaylight.org/gerrit/#/c/21809/
>
> In this patch, besides the dependencies you suggest to update, guava's
> version should not be placed in snmp4sdn's local pom so I removed it, then
> success.

Thanks Christine,

FYI in the future you do not need to create a new Gerrit. You could actually download the existing patch I created and "git commit --amend" it to modify and update a new patchset. ODL's Gerrit allows committers to modify and update any author's patches. Doing this would keep record of the history of the Gerrit patchsets all in one place.

I'll updated autorelease and see if we can get snmp4sdn building in the next release candidate.

Comment by Christine Hsieh [ 08/Jun/15 ]

Thanks, Thanh.

Generated at Wed Feb 07 20:39:17 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.