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

Unified Diff: slave/skia_slave_scripts/update_all_buildslave_checkouts.py

Issue 648353002: Remove Skia's forked buildbot code (Closed) Base URL: https://skia.googlesource.com/buildbot.git@master
Patch Set: Address comment Created 6 years, 2 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « slave/skia_slave_scripts/update.py ('k') | slave/skia_slave_scripts/update_all_slave_hosts.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: slave/skia_slave_scripts/update_all_buildslave_checkouts.py
diff --git a/slave/skia_slave_scripts/update_all_buildslave_checkouts.py b/slave/skia_slave_scripts/update_all_buildslave_checkouts.py
deleted file mode 100644
index 0cfecf85eb143ce4f694d018ec5ceb209675be9f..0000000000000000000000000000000000000000
--- a/slave/skia_slave_scripts/update_all_buildslave_checkouts.py
+++ /dev/null
@@ -1,83 +0,0 @@
-#!/usr/bin/env python
-# Copyright (c) 2014 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-
-"""Update the buildbot checkouts for each buildslave on every host.
-
-This differs from UpdateScripts in that it updates ALL of the buildbot script
-checkouts for ALL buildslaves, as opposed to a single buildslave's checkout of
-the buildbot scripts on a single host machine.
-"""
-
-
-import re
-import skia_vars
-import sys
-
-from build_step import BuildStep, BuildStepWarning
-from scripts import run_cmd
-from utils import force_update_checkout
-
-
-BUILDBOT_GIT_URL = skia_vars.GetGlobalVariable('buildbot_git_url')
-
-
-class UpdateAllBuildslaves(BuildStep):
- def _Run(self):
- script_path = run_cmd.ResolvablePath('slave', 'skia_slave_scripts', 'utils',
- 'force_update_checkout.py')
- sync_cmd = ['python', script_path]
- results = run_cmd.run_on_all_slaves_on_all_hosts(sync_cmd)
- failed = []
- for host in results.iterkeys():
- print host
- # If results[host] is a MultiCommandResults instance, then we have results
- # for buildslaves running on that machine, which implies that we were able
- # to log in to the machine successfully.
- if isinstance(results[host], run_cmd.MultiCommandResults):
- # We successfully logged into the buildslave host machine.
- for buildslave in results[host].iterkeys():
- print ' ', buildslave,
- # Check and report the results of the command for each buildslave on
- # this host machine.
- if results[host][buildslave].returncode != 0:
- # If the command failed, print its output.
- failed.append(buildslave)
- print
- results[host][buildslave].print_results(pretty=True)
- else:
- # If the command succeeded, find and print the commit hash we synced
- # to. If we can't find it, then something must have failed, so
- # print the output and report a failure.
- match = re.search(
- force_update_checkout.GOT_REVISION_PATTERN % ('(\w+)'),
- results[host][buildslave].stdout)
- if match:
- print '\t%s' % match.group(1)
- else:
- failed.append(host)
- print
- results[host][buildslave].print_results(pretty=True)
- else:
- # We were unable to log into the buildslave host machine.
- if results[host].returncode != 0:
- failed.append(host)
- results[host].print_results(pretty=True)
- print
-
- if failed:
- print
- print 'Failed to update the following buildslaves:'
- for failed_host in failed:
- print ' ', failed_host
-
- if failed:
- # TODO(borenet): Make sure that we can log in to all hosts, then make this
- # an error.
- raise BuildStepWarning('Could not update some buildslaves.')
-
-
-if '__main__' == __name__:
- sys.exit(BuildStep.RunBuildStep(UpdateAllBuildslaves))
« no previous file with comments | « slave/skia_slave_scripts/update.py ('k') | slave/skia_slave_scripts/update_all_slave_hosts.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698