Merge lp:~cprov/adt-cloud-worker/uci-nova-console-log into lp:~canonical-ci-engineering/adt-cloud-worker/uci-nova

Proposed by Celso Providelo
Status: Merged
Approved by: Celso Providelo
Approved revision: 6
Merged at revision: 6
Proposed branch: lp:~cprov/adt-cloud-worker/uci-nova-console-log
Merge into: lp:~canonical-ci-engineering/adt-cloud-worker/uci-nova
Diff against target: 31 lines (+7/-1)
1 file modified
uci-nova (+7/-1)
To merge this branch: bzr merge lp:~cprov/adt-cloud-worker/uci-nova-console-log
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+255236@code.launchpad.net

Commit message

Return console-logging cmd-line option back to adt on open(), so it can be properly used when calling cleanup() later on.

Description of the change

Return console-logging cmd-line option back to adt on open(), so it can be properly used when calling cleanup() later on.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

Thanks for fixing this.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'uci-nova'
2--- uci-nova 2015-04-02 02:46:04 +0000
3+++ uci-nova 2015-04-05 03:58:17 +0000
4@@ -55,6 +55,7 @@
5
6 SRVNAME=""
7 NET_ID=""
8+CONSOLE=""
9 DEBUG=""
10
11
12@@ -244,13 +245,18 @@
13 # purge the device host key so that SSH doesn't print a scary warning
14 ssh-keygen -f ~/.ssh/known_hosts -R $ipaddr >/dev/null 2>&1 || true
15
16+ ADT_EXTRA_OPTS="-n $SRVNAME"
17+ if [ -n "$CONSOLE" ]; then
18+ ADT_EXTRA_OPTS="$ADT_EXTRA_OPTS -c $CONSOLE"
19+ fi
20+
21 # Return access information to adt-virt-ssh.
22 cat<<EOF
23 identity=$SSH_IDENTITY
24 login=$SSH_USER
25 hostname=$ipaddr
26 capabilities=$CAPABILITIES
27-extraopts=-n $SRVNAME
28+extraopts=$ADT_EXTRA_OPTS
29 EOF
30 if [ -n "$SUDO_PASSWORD" ]; then
31 echo "password=$SUDO_PASSWORD"

Subscribers

People subscribed via source and target branches