Commit 6a06348b authored by Michael Achenbach's avatar Michael Achenbach

Fix landing candidate pushes.

BUG=chromium:410721
LOG=n
TBR=tandrii@chromium.org
TEST=script_test.py

Review URL: https://codereview.chromium.org/725503003

Cr-Commit-Position: refs/heads/master@{#25318}
parent 2d075e22
...@@ -287,9 +287,6 @@ class VCInterface(object): ...@@ -287,9 +287,6 @@ class VCInterface(object):
def RemoteBranch(self, name): def RemoteBranch(self, name):
raise NotImplementedError() raise NotImplementedError()
def Land(self):
raise NotImplementedError()
def CLLand(self): def CLLand(self):
raise NotImplementedError() raise NotImplementedError()
...@@ -356,9 +353,6 @@ class GitInterface(VCInterface): ...@@ -356,9 +353,6 @@ class GitInterface(VCInterface):
self.step.Git("tag %s %s" % (tag, commit)) self.step.Git("tag %s %s" % (tag, commit))
self.step.Git("push origin %s" % tag) self.step.Git("push origin %s" % tag)
def Land(self):
self.step.Git("push origin")
def CLLand(self): def CLLand(self):
self.step.GitCLLand() self.step.GitCLLand()
......
...@@ -231,10 +231,6 @@ class GitRecipesMixin(object): ...@@ -231,10 +231,6 @@ class GitRecipesMixin(object):
def GitPresubmit(self, **kwargs): def GitPresubmit(self, **kwargs):
self.Git("cl presubmit", "PRESUBMIT_TREE_CHECK=\"skip\"", **kwargs) self.Git("cl presubmit", "PRESUBMIT_TREE_CHECK=\"skip\"", **kwargs)
def GitDCommit(self, **kwargs):
self.Git(
"cl dcommit -f --bypass-hooks", retry_on=lambda x: x is None, **kwargs)
def GitCLLand(self, **kwargs): def GitCLLand(self, **kwargs):
self.Git( self.Git(
"cl land -f --bypass-hooks", retry_on=lambda x: x is None, **kwargs) "cl land -f --bypass-hooks", retry_on=lambda x: x is None, **kwargs)
......
...@@ -51,11 +51,6 @@ class Preparation(Step): ...@@ -51,11 +51,6 @@ class Preparation(Step):
self.PrepareBranch() self.PrepareBranch()
self.DeleteBranch(self.Config("TRUNKBRANCH")) self.DeleteBranch(self.Config("TRUNKBRANCH"))
# Allow directly pushing to candidates.
if not self.Git("config --get remote.origin.push").strip():
self.Git("config --add remote.origin.push "
"refs/remotes/origin/candidates:refs/pending/heads/candidates")
class FreshBranch(Step): class FreshBranch(Step):
MESSAGE = "Create a fresh branch." MESSAGE = "Create a fresh branch."
...@@ -355,7 +350,7 @@ class Land(Step): ...@@ -355,7 +350,7 @@ class Land(Step):
MESSAGE = "Land the patch." MESSAGE = "Land the patch."
def RunStep(self): def RunStep(self):
self.vc.Land() self.vc.CLLand()
class TagRevision(Step): class TagRevision(Step):
......
...@@ -772,9 +772,6 @@ Performance and stability improvements on all platforms.""", commit) ...@@ -772,9 +772,6 @@ Performance and stability improvements on all platforms.""", commit)
Cmd("git fetch", ""), Cmd("git fetch", ""),
Cmd("git branch", " branch1\n* branch2\n"), Cmd("git branch", " branch1\n* branch2\n"),
Cmd("git branch", " branch1\n* branch2\n"), Cmd("git branch", " branch1\n* branch2\n"),
Cmd("git config --get remote.origin.push", ""),
Cmd("git config --add remote.origin.push "
"refs/remotes/origin/candidates:refs/pending/heads/candidates", ""),
Cmd(("git new-branch %s --upstream origin/master" % Cmd(("git new-branch %s --upstream origin/master" %
TEST_CONFIG["BRANCHNAME"]), TEST_CONFIG["BRANCHNAME"]),
""), ""),
...@@ -823,7 +820,7 @@ Performance and stability improvements on all platforms.""", commit) ...@@ -823,7 +820,7 @@ Performance and stability improvements on all platforms.""", commit)
if manual: if manual:
expectations.append(RL("Y")) # Sanity check. expectations.append(RL("Y")) # Sanity check.
expectations += [ expectations += [
Cmd("git push origin", ""), Cmd("git cl land -f --bypass-hooks", ""),
Cmd("git fetch", ""), Cmd("git fetch", ""),
Cmd("git log -1 --format=%H --grep=" Cmd("git log -1 --format=%H --grep="
"\"Version 3.22.5 (based on push_hash)\"" "\"Version 3.22.5 (based on push_hash)\""
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment