Friday, 2018-06-22

*** spsurya has joined #zuul01:07
*** spsurya has quit IRC04:48
*** hashar has joined #zuul05:03
*** nchakrab has joined #zuul05:25
*** nchakrab has quit IRC06:08
*** nchakrab has joined #zuul06:08
*** nchakrab_ has joined #zuul06:09
*** nchakrab has quit IRC06:13
*** nchakrab has joined #zuul06:17
*** nchakrab_ has quit IRC06:21
*** pcaruana has joined #zuul06:30
*** nchakrab has quit IRC06:36
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Improve logging of GithubTriggerEvents  https://review.openstack.org/57501406:53
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two  https://review.openstack.org/57735406:53
*** nchakrab has joined #zuul06:53
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant one  https://review.openstack.org/57735406:53
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two  https://review.openstack.org/57735506:54
*** jpena|off is now known as jpena06:55
tobiashcorvus, clarkb: that are a reproducer and two approaches to fix the github multi-enqueue problem06:55
tobiashcorvus, clarkb: both are somewhat a compromise due to missing information in the data model github offers us for pull requests06:56
tobiashcorvus, 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 hack06:57
tobiashcorvus, clarkb: but that are the only options I found so I'd favor variant two06:58
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: github: prevent AttributeError when missing branch protection  https://review.openstack.org/57735707:02
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant one  https://review.openstack.org/57735407:06
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix missing dequeue on synchronized pull request variant two  https://review.openstack.org/57735507:06
*** nchakrab_ has joined #zuul07:10
*** gtema has joined #zuul07:11
*** nchakrab has quit IRC07:11
*** Rohaan has joined #zuul07:18
*** Rohaan has quit IRC07:36
goernhey'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 #zuul07:43
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool master: Implement a Runc driver  https://review.openstack.org/53555607:59
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool master: Implement a Kubernetes driver  https://review.openstack.org/53555708:01
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool master: Implement an OpenShift resource provider  https://review.openstack.org/57066708:02
*** sshnaidm is now known as sshnaidm|off08:05
*** Rohaan has quit IRC08:24
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690708:38
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690708:51
*** Rohaan has joined #zuul08:51
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix branch parsing in _event_pull  https://review.openstack.org/57737108:54
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix branch parsing in _event_pull  https://review.openstack.org/57737109:04
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690709:17
*** nchakrab_ has quit IRC09:29
*** nchakrab has joined #zuul09:30
*** nchakrab has quit IRC09:43
*** nchakrab has joined #zuul09:43
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690710:01
goernand I think there is also no way to add a label via a pipeline (for example on successful run)10:04
*** nchakrab has quit IRC10:09
tobiashgoern: you can set labels in pipeline requirements and rejects10:16
goerntobiash, but I can only set them to a certain list, not add labels to an existing list of labels!?10:16
tobiashgoern: I think you cannot set them but require/reject them in a pipeline10:17
tobiashbut you cannot define logical expressions10:17
goernja, right. thats what I got from the docs... where can I open a feature request?10:17
*** nchakrab has joined #zuul10:21
*** nchakrab_ has joined #zuul10:22
*** nchakrab has quit IRC10:25
tobiashgoern: we have tickets in storyboard https://storyboard.openstack.org/#!/project/67910:25
tobiashgoern: as I read work-in-progress you also might be interested in combining zuul with other github apps like https://github.com/wip/app10:27
tobiashand it's much easier to interact with pull request status instead of labels for many things10:28
goerntobiash, so that the pipeline will check for the wip application?!10:29
goerntobiash, that sounds even better than using labels10:29
tobiashgoern: the gate pipeline will check every other status which is marked as required in the branch protection10:30
tobiashgoern: but you also can list any status in any pipeline's requirement10:30
goerntobiash, I created https://storyboard.openstack.org/#!/story/2002646 any advice or is the verbosity level ok?!10:39
tobiashgoern: 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 problem10:42
goernack10:44
*** jpena is now known as jpena|lunch11:10
*** fbo has quit IRC11:14
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690711:19
*** jpena|lunch is now known as jpena12:02
*** Rohaan has quit IRC12:10
*** rlandy has joined #zuul12:28
*** nchakrab_ has quit IRC12:29
*** elyezer has joined #zuul12:58
*** nchakrab has joined #zuul13:00
*** trishnag has quit IRC13:19
*** myoung|off is now known as myoung|ruck13:22
*** pwhalen has quit IRC13:31
*** pwhalen has joined #zuul13:42
*** pwhalen has joined #zuul13:42
*** nchakrab has quit IRC13:52
*** trishnag has joined #zuul14:29
*** acozine1 has joined #zuul14:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant-scoped autohold, enqueue (change, ref)  https://review.openstack.org/57690715:10
Shrewsooh, i found a static driver bug15:32
Shrewsjoy15:32
mordredShrews: \o/15:57
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Static driver: fix node registration  https://review.openstack.org/57746416:03
Shrewsthat should fix it16:03
*** pcaruana has quit IRC16:04
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Log more information about what events trigger a reconfig  https://review.openstack.org/57587116:13
tobiashcorvus, clarkb: that fixes the tests ^16:14
tobiashmordred: 570417 would also be an easy review16:27
tobiashand 575050 is a small doc fix16:29
mordredtobiash: both done16:30
tobiashmordred: thanks :)16:31
*** jpena is now known as jpena|off16:43
openstackgerritMerged openstack-infra/zuul master: Don't process lock files as refs in tests  https://review.openstack.org/57041716:49
openstackgerritMerged openstack-infra/zuul master: Clarify usage of the zuul_success variable  https://review.openstack.org/57505016:54
*** gtema has quit IRC16:56
openstackgerritMerged openstack-infra/zuul master: Log more information about what events trigger a reconfig  https://review.openstack.org/57587116:56
*** electrofelix has quit IRC17:23
*** acozine1 has quit IRC17:27
Shrewscorvus: should i create a release note about the static driver change?18:53
*** acozine1 has joined #zuul19:03
*** yolanda__ has joined #zuul19:12
*** yolanda_ has quit IRC19:16
*** yolanda__ has quit IRC19:18
Shrewsmordred: about the change from occ to osdk in nodepool (https://review.openstack.org/566158), that time out seems to be related19:23
Shrewsmordred: left a link to the error in the builder log on that change19:23
*** cmurphy is now known as cmurphy_vacation19:31
*** yolanda has joined #zuul21:15
*** myoung has joined #zuul22:53
*** mhu` has joined #zuul22:54
*** pabelanger_ has joined #zuul22:54
*** _ari__ has joined #zuul22:54
*** hashar has quit IRC22:54
*** _ari_ has quit IRC22:56
*** weshay_PTO has quit IRC22:57
*** weshay has joined #zuul22:57
*** pabelanger has quit IRC22:57
*** mhu has quit IRC22:57
*** myoung|ruck has quit IRC22:58
*** myoung is now known as myoung|off23:03
*** rlandy has quit IRC23:05
*** pabelanger_ is now known as pabelanger23:45
*** acozine1 has quit IRC23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!