| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-13 | |||
| 18:33:37 | mriedem | jroll: umm, maybe | |
| 18:34:14 | mriedem | jroll: there was also the inject nmi patch but i think that was waiting on some dependencies too | |
| 18:34:42 | mriedem | cfriesen: i usually default to think that anything libvirt/qemu does is async | |
| 18:34:45 | jroll | mriedem: yeah, that's not even done in ironic, I don't expect it this cycle | |
| 18:34:49 | mriedem | i think most things are done via an async job | |
| 18:35:01 | mriedem | cfriesen: kashyap or mdbooth or sahid are the people i'd ask | |
| 18:35:06 | mriedem | jroll: ok | |
| 18:36:38 | jroll | mriedem: I'll do the tooz thing now so you can see how simple it is. it passed tests with flying colors about a month ago with some weird depends-ons | |
| 18:37:09 | mriedem | ok, just got back from lunch and still sore from the cells v2 beating i took this morning | |
| 18:37:48 | jroll | oh, easiest merge conflict ever | |
| 18:37:48 | openstackgerrit | Jim Rollenhagen proposed openstack/nova: Move to tooz hash ring implementation https://review.openstack.org/399022 | |
| 18:37:52 | jroll | there you are if curious | |
| 18:38:56 | openstackgerrit | Jim Rollenhagen proposed openstack/nova: Move to tooz hash ring implementation https://review.openstack.org/399022 | |
| 18:39:22 | jroll | much better with the right version | |
| 18:43:31 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 18:51:20 | cfriesen | mriedem: thanks...I did some experiments and it looks like it's actually sync...if I send a SIGSTOP to qemu or libvirtd the nova call to self._domain.resume() hangs until I send a SIGCONT | |
| 18:53:26 | openstackgerrit | Dan Peschman proposed openstack/nova: nova-manage cell_v2 map_cell0 exit 0 https://review.openstack.org/420132 | |
| 18:54:33 | dtp | mriedem ^ going to lunch | |
| 18:54:44 | harlowja | dtp thx for helping here :) | |
| 18:55:00 | dtp | why you're welcome harlowja! :) | |
| 18:55:05 | harlowja | :-P | |
| 19:01:12 | sean-k-mooney | mriedem: i have resolved most of the erros in francesco's patches locally but one of his changes need the fastpath os-vif plugin to work which is incorrect http://logs.openstack.org/61/385061/7/check/gate-nova-python27-db-ubuntu-xenial/336f68e/console.html#_2017-01-12_23_22_43_325595 | |
| 19:01:50 | sean-k-mooney | mriedem: im going to invert the order of his patches so https://review.openstack.org/#/c/410737/ can merge | |
| 19:02:07 | sean-k-mooney | or at least be unblocked by the gate | |
| 19:05:40 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 19:11:12 | mriedem | sean-k-mooney: sounds good to me | |
| 19:19:17 | mriedem | GET /jaypipes | |
| 19:19:50 | openstackgerrit | sean mooney proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port https://review.openstack.org/410737 | |
| 19:19:50 | openstackgerrit | sean mooney proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | |
| 19:20:10 | openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: workaround findmnt behaviour change https://review.openstack.org/420146 | |
| 19:25:06 | openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: workaround findmnt behaviour change https://review.openstack.org/420146 | |
| 19:35:08 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 19:35:17 | mriedem | jroll: is there any way for you to tell if this job failed for a known ironic timeout http://logs.openstack.org/73/398473/22/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/3dbaaef/ ? | |
| 19:35:28 | openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | |
| 19:35:36 | mriedem | jroll: b/c it failed on https://review.openstack.org/#/c/398473/ which is moving a bunch of parts around in the compute resource tracker | |
| 19:36:42 | jroll | mriedem: so, this is pretty typical http://logs.openstack.org/73/398473/22/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/3dbaaef/console.html#_2017-01-13_16_02_01_111488 | |
| 19:36:51 | jroll | but could also be because pxe booting got broken somehow or whatever | |
| 19:37:03 | mriedem | it passed on the patch before and after that one so i assume it's not due to the actual change being tested | |
| 19:37:09 | jroll | wtf empty console logs | |
| 19:37:24 | jroll | yeah, probably not then | |
| 19:38:17 | jroll | mriedem: sometimes node* here is indicative (you'll see it cut off during boot or whatever) but something went wrong and they're empty http://logs.openstack.org/73/398473/22/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/3dbaaef/logs/ironic-bm-logs/ | |
| 19:38:20 | mriedem | cool, just rechecked it | |
| 19:38:26 | jroll | cool | |
| 19:38:33 | lyarwood | mriedem: is the placement job voting in the stable/newton gate btw? | |
| 19:38:36 | jroll | we've been seeing timeouts creep back up lately, unclear why | |
| 19:38:42 | mriedem | lyarwood: no | |
| 19:38:59 | lyarwood | mriedem: I just ack'd the two reviews from yesterday, they both failed the placement job a few days ago | |
| 19:39:28 | mriedem | lyarwood: the placement job was failing on newton b/c of an infra issue with the job definition, it was trying to call a post test hook which didn't exist | |
| 19:39:32 | mriedem | i fixed that on wednesday | |
| 19:40:40 | lyarwood | argh, they failed on Thursday | |
| 19:40:49 | mriedem | lyarwood: maybe i fixed it thursday :) | |
| 19:40:50 | mriedem | http://logs.openstack.org/17/419217/1/check/gate-tempest-dsvm-neutron-placement-full-ubuntu-xenial-nv/086c49e/logs/devstack-gate-post_test_hook.txt.gz | |
| 19:40:53 | mriedem | but ^ was the issue | |
| 19:41:09 | mriedem | https://review.openstack.org/#/c/419243/ | |
| 19:41:11 | mriedem | yeah it was yesterday | |
| 19:41:11 | lyarwood | ah cool | |
| 19:41:13 | mriedem | been a long week | |
| 19:41:21 | lyarwood | yeah, tell me about it | |
| 19:41:46 | mriedem | the other series in newton for placement that i have here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:bug/1651678 | |
| 19:42:00 | mriedem | one of the changes towards the end is failing functional tests for some reason which i need to investigate | |
| 19:42:03 | mriedem | so you can hold off on that series | |
| 19:42:18 | lyarwood | don't worry I was going to :) | |
| 19:42:23 | mriedem | heh | |
| 19:42:50 | mriedem | i mean at quarter to 8 on a friday, reviewing that series is about as much fun as you should have | |
| 19:44:40 | lyarwood | haha aye, unfortunately my presence is required downstream to fight another fire, otherwise I'd be all over that at this time on Friday ;) | |
| 19:51:06 | dims | mriedem : so got one +2 from sdague and a review from cdent in the morning. any chance for getting it merged today? https://review.openstack.org/#/c/419476/ | |
| 19:51:37 | dims | mriedem : thanks in advance! | |
| 19:51:38 | mriedem | not from me right now, i have other fires | |
| 19:52:25 | dims | :) no worries. just did not want to keep asking | |
| 19:52:49 | dims | melwitt : how about you? :) | |
| 19:53:31 | melwitt | maybe later, working on something related to cells v2 fire right now | |
| 19:54:23 | dims | ack thanks melwitt | |
| 20:02:12 | sdague | timing on that sucks, mostly with dansmith heading out to the wood with only a ham radio | |
| 20:02:36 | mriedem | well i think he's around a bit yet | |
| 20:02:44 | mriedem | but yes | |
| 20:03:21 | melwitt | he said he'll be around this afternoon, we'll have him on monday too. but starting tuesday he's out next week | |
| 20:05:27 | mriedem | ok i've replied to the ML thread | |
| 20:05:31 | mriedem | with my brain dump | |
| 20:06:18 | dims | hmm, looks like cdent is done for the day as well. seeing some gabbit related issues | |
| 20:06:54 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 20:08:05 | mriedem | sdague: we also need https://review.openstack.org/#/c/392569/ updaed | |
| 20:08:07 | mriedem | *updated | |
| 20:09:11 | mriedem | although i haven't gone through that whole patch yet | |
| 20:09:28 | mriedem | and the things i pointed out already probably aren't the end of the world | |
| 20:10:15 | melwitt | mriedem: I'm fixing up the create_cell patch right now, just fyi | |
| 20:11:01 | mriedem | melwitt: yup, i know (already) :) | |
| 20:12:59 | EmilienM | mriedem: thanks for your help, very appreciated | |
| 20:13:09 | mriedem | yw | |
| 20:41:00 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 20:46:41 | openstackgerrit | Slawek Kaplonski proposed openstack/nova: Set sysinfo_serial="none" in LibvirtDriverTestCase https://review.openstack.org/420176 | |
| 20:48:19 | openstackgerrit | Matt Riedemann proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | |
| 20:48:19 | openstackgerrit | Matt Riedemann proposed openstack/nova: Make metadata server know about cell mappings https://review.openstack.org/417962 | |
| 20:48:20 | openstackgerrit | Matt Riedemann proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | |
| 20:48:23 | openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | |
| 21:04:00 | mriedem | sdague: i'm cool with this pretty simple patch https://review.openstack.org/#/c/417962/ | |
| 21:08:39 | openstackgerrit | melanie witt proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | |
| 21:14:57 | openstackgerrit | sean mooney proposed openstack/os-vif: [DO NOT MERGE] this a test patch for os-vif gate. https://review.openstack.org/420179 | |
| 21:17:07 | openstackgerrit | Diana Clarke proposed openstack/nova: copy pasta error https://review.openstack.org/420180 | |
| 21:17:22 | diana_clarke | ^ easy review for Friday brains ;) | |
| 21:34:02 | openstackgerrit | Diana Clarke proposed openstack/nova: DNM: check experimental queue https://review.openstack.org/420186 | |