commit 18bc7ed30d9420077fc98fea6e683fa1b7f86cce
author Jamie Lennox <jamielennox@gmail.com> Wed May 10 10:37:55 2017 +1000
committer Jesse Keating <omgjlk@us.ibm.com> Tue May 30 14:53:46 2017 -0700
tree eb8463e62a189fa09b97627ac5dd019fe16fa478
parent 4b462972d8a78f3683aaefdab04fc6999a60b7a4

Include tenant name in github context

Pipelines are namespaced via their tenant in zuulv3 so I think the
context that zuul reports to github should include a tenant name.

Change-Id: Ib4856ee41330bb9eae3b9c622998b8cdd9241911
Signed-off-by: Jamie Lennox <jamielennox@gmail.com>

tests/unit/test_github_driver.py

diff --git a/tests/unit/test_github_driver.py b/tests/unit/test_github_driver.py
index c8e7341..227d659 100644
--- a/tests/unit/test_github_driver.py
+++ b/tests/unit/test_github_driver.py
@@ -260,7 +260,7 @@
         check_status = A.statuses[A.head_sha][0]
         check_url = ('http://zuul.example.com/status/#%s,%s' %
                      (A.number, A.head_sha))
-        self.assertEqual('check', check_status['context'])
+        self.assertEqual('tenant-one/check', check_status['context'])
         self.assertEqual('Standard check', check_status['description'])
         self.assertEqual('pending', check_status['state'])
         self.assertEqual(check_url, check_status['url'])
@@ -274,7 +274,7 @@
         check_status = A.statuses[A.head_sha][0]
         check_url = ('http://zuul.example.com/status/#%s,%s' %
                      (A.number, A.head_sha))
-        self.assertEqual('check', check_status['context'])
+        self.assertEqual('tenant-one/check', check_status['context'])
         self.assertEqual('success', check_status['state'])
         self.assertEqual(check_url, check_status['url'])
         self.assertEqual(1, len(A.comments))
@@ -297,7 +297,7 @@
         # pipeline reports success status
         self.assertEqual(3, len(A.statuses[A.head_sha]))
         report_status = A.statuses[A.head_sha][0]
-        self.assertEqual('reporting', report_status['context'])
+        self.assertEqual('tenant-one/reporting', report_status['context'])
         self.assertEqual('success', report_status['state'])
         self.assertEqual(2, len(A.comments))
 

zuul/driver/github/githubreporter.py

diff --git a/zuul/driver/github/githubreporter.py b/zuul/driver/github/githubreporter.py
index 6808bdd..7efe180 100644
--- a/zuul/driver/github/githubreporter.py
+++ b/zuul/driver/github/githubreporter.py
@@ -67,7 +67,7 @@
     def setPullStatus(self, pipeline, item):
         project = item.change.project.name
         sha = item.change.patchset
-        context = pipeline.name
+        context = '%s/%s' % (pipeline.layout.tenant.name, pipeline.name)
         state = self._commit_status
         url = ''
         if self.connection.sched.config.has_option('zuul', 'status_url'):