Add support for override-checkout, deprecate override-branch
We want to support jobs specifying that they check out a tag
rather than merely a branch. This accidentally worked in Zuul v2
with zuul-cloner, and some jobs have come to rely on the behavior.
There's no reason not to support it, so let's do so.
However, for clarity, change the name of the option to
"override-checkout". This is intended to be intuitive (anything
you can tell git to 'checkout' you can put here), while avoiding
the suggestion that only branches or tags may be checked out.
The old form, "override-branch" is deprecated and will be removed.
Change-Id: Icc2907e72596626e96d2dc9f6ab1c3026f4085ab
diff --git a/tests/unit/test_executor.py b/tests/unit/test_executor.py
index f051ec4..5d27663 100755
--- a/tests/unit/test_executor.py
+++ b/tests/unit/test_executor.py
@@ -378,6 +378,32 @@
self.assertBuildStates(states, projects)
+ @simple_layout('layouts/repo-checkout-tag.yaml')
+ def test_tag_checkout(self):
+ self.executor_server.hold_jobs_in_build = True
+ p1 = "review.example.com/org/project1"
+ p2 = "review.example.com/org/project2"
+ projects = [p1, p2]
+ upstream = self.getUpstreamRepos(projects)
+
+ self.create_branch('org/project2', 'stable/havana')
+ files = {'README': 'tagged readme'}
+ self.addCommitToRepo('org/project2', 'tagged commit',
+ files, branch='stable/havana', tag='test-tag')
+
+ A = self.fake_gerrit.addFakeChange('org/project1', 'master', 'A')
+ self.fake_gerrit.addEvent(A.getPatchsetCreatedEvent(1))
+ self.waitUntilSettled()
+
+ states = [
+ {p1: dict(present=[A], branch='master'),
+ p2: dict(commit=str(upstream[p2].commit('test-tag')),
+ absent=[A]),
+ },
+ ]
+
+ self.assertBuildStates(states, projects)
+
class TestAnsibleJob(ZuulTestCase):
tenant_config_file = 'config/ansible/main.yaml'