Page MenuHomePhabricator
Create Task
Maniphest T64429

Zuul: run 'test' jobs on jenkins when trusted user votes +1 and only 'check' jobs was ran
Closed, ResolvedPublic
Actions

Description

Just an idea:

Run 'test' jobs on jenkins when trusted user votes +1 on gerrit and only 'check' jobs was ran on that patch set.

Version: wmf-deployment
Severity: enhancement
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=64015

Details

Reference
bz62429
SubjectRepoBranchLines +/-
Code review by whitelisted users should triggers testsintegration/configmaster+50 -0
Customize query in gerrit

Related Objects

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:53 AM
• bzimport added a project: Continuous-Integration-Infrastructure.
• bzimport set Reference to bz62429.
• bzimport added a subscriber: Unknown Object (MLST).
duplicatebug created this task.Mar 8 2014, 8:01 AM
hashar added a comment.Aug 19 2014, 2:39 PM

Should be doable with some magic Zuul configuration. I have a draft change at https://gerrit.wikimedia.org/r/#/c/127259/ which might be doing it. Need a way to test it on labs first.

hashar added a comment.Aug 19 2014, 2:41 PM

This was later proposed as a comment on bug 64015 which was about enabling trusted user to retrigger by commenting 'recheck' ( https://bugzilla.wikimedia.org/show_bug.cgi?id=64015#c5 ).

gerritbot added a comment.Aug 19 2014, 2:41 PM

Change 127259 had a related patch set uploaded by Hashar:
Code review by whitelisted users should triggers tests

https://gerrit.wikimedia.org/r/127259

gerritbot added a comment.Oct 10 2014, 1:15 PM

Change 127259 abandoned by Hashar:
Code review by whitelisted users should triggers tests

Reason:
Configuration files are now in integration/config.git Might give that a poke later on.

https://gerrit.wikimedia.org/r/127259

hashar lowered the priority of this task from Medium to Lowest.Nov 24 2014, 10:28 AM
hashar set Security to None.
duplicatebug unsubscribed.Dec 13 2014, 4:32 PM
• demon unsubscribed.Dec 16 2014, 6:05 PM
gerritbot added a comment.Jan 14 2015, 2:12 PM

Change 184886 had a related patch set uploaded (by Hashar):
Code review by whitelisted users should triggers tests

https://gerrit.wikimedia.org/r/184886

Patch-For-Review

hashar claimed this task.Jan 14 2015, 7:39 PM
hashar moved this task from Untriaged to In-progress on the Continuous-Integration-Infrastructure board.
jayvdb mentioned this in T87413: Whitelist people with +2 rights.Jan 23 2015, 7:06 AM
Ricordisamoa subscribed.Jan 23 2015, 7:12 AM
jayvdb subscribed.Jan 23 2015, 7:15 AM
hashar moved this task from In-progress to Ready on the Continuous-Integration-Infrastructure board.Feb 3 2015, 8:36 PM
hashar moved this task from Ready to In-progress on the Continuous-Integration-Infrastructure board.
hashar removed hashar as the assignee of this task.Mar 17 2015, 9:13 AM
hashar moved this task from In-progress to Untriaged on the Continuous-Integration-Infrastructure board.

Although there is a patch pending, I don't have spare bandwith to test/deploy it :( Moving to backlog.

Krinkle moved this task from Untriaged to Backlog on the Continuous-Integration-Infrastructure board.Apr 7 2015, 2:31 PM
gerritbot subscribed.Nov 17 2015, 5:30 PM

Change 184886 merged by jenkins-bot:
Code review by whitelisted users should triggers tests

https://gerrit.wikimedia.org/r/184886

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 17 2015, 5:30 PM
JanZerebecki closed this task as Resolved.Nov 17 2015, 6:38 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits