| *** ianychoi has joined #zuul | 00:00 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: executor: run cleanup playbook on stop https://review.opendev.org/661881 | 00:33 |
|---|---|---|
| openstackgerrit | Paul Belanger proposed zuul/zuul master: Discard GitHub PullReview if incomplete https://review.opendev.org/662347 | 00:55 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: docs: add cleanup-run documentation https://review.opendev.org/662147 | 03:18 |
| *** toabctl has joined #zuul | 04:41 | |
| *** raukadah is now known as chandankumar | 04:51 | |
| *** pcaruana has joined #zuul | 05:03 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: executor: run cleanup playbook on stop https://review.opendev.org/661881 | 05:19 |
| tobiash | pabelanger, jlk: tossing the review is generally not the correct thing, I've tried that, more details in https://review.opendev.org/662347 | 06:59 |
| *** gtema has joined #zuul | 07:30 | |
| *** tosky has joined #zuul | 07:33 | |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 09:24 |
| *** panda is now known as panda|ruck | 09:24 | |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 09:29 |
| *** electrofelix has joined #zuul | 09:38 | |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 10:12 |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 10:12 |
| *** gtema has quit IRC | 10:45 | |
| pabelanger | tobiash: replied, sounds like we just keep doing normal business with zuul then | 10:57 |
| *** gtema has joined #zuul | 11:08 | |
| *** flepied has quit IRC | 11:21 | |
| *** flepied has joined #zuul | 11:23 | |
| *** tjgresha has quit IRC | 11:30 | |
| *** tjgresha has joined #zuul | 11:33 | |
| *** EmilienM is now known as EvilienM | 11:45 | |
| *** gtema has quit IRC | 11:58 | |
| *** logan- has quit IRC | 12:07 | |
| *** logan- has joined #zuul | 12:08 | |
| *** gtema has joined #zuul | 12:08 | |
| *** openstack has joined #zuul | 12:30 | |
| *** ChanServ sets mode: +o openstack | 12:30 | |
| ofosos | SpamapS, corvus: Could you give https://review.opendev.org/#/c/658335 another go? | 12:34 |
| *** rlandy has joined #zuul | 12:41 | |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 13:26 |
| *** electrofelix has quit IRC | 13:43 | |
| pabelanger | clarkb: tobiash: jlk: applying https://github.com/sigmavirus24/github3.py/pull/944 locally has supressed the exception in zuul. For now, I've abandoned my patch to zuul about removing the review, since I see no place where we actually depend on commit_id | 13:50 |
| pabelanger | zuul.o.o is also affected, so once github3.py is released we should consider upgrading it there too | 13:51 |
| tobiash | pabelanger: ok, as a further improvement we should filter the positive reviews based on the branch protection settings | 13:51 |
| pabelanger | tobiash: yah, I think that is a great idea, I don't have the time currently to work on that. | 13:52 |
| tobiash | I'll put it in my backlog | 13:52 |
| pabelanger | tobiash: on to next issue, trying to figure out http timeout issues to github | 13:53 |
| pabelanger | jlk: oh, I may be albe to commit some time to checks api work, I seen your PR last night again. | 13:53 |
| pabelanger | we've seen some interested from ansible folks to start using it | 13:54 |
| pabelanger | been said that zuul comments are too spammy | 13:54 |
| dmsimard | pabelanger: spammy how | 14:12 |
| pabelanger | dmsimard: by leaving comments with test results | 14:13 |
| pabelanger | dmsimard: I mean, compared to ansibullbot, is it pretty mild | 14:13 |
| pabelanger | :) | 14:14 |
| dmsimard | pabelanger: as opposed to shippable which uses the check api ? | 14:14 |
| pabelanger | but, I've mentioned we are looking to using check api (which shipable does today) | 14:14 |
| pabelanger | yah | 14:14 |
| pabelanger | so, humans at ansible are comparing it to shippable (good or bad) and would like the results in the same place | 14:14 |
| dmsimard | is history preserved with the check api ? | 14:14 |
| pabelanger | I don't know | 14:14 |
| pabelanger | I haven't really looked at it | 14:15 |
| pabelanger | I think we'd link to zuul dashboard regardless | 14:15 |
| dmsimard | iirc since you force push, the previous commit "disappears" so you can't easily find the results of a check for a previous commit | 14:15 |
| pabelanger | dmsimard: example: https://github.com/ansible/ansible/pull/45469 | 14:16 |
| pabelanger | in ansible network case, we could report back a lot of jobs, depending on what changes | 14:16 |
| *** chandankumar is now known as raukadah | 14:18 | |
| *** tflink_ is now known as tflink | 14:22 | |
| dmsimard | pabelanger: that PR is not a good example of spammy behavior though :p | 14:22 |
| dmsimard | I mean, unless there's more than the one comment that I'm seeing | 14:23 |
| pabelanger | dmsimard: right, we don't think it is, but some humans do. When you compare to shipable | 14:23 |
| dmsimard | one comment is spammy ? | 14:23 |
| pabelanger | I think it is more test results via comments in general | 14:24 |
| dmsimard | ah | 14:24 |
| pabelanger | there are humans that won't even look at openstack-zuul results on OS clouds modules, because they consider them spam | 14:24 |
| pabelanger | so yah, it feedback like this about how to make zuul 'better' | 14:25 |
| pabelanger | (their words) | 14:25 |
| pabelanger | dmsimard: however, there are other humans that are loving the pre-merge testing, for ansible network things. They really like out fast it is to iterate on a patch | 14:25 |
| pabelanger | and gaining more confidence that merging code isn't going to break things | 14:26 |
| pabelanger | so, that is a success | 14:26 |
| *** openstackstatus has joined #zuul | 14:35 | |
| *** ChanServ sets mode: +v openstackstatus | 14:35 | |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add autohold-info CLI command https://review.opendev.org/662487 | 14:36 |
| *** flepied has quit IRC | 14:37 | |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Store autohold requests in zookeeper https://review.opendev.org/661114 | 14:38 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add autohold-info CLI command https://review.opendev.org/662487 | 14:38 |
| *** jangutter has quit IRC | 14:41 | |
| *** flepied has joined #zuul | 14:53 | |
| openstackgerrit | Mark Meyer proposed zuul/zuul master: Extend event reporting https://review.opendev.org/662134 | 14:55 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Disable gc in test_scheduler.TestExecutor https://review.opendev.org/661316 | 15:04 |
| openstackgerrit | Slawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node https://review.opendev.org/643733 | 15:05 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Set the DiskAccountant log as warning level https://review.opendev.org/660996 | 15:10 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Record held node IDs with authold request https://review.opendev.org/662498 | 15:10 |
| -openstackstatus- NOTICE: Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running. | 15:11 | |
| *** ChanServ changes topic to "Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running." | 15:11 | |
| ofosos | Nice, just when I want to do the last push for the day :) Weekend is apparently coming early... | 15:11 |
| fungi | ;) | 15:12 |
| fungi | we waited just for your patch! | 15:12 |
| fungi | (not really though) | 15:13 |
| fungi | but great timing nonetheless | 15:13 |
| jlk | pabelanger: abandonment is better upon reflection. I was too hasty in my thoughts yesterday. | 15:20 |
| *** gtema has quit IRC | 15:36 | |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Record held node IDs with autohold request https://review.opendev.org/662498 | 15:48 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Record held node IDs with autohold request https://review.opendev.org/662498 | 15:51 |
| SpamapS | ofosos:I'll peek later today | 15:55 |
| *** ChanServ changes topic to "Discussion of the project gating system Zuul | Website: https://zuul-ci.org/ | Docs: https://zuul-ci.org/docs/ | Source: https://git.zuul-ci.org/ | Channel logs: http://eavesdrop.openstack.org/irclogs/%23zuul/ | Weekly updates: https://etherpad.openstack.org/p/zuul-update-email" | 16:01 | |
| *** rlandy is now known as rlandy|brb | 16:02 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 16:15 |
| *** rlandy|brb is now known as rlandy | 16:45 | |
| -openstackstatus- NOTICE: Gerrit is back up and running again. Thank you for your patience and sorry for the delay in this notification (we thought the statusbot was still busy updating channel topics). | 16:48 | |
| ofosos | SpamapS: thanks | 16:49 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 17:31 |
| *** panda|ruck has quit IRC | 17:59 | |
| *** panda has joined #zuul | 18:01 | |
| *** tosky has quit IRC | 18:09 | |
| openstackgerrit | Brian Haley proposed zuul/zuul-jobs master: Update repo file for fedora-29 https://review.opendev.org/662536 | 18:11 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 18:11 |
| *** flepied has quit IRC | 18:35 | |
| *** openstackgerrit has quit IRC | 19:01 | |
| *** openstackgerrit has joined #zuul | 19:10 | |
| openstackgerrit | Slawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node https://review.opendev.org/643733 | 19:10 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 19:37 |
| *** dmsimard has quit IRC | 19:55 | |
| *** dmsimard has joined #zuul | 19:56 | |
| *** dmsimard has quit IRC | 19:56 | |
| *** mattw4 has joined #zuul | 19:58 | |
| *** dmsimard has joined #zuul | 20:01 | |
| mattw4 | Hi all, I just noticed my test instance of Zuul-scheduler is reprocessing trigger events from openstack/devstack. Is there a maintenance event going on? Is this reprocessing normal or did I break something? | 20:11 |
| AJaeger | mattw4: we had a maintenance event where some repos were renamed, this included a gerrit downtime. | 20:13 |
| mattw4 | thanks AJaeger. Where should I look for those notices? | 20:13 |
| AJaeger | on the openstack-discuss mailing list - or here on IRC, we push those out. Also, you can follow on wiki and twitter, let me find links... | 20:14 |
| AJaeger | https://twitter.com/openstackinfra | 20:14 |
| AJaeger | https://wiki.openstack.org/wiki/Infrastructure_Status | 20:15 |
| AJaeger | mattw4: those two should get the same messages | 20:15 |
| mattw4 | Thank you AJaeger, following now :) | 20:16 |
| clarkb | I sent the notice to the zuul list as well | 20:18 |
| clarkb | though I think we want to move to a dedicated opendev announcements list in the future (we'll send email to the other lists encouraging people to subscribe when that happens) | 20:18 |
| mattw4 | thanks clarkb & AJaeger ... I will follow the Zuul list for sure | 20:19 |
| *** dmsimard has joined #zuul | 20:21 | |
| *** rfolco has quit IRC | 20:45 | |
| *** pwhalen has joined #zuul | 21:20 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 21:21 |
| *** pwhalen has quit IRC | 21:26 | |
| corvus | regarding our conversation earlier about where to put jobs like that -- maybe it's ok to keep that in zuul-jobs after all. it does have a nodeset with opendev-specific labels, but that won't cause a problem for anyone unless they try to run it (it's an anonymous nodeset, so it's not like it's creating a named nodeset definition). so maybe we can keep test jobs like that in the zuul-jobs repo, but just | 21:44 |
| corvus | segregate them from the "real" jobs so they don't get in the way. | 21:44 |
| clarkb | will people have to exclude projects when they load zuul-jobs? | 21:46 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 21:48 |
| *** pcaruana has quit IRC | 21:54 | |
| *** rlandy has quit IRC | 22:32 | |
| corvus | clarkb: nah, we can continue doing the thing where we put the project stanza in project-config | 23:02 |
| clarkb | ah | 23:08 |
| *** mattw4 has quit IRC | 23:15 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: registry test job https://review.opendev.org/661327 | 23:26 |
| *** panda has quit IRC | 23:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!