Linaro Android Build Tools

Merge lp:~gesha/linaro-android-build-tools/fix-return-code into lp:linaro-android-build-tools

  1. fix-return-code
  2. Merge into trunk
Proposed by Georgy Redkozubov
Status: Merged
Approved by: Paul Sokolovsky
Approved revision: 610
Merged at revision: 610
Proposed branch: lp:~gesha/linaro-android-build-tools/fix-return-code
Merge into: lp:linaro-android-build-tools
Diff against target: 15 lines (+2/-0)
1 file modified
utils/new-publish/propagate.py (+2/-0)
To merge this branch: bzr merge lp:~gesha/linaro-android-build-tools/fix-return-code
Reviewer Review Type Date Requested Status
linaro-landing-team-hisilicon-bot (community) Approve
Linaro Automation & Validation Pending
Review via email: mp+162764@code.launchpad.net

Description of the change

This branch adds return code handling after ssh command execution in new publishing method.
Script will now exit with the same return code as command executed by ssh returned.

To post a comment you must log in.
Revision history for this message
linaro-landing-team-hisilicon-bot (linaro-landing-team-hisilicon-bot) wrote :

Thanks for looking into this!

review: Approve
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Previous comment was by me.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'utils/new-publish/propagate.py'
2--- utils/new-publish/propagate.py 2013-04-26 16:54:12 +0000
3+++ utils/new-publish/propagate.py 2013-05-07 12:31:28 +0000
4@@ -65,9 +65,11 @@
5 client.connect(remote_host, username=TRIGGER_USER_NAME, key_filename=TRIGGER_KEY_FILE)
6 stdin, stdout, stderr = client.exec_command("reshuffle-files -t android -j %s -n %s -m %s" % (job, build, opt_staging))
7 stdin.close()
8+ rc = stdout.channel.recv_exit_status()
9 print "=== stdout ==="
10 print stdout.read()
11 print "=== stderr ==="
12 print stderr.read()
13 print "=============="
14 client.close()
15+ sys.exit(rc)

Subscribers

People subscribed via source and target branches