[AAA-145] aaa-cli-jar missing in stable/nitrogen and master Created: 02/Sep/17 Updated: 21/Mar/19 Resolved: 06/Sep/17 |
|
| Status: | Resolved |
| Project: | aaa |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Daniel Farrell | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 9087 |
| Description |
|
In |
| Comments |
| Comment by Daniel Farrell [ 02/Sep/17 ] |
|
It seems like the jar is in the distro, but not in bin/. With ODL 6.2.0-0.1.20170901snap497 (latest stable/carbon) ``` With ODL 7.0.0-0.1.20170830rel1949 (Nitrogen RC2) ``` With ODL 8.0.0-0.1.20170901snap214 (latest master) ``` Can we get it in bin/? |
| Comment by Michael Vorburger [ 04/Sep/17 ] |
|
> It seems like the jar is in the distro, but not in bin/. That's actually the wrong one - just FYI there's some doc I had put on https://wiki.opendaylight.org/view/AAA:Changing_Account_Passwords#aaa-cli-jar > Can we get it in bin/? Yeah, I'll dig into this. Am surprised TBH that we "lost" this - but will port. |
| Comment by Michael Vorburger [ 05/Sep/17 ] |
|
+Tim Rozet (who originally internally raised this, as it breaks OPNFV!) +Vratko, because... > Yeah, I'll dig into this. Am surprised TBH that we "lost" this ... I just did a bit of Git archaeology, and found that Vratko commented out what gets aaa-cli-jar into bin, as part of his massive integration/distribution move to Karaf 4 in f7a8d73e603ab5f0046e5e87956cc78e772a28d7, with comment: "This leads to "Unsupported 'Bundle-ManifestVersion' value: 1". Uncomment when fixed.", so I guess I'll look more into and fix that as part of this issue... PS, note to self: Check netvirt/distribution/karaf as well! |
| Comment by Michael Vorburger [ 05/Sep/17 ] |
|
+ Cc: Stephen Kitt just FYI re. this: > PS, note to self: Check netvirt/distribution/karaf as well! actually I'm finding that this (aaa-cli) never was in netvirt's distribution... Stephen tried this in https://git.opendaylight.org/gerrit/#/c/60288/, but had to revert it again in https://git.opendaylight.org/gerrit/#/c/60467/ because it "breaks the build of the Karaf distribution (on Jenkin)". Therefore I won't deal with that aspect in this issue here, but focus on getting it back into integration's distribution. > In just FTR for future archaeology, the specific change which introduced what's this bug reports as being missing here and must be restored was https://git.opendaylight.org/gerrit/#/c/49278/ (not really > Vratko commented out what gets aaa-cli-jar into bin, see https://git.opendaylight.org/gerrit/#/c/51762/ (commit message mentions this) |
| Comment by Michael Vorburger [ 05/Sep/17 ] |
|
master: https://git.opendaylight.org/gerrit/62723 |