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
=== modified file 'uci-nova'
--- uci-nova 2015-04-02 02:46:04 +0000
+++ uci-nova 2015-04-05 03:58:17 +0000
@@ -55,6 +55,7 @@
5555
56SRVNAME=""56SRVNAME=""
57NET_ID=""57NET_ID=""
58CONSOLE=""
58DEBUG=""59DEBUG=""
5960
6061
@@ -244,13 +245,18 @@
244 # purge the device host key so that SSH doesn't print a scary warning245 # purge the device host key so that SSH doesn't print a scary warning
245 ssh-keygen -f ~/.ssh/known_hosts -R $ipaddr >/dev/null 2>&1 || true246 ssh-keygen -f ~/.ssh/known_hosts -R $ipaddr >/dev/null 2>&1 || true
246247
248 ADT_EXTRA_OPTS="-n $SRVNAME"
249 if [ -n "$CONSOLE" ]; then
250 ADT_EXTRA_OPTS="$ADT_EXTRA_OPTS -c $CONSOLE"
251 fi
252
247 # Return access information to adt-virt-ssh.253 # Return access information to adt-virt-ssh.
248 cat<<EOF254 cat<<EOF
249identity=$SSH_IDENTITY255identity=$SSH_IDENTITY
250login=$SSH_USER256login=$SSH_USER
251hostname=$ipaddr257hostname=$ipaddr
252capabilities=$CAPABILITIES258capabilities=$CAPABILITIES
253extraopts=-n $SRVNAME259extraopts=$ADT_EXTRA_OPTS
254EOF260EOF
255 if [ -n "$SUDO_PASSWORD" ]; then261 if [ -n "$SUDO_PASSWORD" ]; then
256 echo "password=$SUDO_PASSWORD"262 echo "password=$SUDO_PASSWORD"

Subscribers

People subscribed via source and target branches