Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Side by Side Diff: bin/cbuildbot.py

Issue 5595004: Have upload_prebuilts respect the debug flag. (Closed) Base URL: http://git.chromium.org/git/crosutils.git@master
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. 3 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 """CBuildbot is wrapper around the build process used by the pre-flight queue""" 7 """CBuildbot is wrapper around the build process used by the pre-flight queue"""
8 8
9 import errno 9 import errno
10 import heapq 10 import heapq
(...skipping 683 matching lines...) Expand 10 before | Expand all | Expand 10 after
694 test_results_dir=test_results_dir, 694 test_results_dir=test_results_dir,
695 gsutil=options.gsutil, 695 gsutil=options.gsutil,
696 archive_dir=archive_full_path, 696 archive_dir=archive_full_path,
697 acl=options.acl) 697 acl=options.acl)
698 698
699 if buildconfig['uprev']: 699 if buildconfig['uprev']:
700 # Don't push changes for developers. 700 # Don't push changes for developers.
701 if buildconfig['master']: 701 if buildconfig['master']:
702 # Master bot needs to check if the other slaves completed. 702 # Master bot needs to check if the other slaves completed.
703 if cbuildbot_comm.HaveSlavesCompleted(config): 703 if cbuildbot_comm.HaveSlavesCompleted(config):
704 _UploadPrebuilts(buildroot, board, buildconfig['rev_overlays'], 704 if not options.debug:
705 [new_binhost]) 705 _UploadPrebuilts(buildroot, board, buildconfig['rev_overlays'],
706 [new_binhost])
706 _UprevPush(buildroot, tracking_branch, buildconfig['board'], 707 _UprevPush(buildroot, tracking_branch, buildconfig['board'],
707 push_overlays, options.debug) 708 push_overlays, options.debug)
708 else: 709 else:
709 Die('CBUILDBOT - One of the slaves has failed!!!') 710 Die('CBUILDBOT - One of the slaves has failed!!!')
710 711
711 else: 712 else:
712 # Publish my status to the master if its expecting it. 713 # Publish my status to the master if its expecting it.
713 if buildconfig['important'] and not options.debug: 714 if buildconfig['important'] and not options.debug:
714 cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_COMPLETE) 715 cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_COMPLETE)
715 716
716 except: 717 except:
717 # Send failure to master bot. 718 # Send failure to master bot.
718 if not buildconfig['master'] and buildconfig['important']: 719 if not buildconfig['master'] and buildconfig['important']:
719 cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_FAILED) 720 cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_FAILED)
720 721
721 raise 722 raise
722 723
723 724
724 if __name__ == '__main__': 725 if __name__ == '__main__':
725 main() 726 main()
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698