diff options
| author | James E. Blair <jeblair@redhat.com> | 2020-06-26 19:13:30 -0700 |
|---|---|---|
| committer | James E. Blair <jeblair@redhat.com> | 2020-08-04 13:51:56 -0700 |
| commit | 3472114a679e9c9664d709fde0707cb5cea5f982 (patch) | |
| tree | f6b64f672ea76040fc810d7eb832e55a3e7d6422 /zuul/driver/gerrit/gerritconnection.py | |
| parent | eec7b303fb73335647282ce964fb38d7f8a0da83 (diff) | |
| download | zuul-3472114a679e9c9664d709fde0707cb5cea5f982.tar.gz | |
Always report the status url in Gerrit checks
When using the Gerrit checks API, we only included a URL when the
build completed. However, we can also supply the URL while the
build is in progress. This will (at least until we start recording
in-progress buildsets in the database) be the change-specific status
page.
Change-Id: I3202764683b57c19df25169c2b47bc9561df0e93
Diffstat (limited to 'zuul/driver/gerrit/gerritconnection.py')
| -rw-r--r-- | zuul/driver/gerrit/gerritconnection.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/zuul/driver/gerrit/gerritconnection.py b/zuul/driver/gerrit/gerritconnection.py index 186a52d70..2abdf74eb 100644 --- a/zuul/driver/gerrit/gerritconnection.py +++ b/zuul/driver/gerrit/gerritconnection.py @@ -1071,9 +1071,9 @@ class GerritConnection(BaseConnection): checkinfo['started'] = fmt(item.enqueue_time) if item.report_time: checkinfo['finished'] = fmt(item.report_time) - url = item.formatStatusUrl() - if url: - checkinfo['url'] = url + url = item.formatStatusUrl() + if url: + checkinfo['url'] = url if checkinfo: for x in range(1, 4): try: |