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

Side by Side Diff: tools/push-to-trunk/chromium_roll.py

Issue 315133003: Add clusterfuzz check to v8 auto-roll script and use CQ. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Removed client-side filters as query parameters are indexed now. Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/push-to-trunk/auto_roll.py ('k') | tools/push-to-trunk/common_includes.py » ('j') | 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/env python 1 #!/usr/bin/env python
2 # Copyright 2014 the V8 project authors. All rights reserved. 2 # Copyright 2014 the V8 project authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import argparse 6 import argparse
7 import os 7 import os
8 import sys 8 import sys
9 9
10 from common_includes import * 10 from common_includes import *
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 self.DieNoManualMode("A reviewer must be specified in forced mode.") 98 self.DieNoManualMode("A reviewer must be specified in forced mode.")
99 rev = self.ReadLine() 99 rev = self.ReadLine()
100 100
101 commit_title = "Update V8 to %s." % self["push_title"].lower() 101 commit_title = "Update V8 to %s." % self["push_title"].lower()
102 sheriff = "" 102 sheriff = ""
103 if self["sheriff"]: 103 if self["sheriff"]:
104 sheriff = ("\n\nPlease reply to the V8 sheriff %s in case of problems." 104 sheriff = ("\n\nPlease reply to the V8 sheriff %s in case of problems."
105 % self["sheriff"]) 105 % self["sheriff"])
106 self.GitCommit("%s%s\n\nTBR=%s" % (commit_title, sheriff, rev)) 106 self.GitCommit("%s%s\n\nTBR=%s" % (commit_title, sheriff, rev))
107 self.GitUpload(author=self._options.author, 107 self.GitUpload(author=self._options.author,
108 force=self._options.force_upload) 108 force=self._options.force_upload,
109 cq=self._options.use_commit_queue)
109 print "CL uploaded." 110 print "CL uploaded."
110 111
111 112
112 class SwitchV8(Step): 113 class SwitchV8(Step):
113 MESSAGE = "Returning to V8 checkout." 114 MESSAGE = "Returning to V8 checkout."
114 REQUIRES = "chrome_path" 115 REQUIRES = "chrome_path"
115 116
116 def RunStep(self): 117 def RunStep(self):
117 os.chdir(self["v8_path"]) 118 os.chdir(self["v8_path"])
118 119
(...skipping 17 matching lines...) Expand all
136 help="Don't prompt the user.", 137 help="Don't prompt the user.",
137 default=False, action="store_true") 138 default=False, action="store_true")
138 group.add_argument("-m", "--manual", 139 group.add_argument("-m", "--manual",
139 help="Prompt the user at every important step.", 140 help="Prompt the user at every important step.",
140 default=False, action="store_true") 141 default=False, action="store_true")
141 parser.add_argument("-c", "--chromium", 142 parser.add_argument("-c", "--chromium",
142 help=("The path to your Chromium src/ " 143 help=("The path to your Chromium src/ "
143 "directory to automate the V8 roll.")) 144 "directory to automate the V8 roll."))
144 parser.add_argument("-l", "--last-push", 145 parser.add_argument("-l", "--last-push",
145 help="The git commit ID of the last push to trunk.") 146 help="The git commit ID of the last push to trunk.")
147 parser.add_argument("--use-commit-queue",
148 help="Check the CQ bit on upload.",
149 default=False, action="store_true")
146 150
147 def _ProcessOptions(self, options): # pragma: no cover 151 def _ProcessOptions(self, options): # pragma: no cover
148 if not options.manual and not options.reviewer: 152 if not options.manual and not options.reviewer:
149 print "A reviewer (-r) is required in (semi-)automatic mode." 153 print "A reviewer (-r) is required in (semi-)automatic mode."
150 return False 154 return False
151 if not options.manual and not options.chromium: 155 if not options.manual and not options.chromium:
152 print "A chromium checkout (-c) is required in (semi-)automatic mode." 156 print "A chromium checkout (-c) is required in (semi-)automatic mode."
153 return False 157 return False
154 if not options.manual and not options.author: 158 if not options.manual and not options.author:
155 print "Specify your chromium.org email with -a in (semi-)automatic mode." 159 print "Specify your chromium.org email with -a in (semi-)automatic mode."
(...skipping 11 matching lines...) Expand all
167 SwitchChromium, 171 SwitchChromium,
168 UpdateChromiumCheckout, 172 UpdateChromiumCheckout,
169 UploadCL, 173 UploadCL,
170 SwitchV8, 174 SwitchV8,
171 CleanUp, 175 CleanUp,
172 ] 176 ]
173 177
174 178
175 if __name__ == "__main__": # pragma: no cover 179 if __name__ == "__main__": # pragma: no cover
176 sys.exit(ChromiumRoll(CONFIG).Run()) 180 sys.exit(ChromiumRoll(CONFIG).Run())
OLDNEW
« no previous file with comments | « tools/push-to-trunk/auto_roll.py ('k') | tools/push-to-trunk/common_includes.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698