| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-13 | |||
| 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 | |
| 21:38:45 | mriedem | sean-k-mooney: i think https://review.openstack.org/#/c/410737/ is depending on the wrong patch | |
| 21:39:05 | mriedem | that's why gate-nova-requirements failed | |
| 21:39:43 | mriedem | i'll update it quick | |
| 21:40:19 | openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | |
| 21:40:20 | openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port https://review.openstack.org/410737 | |
| 21:51:22 | dtp | anyone need anything re cells v2? | |
| 21:51:40 | mriedem | dtp: for right now i don't think so | |
| 21:51:53 | dtp | ok | |
| 22:22:04 | melwitt | mriedem: trying to make sure I have this right in my head ... is the expectation during an upgrade to ocata that the appropriate cells v2 setup commands be run using the ocata code, before running the db sync? or is the expectation to run the setup using the newton code before starting the upgrade | |
| 22:24:25 | mriedem | melwitt: in grenade we run simple_cell_setup in newton before going to ocata: https://review.openstack.org/#/c/393441/2/projects/60_nova/from-newton/upgrade-nova | |
| 22:24:28 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Ignore me - ry py35 functional with a clean slate https://review.openstack.org/419968 | |
| 22:24:42 | mriedem | but simple_cell_setup is meant to be idempotent so you can run it as many times as needed | |
| 22:25:04 | mriedem | but you're asking because we don't have create_cell in newton | |
| 22:25:08 | melwitt | okay. I was wondering if it's also fine to run them in ocata, just before the db sync | |
| 22:25:11 | melwitt | yes, pretty much | |
| 22:25:34 | mriedem | well, for the nova_api db sync to work, you have to have created the cell mappings and cell0 | |
| 22:25:49 | mriedem | so (1) simple_cell_setup, (2) api-db sync | |
| 22:25:50 | melwitt | and also the simple_cell_setup has an enhancement in ocata to take the transport url from the conf, which is pretty important. I think it can be backported (I checked oslo.messaging stuff) | |
| 22:25:55 | mriedem | because (2) has the blocking schema migration | |
| 22:25:57 | melwitt | but I was thinking, do we need to backport all this stuff or no | |
| 22:26:10 | melwitt | right | |
| 22:26:20 | mriedem | melwitt: i remember you asking about backporting that one, i think if it was ok w/ the oslo.messaging version in newton i think we can backport it | |
| 22:26:23 | mriedem | if you haven't already done so | |
| 22:26:34 | openstackgerrit | Dan Smith proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | |
| 22:26:42 | mriedem | well look who it is | |
| 22:26:44 | dansmith | mriedem: melwitt ^ | |
| 22:26:46 | melwitt | yeah, I was just wondering if it's necessary. maybe not necessary but it would be nice to have the option to run them in newton? | |
| 22:27:33 | dansmith | after writing that, I almost think maybe we should do away with the simple command, or at least not stress its usage | |
| 22:27:40 | dansmith | once we have the create command | |