*** spsurya has joined #zuul | 01:07 | |
*** spsurya has quit IRC | 04:48 | |
*** hashar has joined #zuul | 05:03 | |
*** nchakrab has joined #zuul | 05:25 | |
*** nchakrab has quit IRC | 06:08 | |
*** nchakrab has joined #zuul | 06:08 | |
*** nchakrab_ has joined #zuul | 06:09 | |
*** nchakrab has quit IRC | 06:13 | |
*** nchakrab has joined #zuul | 06:17 | |
*** nchakrab_ has quit IRC | 06:21 | |
*** pcaruana has joined #zuul | 06:30 | |
*** nchakrab has quit IRC | 06:36 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Improve logging of GithubTriggerEvents https://review.openstack.org/575014 | 06:53 |
---|---|---|
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two https://review.openstack.org/577354 | 06:53 |
*** nchakrab has joined #zuul | 06:53 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant one https://review.openstack.org/577354 | 06:53 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two https://review.openstack.org/577355 | 06:54 |
*** jpena|off is now known as jpena | 06:55 | |
tobiash | corvus, clarkb: that are a reproducer and two approaches to fix the github multi-enqueue problem | 06:55 |
tobiash | corvus, clarkb: both are somewhat a compromise due to missing information in the data model github offers us for pull requests | 06:56 |
tobiash | corvus, clarkb: I think variant one has a slight risk of replacing newer pull requests by older ones where variant two feels a bit like a hack | 06:57 |
tobiash | corvus, clarkb: but that are the only options I found so I'd favor variant two | 06:58 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: github: prevent AttributeError when missing branch protection https://review.openstack.org/577357 | 07:02 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant one https://review.openstack.org/577354 | 07:06 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two https://review.openstack.org/577355 | 07:06 |
*** nchakrab_ has joined #zuul | 07:10 | |
*** gtema has joined #zuul | 07:11 | |
*** nchakrab has quit IRC | 07:11 | |
*** Rohaan has joined #zuul | 07:18 | |
*** Rohaan has quit IRC | 07:36 | |
goern | hey'll I am using github and would like to merge PR based on certain conditions encoded as labels: 'bot and not (do-not-merge or do-not-merge/work-in-progress)' I think right now it is not possible to do that with zuul, or am I wrong? | 07:38 |
*** Rohaan has joined #zuul | 07:43 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement a Runc driver https://review.openstack.org/535556 | 07:59 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement a Kubernetes driver https://review.openstack.org/535557 | 08:01 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement an OpenShift resource provider https://review.openstack.org/570667 | 08:02 |
*** sshnaidm is now known as sshnaidm|off | 08:05 | |
*** Rohaan has quit IRC | 08:24 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 08:38 |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 08:51 |
*** Rohaan has joined #zuul | 08:51 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix branch parsing in _event_pull https://review.openstack.org/577371 | 08:54 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix branch parsing in _event_pull https://review.openstack.org/577371 | 09:04 |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 09:17 |
*** nchakrab_ has quit IRC | 09:29 | |
*** nchakrab has joined #zuul | 09:30 | |
*** nchakrab has quit IRC | 09:43 | |
*** nchakrab has joined #zuul | 09:43 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 10:01 |
goern | and I think there is also no way to add a label via a pipeline (for example on successful run) | 10:04 |
*** nchakrab has quit IRC | 10:09 | |
tobiash | goern: you can set labels in pipeline requirements and rejects | 10:16 |
goern | tobiash, but I can only set them to a certain list, not add labels to an existing list of labels!? | 10:16 |
tobiash | goern: I think you cannot set them but require/reject them in a pipeline | 10:17 |
tobiash | but you cannot define logical expressions | 10:17 |
goern | ja, right. thats what I got from the docs... where can I open a feature request? | 10:17 |
*** nchakrab has joined #zuul | 10:21 | |
*** nchakrab_ has joined #zuul | 10:22 | |
*** nchakrab has quit IRC | 10:25 | |
tobiash | goern: we have tickets in storyboard https://storyboard.openstack.org/#!/project/679 | 10:25 |
tobiash | goern: as I read work-in-progress you also might be interested in combining zuul with other github apps like https://github.com/wip/app | 10:27 |
tobiash | and it's much easier to interact with pull request status instead of labels for many things | 10:28 |
goern | tobiash, so that the pipeline will check for the wip application?! | 10:29 |
goern | tobiash, that sounds even better than using labels | 10:29 |
tobiash | goern: the gate pipeline will check every other status which is marked as required in the branch protection | 10:30 |
tobiash | goern: but you also can list any status in any pipeline's requirement | 10:30 |
goern | tobiash, I created https://storyboard.openstack.org/#!/story/2002646 any advice or is the verbosity level ok?! | 10:39 |
tobiash | goern: I think an explanation of the real use case would be good so there can be a discussion if this is the right solution of your problem | 10:42 |
goern | ack | 10:44 |
*** jpena is now known as jpena|lunch | 11:10 | |
*** fbo has quit IRC | 11:14 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 11:19 |
*** jpena|lunch is now known as jpena | 12:02 | |
*** Rohaan has quit IRC | 12:10 | |
*** rlandy has joined #zuul | 12:28 | |
*** nchakrab_ has quit IRC | 12:29 | |
*** elyezer has joined #zuul | 12:58 | |
*** nchakrab has joined #zuul | 13:00 | |
*** trishnag has quit IRC | 13:19 | |
*** myoung|off is now known as myoung|ruck | 13:22 | |
*** pwhalen has quit IRC | 13:31 | |
*** pwhalen has joined #zuul | 13:42 | |
*** pwhalen has joined #zuul | 13:42 | |
*** nchakrab has quit IRC | 13:52 | |
*** trishnag has joined #zuul | 14:29 | |
*** acozine1 has joined #zuul | 14:40 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref) https://review.openstack.org/576907 | 15:10 |
Shrews | ooh, i found a static driver bug | 15:32 |
Shrews | joy | 15:32 |
mordred | Shrews: \o/ | 15:57 |
openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Static driver: fix node registration https://review.openstack.org/577464 | 16:03 |
Shrews | that should fix it | 16:03 |
*** pcaruana has quit IRC | 16:04 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Log more information about what events trigger a reconfig https://review.openstack.org/575871 | 16:13 |
tobiash | corvus, clarkb: that fixes the tests ^ | 16:14 |
tobiash | mordred: 570417 would also be an easy review | 16:27 |
tobiash | and 575050 is a small doc fix | 16:29 |
mordred | tobiash: both done | 16:30 |
tobiash | mordred: thanks :) | 16:31 |
*** jpena is now known as jpena|off | 16:43 | |
openstackgerrit | Merged openstack-infra/zuul master: Don't process lock files as refs in tests https://review.openstack.org/570417 | 16:49 |
openstackgerrit | Merged openstack-infra/zuul master: Clarify usage of the zuul_success variable https://review.openstack.org/575050 | 16:54 |
*** gtema has quit IRC | 16:56 | |
openstackgerrit | Merged openstack-infra/zuul master: Log more information about what events trigger a reconfig https://review.openstack.org/575871 | 16:56 |
*** electrofelix has quit IRC | 17:23 | |
*** acozine1 has quit IRC | 17:27 | |
Shrews | corvus: should i create a release note about the static driver change? | 18:53 |
*** acozine1 has joined #zuul | 19:03 | |
*** yolanda__ has joined #zuul | 19:12 | |
*** yolanda_ has quit IRC | 19:16 | |
*** yolanda__ has quit IRC | 19:18 | |
Shrews | mordred: about the change from occ to osdk in nodepool (https://review.openstack.org/566158), that time out seems to be related | 19:23 |
Shrews | mordred: left a link to the error in the builder log on that change | 19:23 |
*** cmurphy is now known as cmurphy_vacation | 19:31 | |
*** yolanda has joined #zuul | 21:15 | |
*** myoung has joined #zuul | 22:53 | |
*** mhu` has joined #zuul | 22:54 | |
*** pabelanger_ has joined #zuul | 22:54 | |
*** _ari__ has joined #zuul | 22:54 | |
*** hashar has quit IRC | 22:54 | |
*** _ari_ has quit IRC | 22:56 | |
*** weshay_PTO has quit IRC | 22:57 | |
*** weshay has joined #zuul | 22:57 | |
*** pabelanger has quit IRC | 22:57 | |
*** mhu has quit IRC | 22:57 | |
*** myoung|ruck has quit IRC | 22:58 | |
*** myoung is now known as myoung|off | 23:03 | |
*** rlandy has quit IRC | 23:05 | |
*** pabelanger_ is now known as pabelanger | 23:45 | |
*** acozine1 has quit IRC | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!