Last modified: 2014-08-19 14:44:04 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T65744, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 63744 - Zuul: Implement support for customizing status_url to include the change.id
Zuul: Implement support for customizing status_url to include the change.id
Status: NEW
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
wmf-deployment
All All
: Low enhancement (vote)
: ---
Assigned To: Antoine "hashar" Musso (WMF)
:
Depends on:
Blocks: 63399
  Show dependency treegraph
 
Reported: 2014-04-09 20:02 UTC by Antoine "hashar" Musso (WMF)
Modified: 2014-08-19 14:44 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Antoine "hashar" Musso (WMF) 2014-04-09 20:02:42 UTC
Timo worked on a patch for the Zuul status page which let one pass the ChangeId,Patchset to the URL which would highlight it on the status page.

The URL looks like:

https://integration.wikimedia.org/zuul/#change-124992,2


When Zuul start proceeding a change in a pipeline it will report:

 Starting gate-and-submit jobs.
 https://integration.wikimedia.org/zuul/


That URL is crafted by the scheduler:

  if self.sched.config.has_option('zuul', 'status_url'):
      msg += "\n" + self.sched.config.get('zuul', 'status_url')

Where status_url is set to https://integration.wikimedia.org/zuul/


What we would need is to use format() much like for url_pattern and pass the change and patchset numbers.


We can do it as a hack on our Zuul fork and propose it upstream.
Comment 1 Antoine "hashar" Musso (WMF) 2014-04-11 13:45:46 UTC
Sent to upstream as  https://review.openstack.org/86900 . Most of them are attending the US Python Conference so there will be most probably a bunch of delay before it is being reviewed.

Whenever they are happy with the change I will cherry-pick to our fork.
Comment 2 Antoine "hashar" Musso (WMF) 2014-05-06 15:08:15 UTC
Patch still pending upstream https://review.openstack.org/#/c/86900/
Comment 3 Antoine "hashar" Musso (WMF) 2014-05-19 11:17:14 UTC
Patch review requested a test to be added to confirm status_url is properly formatted. I amended my patch.
Comment 4 Antoine "hashar" Musso (WMF) 2014-08-19 14:44:04 UTC
The patch I proposed upstream breaks their test suite.  I dont have bandwidth right now  to complete the patch :(

Patch is https://review.openstack.org/#/c/86900/

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links