| clarkb | mordred: ^ do you know what PR description means? is that topmost comment? | 00:05 |
|---|---|---|
| pabelanger | clarkb: depends-on for commit message should be fine | 00:10 |
| clarkb | pabelanger: it didn't work on the commit mesasge for sure, I waited for it to fail that before I updated the PR comment itself | 00:11 |
| clarkb | so I think the docs are right | 00:11 |
| clarkb | new problem is it thinks the depends on cannot merge which it totally can (I think double checking that) | 00:11 |
| clarkb | I'm reading the scheduler logs and they don't really say why it thinks it can't merge just goes from here is the dependency to cannot merge, report failure | 00:12 |
| clarkb | http://paste.openstack.org/show/723011/ I'm pretty stumped | 00:14 |
| clarkb | I actually think this may be a bug. I'm going to push a new commit and see if that works because i think that goes through the addChange() path which this didn't, this was just an update to the PR | 00:30 |
| clarkb | ya that seems to be doing more what I expect | 00:31 |
| clarkb | tobiash corvus ^ I'll try to turn that into a bug report later | 00:31 |
| clarkb | also if you push multiple commits one after the other it may get confused | 00:38 |
| clarkb | it didn't remove the PR from the queue when a new "patchset" was pushed to it | 00:39 |
| *** harlowja has quit IRC | 01:25 | |
| *** nhicher has quit IRC | 01:55 | |
| *** fbo_ has quit IRC | 01:55 | |
| tobiash | clarkb: yes I also observed that sometimes a pr is in the same pipeline twice | 03:23 |
| tobiash | I way planning to analyse this next week | 03:23 |
| tobiash | s/way/was | 03:24 |
| *** harlowja has joined #zuul | 03:27 | |
| *** harlowja has quit IRC | 04:14 | |
| *** swest has quit IRC | 04:41 | |
| *** swest has joined #zuul | 04:49 | |
| *** swest has quit IRC | 07:59 | |
| *** swest has joined #zuul | 08:50 | |
| *** Rohaan has joined #zuul | 09:26 | |
| *** Rohaan has quit IRC | 09:57 | |
| *** Rohaan has joined #zuul | 10:08 | |
| *** Rohaan has quit IRC | 10:39 | |
| *** swest has quit IRC | 13:27 | |
| *** elyezer has quit IRC | 14:54 | |
| *** elyezer_ has quit IRC | 14:54 | |
| *** elyezer has joined #zuul | 14:54 | |
| *** elyezer has quit IRC | 15:02 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Accumulate errors in context managers - part 2 https://review.openstack.org/574028 | 16:58 |
| *** gianni has joined #zuul | 17:14 | |
| gianni | ciao | 17:14 |
| *** gianni is now known as Guest78116 | 17:14 | |
| *** Guest78116 has left #zuul | 17:15 | |
| *** aspiers[m] has quit IRC | 18:06 | |
| openstackgerrit | Merged openstack-infra/zuul master: Accumulate errors in context managers https://review.openstack.org/573438 | 19:00 |
| *** aspiers[m] has joined #zuul | 20:30 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!