| mordred | clarkb, pabelanger: woot! | 01:22 |
|---|---|---|
| mordred | jlk: nicely done | 01:22 |
| *** jamielennox is now known as jamielennox|away | 02:07 | |
| *** jamielennox|away is now known as jamielennox | 03:23 | |
| tobiash | pabelanger, mordred, thx | 05:01 |
| *** dmsimard has quit IRC | 06:55 | |
| *** dmsimard has joined #zuul | 06:56 | |
| *** corvus has quit IRC | 06:56 | |
| *** corvus has joined #zuul | 06:56 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul feature/zuulv3: Remove outdated TODO https://review.openstack.org/456089 | 07:13 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul feature/zuulv3: Set the git sshkey for the connection https://review.openstack.org/456090 | 07:17 |
| *** hashar has joined #zuul | 07:25 | |
| *** dmellado has joined #zuul | 08:23 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul feature/zuulv3: Only grab the gerrit change if necessary https://review.openstack.org/456162 | 10:41 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul feature/zuulv3: Only grab the gerrit change if necessary https://review.openstack.org/456162 | 10:42 |
| *** hashar has quit IRC | 10:46 | |
| *** hashar has joined #zuul | 12:09 | |
| *** hashar has quit IRC | 12:37 | |
| *** dkranz has quit IRC | 12:56 | |
| *** hashar has joined #zuul | 12:58 | |
| *** dkranz has joined #zuul | 14:14 | |
| Shrews | mordred: the toughest part of this log streaming thing will be figuring out where to get the mapping of job id to assigned node :/ | 14:30 |
| Shrews | mordred: oh, also, if we have a multinode job, which console is streamed? | 14:32 |
| mordred | Shrews: all of them | 14:45 |
| mordred | Shrews: that part should actually already be done - the streaming of the logs from the node to the executor is already happening | 14:46 |
| mordred | Shrews: and it works for multi-node already - so the stream of stuff should be winding up in ansible_log.txt in the workdir | 14:46 |
| Shrews | mordred: ah, i see | 14:49 |
| pabelanger | new voluptuous release today | 15:08 |
| pabelanger | don't think zuul likes it | 15:08 |
| mordred | pabelanger: oh goodie | 15:48 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Handle dict extras in Voluptuous Schema https://review.openstack.org/456270 | 15:55 |
| clarkb | ^ is fix | 15:56 |
| clarkb | I'm about to go look at v3 now to see if it needs fixing too | 15:56 |
| clarkb | ya it iwll need similar treatment so working on that patch now | 15:57 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul feature/zuulv3: Handle dict extras in Voluptuous Schema https://review.openstack.org/456273 | 16:01 |
| mordred | clarkb: how is Schema(dict) ... oh, nevermind. typing the question helped me understand the answer | 16:08 |
| clarkb | mordred: see the Extra dictionary keys section at https://pypi.python.org/pypi/voluptuous | 16:09 |
| mordred | yah- no, I get it | 16:09 |
| clarkb | I'm guessing they changed how this works at some point but haven't tracked that down yet. I guess I should to set voluptuous lower bound | 16:09 |
| mordred | clarkb: my question was going to be "how is Schema(dict) different than Schema({})" - but typing helped me notice the lack of () after dict | 16:09 |
| clarkb | looks like version 10 is where Schema(dict) becomes documented | 16:11 |
| clarkb | do you want me to udpate my patches to >=0.10.2 on voluptuous too? | 16:11 |
| clarkb | probably a good idea | 16:11 |
| mordred | clarkb: ++ or do a follow up - I believe pabelanger and I have +A's both of your patches already | 16:13 |
| openstackgerrit | Merged openstack-infra/zuul master: Handle dict extras in Voluptuous Schema https://review.openstack.org/456270 | 16:13 |
| clarkb | ah ok will do followup | 16:14 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul feature/zuulv3: Update voluptuous requirement https://review.openstack.org/456281 | 16:15 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Update voluptuous requirement https://review.openstack.org/456282 | 16:15 |
| clarkb | and two followups | 16:16 |
| pabelanger | mordred: clarkb: I think we need a new zuul release for our project-config job. However 2.5.1 was release back in Sept 2016. Not sure we want to tag all the changes since then | 16:16 |
| clarkb | pabelanger: we run tip of master zuul so tagging should be fine | 16:17 |
| mordred | pabelanger: I _think_ all of the changes in master should be safe for folks following 2.5 | 16:17 |
| clarkb | and we havne't added features just fixes right? | 16:17 |
| mordred | thing so | 16:17 |
| mordred | checking | 16:17 |
| pabelanger | okay | 16:17 |
| clarkb | we should get the followups in too (should've gotten those in as part of original change oh well) | 16:18 |
| mordred | yah. I think the changes in master look good for a 2.5.2 | 16:19 |
| mordred | it contains the change of how we run ansible | 16:19 |
| mordred | which if people _are_ using 2.5, which they shouldn't be, but some are - their ansible will become more resilient | 16:19 |
| clarkb | gah I made a silly on my v3 changes | 16:26 |
| clarkb | it looks like gear may be broken on new pbr too | 16:32 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul feature/zuulv3: Handle dict extras in Voluptuous Schema https://review.openstack.org/456273 | 16:34 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul feature/zuulv3: Update voluptuous requirement https://review.openstack.org/456281 | 16:34 |
| clarkb | mordred: pabelanger ^ that cleans up my derp. Gonna go get some tea and breakfast then sort out gear's pbr<2.0 pin which appears to be breaking my local tox -epep8 on zuulv3 | 16:34 |
| clarkb | not sure how that is working elsewhere | 16:34 |
| *** hashar has quit IRC | 16:55 | |
| clarkb | mordred: https://review.openstack.org/#/c/456273/2 could use rereview after I fixed my derp | 17:05 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Handle dict extras in Voluptuous Schema https://review.openstack.org/456273 | 17:49 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Update voluptuous requirement https://review.openstack.org/456281 | 17:50 |
| pabelanger | clarkb: mordred: do you mind tagging 2.5.2, if we are ready | 18:56 |
| openstackgerrit | Paul Belanger proposed openstack-infra/nodepool master: Add mirror support for fedora-25 DIB https://review.openstack.org/456372 | 19:49 |
| openstackgerrit | Paul Belanger proposed openstack-infra/nodepool master: Switch to /etc/ci/mirror_info.sh for nodepool mirrors https://review.openstack.org/456374 | 19:53 |
| openstackgerrit | Merged openstack-infra/zuul master: Update voluptuous requirement https://review.openstack.org/456282 | 20:13 |
| *** cmurphy has joined #zuul | 20:24 | |
| *** dkranz has quit IRC | 20:43 | |
| *** dkranz has joined #zuul | 21:17 | |
| openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Allow using webapp from connections https://review.openstack.org/439831 | 21:38 |
| openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Support GitHub PR webhooks https://review.openstack.org/439834 | 21:38 |
| jlk | oh great, I'm back in the state where tests will pass locally but fail upstream :9 | 22:47 |
| jhesketh | Morning | 22:57 |
| SpamapS | jlk: what did you change to get local tests going? | 23:37 |
| SpamapS | I'm pretty concerned we have some weird interactions going on still. | 23:37 |
| * SpamapS still shaking off vacation fog | 23:37 | |
| clarkb | SpamapS: I think we do, my change to run test suite twice seems to fail | 23:37 |
| clarkb | and locally it works if I do tox -e py27 -- --concurrency=1 to have only a single runner | 23:38 |
| clarkb | so pretty sure something is being leaky | 23:38 |
| SpamapS | If I had to guess, it's in mysql or zk | 23:38 |
| SpamapS | but that's just a hunch | 23:38 |
| jesusaur | we did recently add the need for mysql, so that's a decent hunch | 23:38 |
| SpamapS | clarkb: if concurrency = 1 works consistently , then --analyze-isolation should reveal the culprit. | 23:39 |
| SpamapS | basically, run once, get fail, run with --analyze-isolation, and it should be able to bisect to get the interaction | 23:39 |
| clarkb | SpamapS: ya I have been distracted by other things and haven't gotten there | 23:39 |
| SpamapS | (not git bisect, test run bisect) | 23:39 |
| clarkb | when we ported openstack projects to testr I became very fond of that little tool | 23:40 |
| SpamapS | yep | 23:41 |
| SpamapS | though to be fair.. testr also _causes_ the problem by running things in parallel ;) | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!