[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
Platform: All


External issue ID: 9087

 Description   

In AAA-138 and Gerrit 60049 Michael added a tool for changing ODL NB username/password without starting up the controller. This is required for deployments using configuration management tools like puppet-opendaylight and ansible-opendaylight. Major downstream consumers like OPNFV use this extensively in production for stable/carbon, but it's missing and causing failures for stable/nitrogen and master. Need to add the tool to those branches.



 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)

```
$ find . -name "aaa-cli"
./system/org/opendaylight/aaa/aaa-cli
./system/org/opendaylight/aaa/aaa-cli/0.5.2-SNAPSHOT/aaa-cli-0.5.2-SNAPSHOT.jar
./bin/aaa-cli-jar.jar
```

With ODL 7.0.0-0.1.20170830rel1949 (Nitrogen RC2)

```
$ find . -name "aaa-cli"
./system/org/opendaylight/aaa/aaa-cli
./system/org/opendaylight/aaa/aaa-cli/0.6.0/aaa-cli-0.6.0.jar
./system/org/opendaylight/aaa/odl-aaa-cli
./system/org/opendaylight/aaa/odl-aaa-cli/0.6.0/odl-aaa-cli-0.6.0-features.xml
```

With ODL 8.0.0-0.1.20170901snap214 (latest master)

```
find /opt/opendaylight -name 'aaa-cli'
/opt/opendaylight/system/org/opendaylight/aaa/aaa-cli
/opt/opendaylight/system/org/opendaylight/aaa/aaa-cli/0.7.0-SNAPSHOT/aaa-cli-0.7.0-SNAPSHOT.jar
/opt/opendaylight/system/org/opendaylight/aaa/odl-aaa-cli
/opt/opendaylight/system/org/opendaylight/aaa/odl-aaa-cli/0.7.0-SNAPSHOT/odl-aaa-cli-0.7.0-SNAPSHOT-features.xml
```

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 AAA-138 and Gerrit 60049 Michael added

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 AAA-138 and Gerrit 60049)

> Vratko commented out what gets aaa-cli-jar into bin,
> as part of his massive integration/distribution move to Karaf 4 in
> f7a8d73e603ab5f0046e5e87956cc78e772a28d7

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
nitrogen: https://git.opendaylight.org/gerrit/#/c/62724/

Generated at Wed Feb 07 19:08:45 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.