| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable remaining test_crd_check_* tests https://review.openstack.org/452996 | 00:00 |
|---|---|---|
| SpamapS | jeblair: ^^ The patch above has me a bit confused | 00:01 |
| SpamapS | jeblair: the two assertions that are commented out, with "BUG" comments before.. when looking at the logs, those two events don't match any jobs, and thus, don't add things to the history. | 00:02 |
| SpamapS | jeblair: It's not clear to me why they don't match jobs though, as they're for projects which have jobs defined .. so.. o_O | 00:03 |
| SpamapS | and now I have to EOD earlier than expected.. so will have to keep hacking later tonight. | 00:03 |
| jeblair | SpamapS: i also have to eod; i'll look first thing tomorrow | 00:22 |
| *** harlowja has quit IRC | 02:06 | |
| *** harlowja has joined #zuul | 02:50 | |
| *** harlowja has quit IRC | 03:56 | |
| *** harlowja has joined #zuul | 04:07 | |
| *** harlowja has quit IRC | 04:33 | |
| *** toabctl has quit IRC | 05:06 | |
| *** toabctl has joined #zuul | 05:10 | |
| *** harlowja has joined #zuul | 05:11 | |
| *** bstinson has quit IRC | 05:14 | |
| *** SpamapS has quit IRC | 05:14 | |
| *** bstinson has joined #zuul | 05:15 | |
| *** jamielennox is now known as jamielennox|away | 05:15 | |
| *** SpamapS has joined #zuul | 05:23 | |
| *** jamielennox|away is now known as jamielennox | 05:37 | |
| *** harlowja has quit IRC | 06:02 | |
| *** madgoat has joined #zuul | 06:03 | |
| *** madgoat has left #zuul | 06:03 | |
| *** jamielennox is now known as jamielennox|away | 07:45 | |
| *** hashar has joined #zuul | 07:46 | |
| *** bhavik1 has joined #zuul | 07:57 | |
| *** jasondotstar has quit IRC | 09:00 | |
| *** jasondotstar has joined #zuul | 09:03 | |
| *** bhavik1 has quit IRC | 09:38 | |
| *** hashar has quit IRC | 10:34 | |
| *** jamielennox|away is now known as jamielennox | 11:03 | |
| *** hashar has joined #zuul | 12:12 | |
| jeblair | SpamapS: spotted the bug on 452996; left comment | 15:27 |
| SpamapS | jeblair: ty | 16:00 |
| SpamapS | jeblair: doh, I should have questioned common-config more ;) | 16:01 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable remaining test_crd_check_* tests https://review.openstack.org/452996 | 16:06 |
| jeblair | jesusaur: i left a real review on https://review.openstack.org/446275 | 17:06 |
| jesusaur | sweet! | 17:07 |
| *** hashar has quit IRC | 17:09 | |
| jesusaur | jeblair: I've replied to a couple of your comments | 17:42 |
| SpamapS | http://logs.openstack.org/96/452996/2/check/gate-zuul-python27-ubuntu-xenial/75ecd6f/console.html | 17:51 |
| SpamapS | weird failure in test_playbook | 17:51 |
| SpamapS | looks like it just failed the "are things still running" test | 17:51 |
| SpamapS | http://logs.openstack.org/96/452996/2/check/gate-zuul-python27-ubuntu-xenial/75ecd6f/console.html#_2017-04-04_16_18_21_694161 | 17:51 |
| SpamapS | Weird 2 minute pause there. | 17:51 |
| SpamapS | I wonder if the instance got migrated or something. | 17:52 |
| jeblair | jesusaur: i responded to your responses :) | 17:58 |
| jesusaur | jeblair: so, the case I'm thinking about has to do with a single project that has both dependent and independent pipelines (check and gate), and how to quickly report merge conflicts in check, but also report merge conflicts later in check that are cause by activity in gate | 18:04 |
| jesusaur | jeblair: i can write that up as another review response if you like, or we can chat here | 18:04 |
| jesusaur | jeblair: because in my experience, we can get queue times over an hour long, and waiting a couple hours to find out your tests were never going to start can be frustrating | 18:05 |
| clarkb | jesusaur: gerrit sort of helps with that by showing you what you conflict with, maybe just check against that in zuul and advise if any oare in the gate queu? | 18:06 |
| jeblair | jesusaur: well, a merge conflict in check will be reported as soon as the change in equeued into check | 18:10 |
| jeblair | jesusaur: in openstack-infra, we have a merge-check pipeline which will report merge conflicts asynchronously -- ie, if change B merges, it will report a merge-conflict on change A (which otherwise isn't in any pipeline). that uses the zuul trigger to check all of the open changes for the same project. | 18:14 |
| jeblair | jesusaur: that serves to keep changes out of the gate pipeline which are never going to merge, which i think is the case you were perhaps describing above? | 18:15 |
| jesusaur | jeblair: oh, right, the merge-check pipeline would also address my concern with stale changes in check, that works for me | 18:16 |
| jesusaur | jeblair: so i can remove the second merge job, but are you ok with scheduling the first early instead of waiting for the active window? | 18:16 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix constructor arguments to source https://review.openstack.org/451102 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add hostname to TriggerEvent https://review.openstack.org/452348 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add canonical hostname to source object https://review.openstack.org/451110 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fully qualify project configuration names https://review.openstack.org/451970 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add source to project and remove unused tenant attrs https://review.openstack.org/451969 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass source to project instantiations https://review.openstack.org/451596 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add a project index to Tenant https://review.openstack.org/451597 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove unused Tenant.getRepo method https://review.openstack.org/451929 | 18:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Use new tenant project index for zuul role refs https://review.openstack.org/451928 | 18:17 |
| jeblair | jesusaur: i'd like to keep the current behavior and only do it inside the active window. | 18:17 |
| SpamapS | mmmmmmmmmmmm stack-o-patches | 18:17 |
| jeblair | (that was rebase-only to fix merge conflicts) | 18:17 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Use libyaml if possible https://review.openstack.org/451113 | 18:36 |
| SpamapS | jeblair: ^^ much smaller patch, and actually fixes a bug in the original that had it using Dumper instead of SafeDumper. | 18:37 |
| SpamapS | may 'splode everything though.. I can't get the test suite to pass for me locally anymore. :-/ | 18:38 |
| mordred | SpamapS: splody-splode | 18:47 |
| SpamapS | yeah | 18:49 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Use libyaml if possible https://review.openstack.org/451113 | 18:49 |
| SpamapS | sadly, even with the yaml going 10% or so faster... I still get timeouts running tests on my local machine no matter what branch/patch it is. | 18:49 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_timer_sshkey https://review.openstack.org/450897 | 18:55 |
| SpamapS | mordred: when you get the finger protocol thing working, am I going to be able to predict the finger address? Like 'finger 450897@zuul.openstack.org' ? | 18:58 |
| SpamapS | cause that would be t3h bomb | 18:59 |
| mordred | SpamapS: yup | 19:00 |
| mordred | SpamapS: at least, that's the intent | 19:00 |
| mordred | SpamapS: I expect finger 450897@zuul.openstack.org will give you a list of job names that are running and their build id - and maybe thelauncher they're on (tbd) | 19:01 |
| mordred | SpamapS: so thatyou could then do finger {uuid}@zl01.openstack.org | 19:01 |
| SpamapS | mordred: multiplex them all prefixed by uuid? ;-) | 19:09 |
| mordred | SpamapS: that'll be TOTALLY readable | 19:12 |
| SpamapS | mordred: call that class PortsmouthSymphoniaMultiplexer | 19:14 |
| openstackgerrit | K Jonathan Harker proposed openstack-infra/zuul feature/zuulv3: Perform pre-launch merge checks https://review.openstack.org/446275 | 19:22 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_live_reconfiguration_del_project https://review.openstack.org/453303 | 19:24 |
| *** harlowja has joined #zuul | 19:24 | |
| SpamapS | by my count we're down to 12 skips not in dev | 19:30 |
| SpamapS | Not sure I'll know what to do with myself if we have no @skip's anymore. ;) | 19:30 |
| jesusaur | you could always write new test cases ;) | 19:31 |
| SpamapS | we're at around 10 - 15 minutes of test run time now... so we can pile a good 45 more minutes of test time on and still keep our sanity, right? | 19:32 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_timer_sshkey https://review.openstack.org/450897 | 19:33 |
| SpamapS | jesusaur: ^^ I'm hoping once yours pass, that one magically passes too ;) | 19:34 |
| SpamapS | likely still some tweaks to do tho | 19:34 |
| jeblair | i'm going to be unavailable next week. hopefully i can finish up the week without anyone being blocked on me. :) | 20:00 |
| jeblair | SpamapS, jesusaur, jlk, mordred, pabelanger, Shrews: fyi ^ | 20:01 |
| jlk | cheers | 20:01 |
| jlk | We'll happily merge everything in your absence. | 20:02 |
| * mordred enjoys merging patches without reading them first | 20:02 | |
| jlk | Question, should I be able to run tools/test-setup.sh repeatedly? | 20:03 |
| jeblair | jlk: i think so | 20:04 |
| pabelanger | jeblair: thanks for the heads up | 20:05 |
| jlk | I run it once, then if I run it immediately after, I get mysqladmin: connect to server at 'localhost' failed | 20:06 |
| jlk | I wonder if this is a hostname issue | 20:06 |
| jlk | pabelanger: did you have to do anything special on Fedora to get zuul tests to properly use mysql on the host? | 20:10 |
| jlk | how does the zuul tests determine which mysql host ot use? | 20:11 |
| pabelanger | jlk: nothing special, just worked | 20:12 |
| pabelanger | MySQLSchemaFixture is contains the URI info | 20:13 |
| pabelanger | could try using 127.0.0.1 | 20:13 |
| jlk | oh wtf. | 20:14 |
| *** hashar has joined #zuul | 20:27 | |
| * mordred guesses the anonymous user hasn't been deleted from your MySQL | 20:42 | |
| * mordred continues to be angry that people still create the damned thing | 20:42 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add hostname to TriggerEvent https://review.openstack.org/452348 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fully qualify project configuration names https://review.openstack.org/451970 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add source to project and remove unused tenant attrs https://review.openstack.org/451969 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass source to project instantiations https://review.openstack.org/451596 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add a project index to Tenant https://review.openstack.org/451597 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove unused Tenant.getRepo method https://review.openstack.org/451929 | 20:56 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Use new tenant project index for config references https://review.openstack.org/451928 | 20:56 |
| * jeblair hands mordred an anonymous pie | 20:56 | |
| jlk | Just ran into something. Pillow just released 4.1.0 a few hours ago, and now it requires libjpeg dev libraries to install. 4.0.0 did not. | 21:15 |
| jeblair | jlk: i guess we should add that to bindep.txt with [test] | 21:20 |
| jeblair | (i just noticed my zuul/.testrepository directory is 5.8G) | 21:20 |
| pabelanger | what is it using libjpeg for? | 21:30 |
| jeblair | pabelanger: diagrams in docs | 21:30 |
| pabelanger | I see | 21:30 |
| jlk | it seems to be a transitive dep, we don't directly list Pillow | 21:30 |
| jeblair | (though i don't think we actually use jpeg) | 21:30 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Replace config/project repos with trusted/untrusted projects https://review.openstack.org/453347 | 21:42 |
| jeblair | mordred: ^ | 21:42 |
| SpamapS | jeblair: good timing, I'm on vacation next Monday and Tuesday. | 22:03 |
| jeblair | it's a good week for it | 22:03 |
| *** hashar has quit IRC | 22:24 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove source from pipelines https://review.openstack.org/453362 | 22:50 |
| jeblair | jlk: i think the stack from 451102 to 453362 is getting close to being ready. that's the thing we talked about last week which makes sense of having projects in a multiple-connection world. i believe you were running into things in the github series which would benefit from this and so blocked on it. | 22:54 |
| jeblair | jlk: i think now might be a good time to start looking that over if you're so inclined. | 22:55 |
| jeblair | (i mean, the last two fail *all the tests* but they are also messy 'update all the tests' changes, so i'll probably be polishing them for a little bit) | 22:56 |
| jhesketh | Morning | 22:56 |
| jlk | jeblair: I'll start looking over them. | 22:57 |
| jlk | I'm strugging just to get the top of feature/zuulv3 to pass tests on my new VM :( | 22:58 |
| jlk | I may have to give up trying to do this on Fedora :( | 22:59 |
| jeblair | jlk: hrm, i'd hope it wouldn't be too different. :( | 23:02 |
| jlk | yeah I'm comparing now with xenial just to see if I can make it work | 23:03 |
| jlk | hrm, unless you just merged it in, you already have libjpeg-dev in bindep | 23:06 |
| mordred | jlk: we don't do any automatic local installation of things in bindep- only in the gate - so you may need to still run bindep to see what you'remissing and pipe that to dnf or something | 23:07 |
| jlk | mordred: right, yeah we aren't touching it at all. We'd gotten away with it in zuul 2.5 land | 23:08 |
| jlk | the Pillow release today broke us | 23:08 |
| mordred | nod | 23:08 |
| mordred | silly pillow | 23:08 |
| SpamapS | you're a silly pillow | 23:09 |
| jlk | Huh. So on Ubuntu, you can run test-setup.sh repeatedly | 23:14 |
| jlk | but not on Fedora | 23:14 |
| SpamapS | jlk: I believe Ubuntu removes the anonymous mysql user | 23:15 |
| SpamapS | could be why | 23:15 |
| jlk | mayhap. | 23:15 |
| jlk | but why would that prevent? | 23:15 |
| SpamapS | wonky assumptions and fallbacks most likely | 23:16 |
| SpamapS | what's your fail on fedora? | 23:16 |
| jlk | it's... hard to say | 23:16 |
| jlk | there's a bunch of skips, which I assume is because mysql can't get accessed. | 23:16 |
| jlk | and then there's just a generic fail message, without anything helpful. | 23:16 |
| jlk | I don't want to shave that yak too hard, so I'm seeing if I can get HEAD to pass on a xenial vm | 23:17 |
| SpamapS | jlk: try 'mysql -c "show grants"' anything? | 23:17 |
| jlk | I deleted the VM, we're short on cloud space ;) | 23:18 |
| jlk | there was a couple things there | 23:18 |
| SpamapS | aw derp | 23:18 |
| jlk | I can auth into mysql using the credentials the script created | 23:18 |
| SpamapS | right but there may be weirdness with non-credentialed access.. dunno | 23:18 |
| jlk | just not sure why a second run of the script fails, or if it has anything to do with tests failing | 23:18 |
| SpamapS | yeah I hear you. | 23:21 |
| SpamapS | I'm getting a lot of weird timeouts on my Xenial laptop FWIW | 23:21 |
| jlk | TIL having a VM that will consistently pass tests is a treasured resource :( | 23:21 |
| jlk | I lost mine when a cloud got rebuilt, so I'm trying to recreate it | 23:21 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove source from pipelines https://review.openstack.org/453362 | 23:34 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Replace config/project repos with trusted/untrusted projects https://review.openstack.org/453347 | 23:34 |
| jlk | Well damn. | 23:43 |
| jlk | essentially the same steps on Xenial result in passing tests :( | 23:43 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!