| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-26 | |||
| 20:53:31 | fungi | dansmith: mriedem: i've reenqueued and promoted 425620,7 now | |
| 20:53:46 | dansmith | fungi: and my change too, right? :D | |
| 20:53:49 | dansmith | no, I know. | |
| 20:54:29 | dansmith | I know | |
| 20:55:06 | dansmith | bauzas: looks like your bottom two might be merge-able soon | |
| 20:56:12 | bauzas | dansmith: I'm still puzzled by why grenade-multi is unhappy after our changes | |
| 20:57:47 | bauzas | http://logs.openstack.org/61/417961/31/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/cd57fce/logs/new/screen-n-sch.txt.gz?#_2017-01-26_20_06_58_560 tells me that node-[...]154 is not present | |
| 21:00:23 | cdent | bauzas, dansmith: I need to checkout, anything I should chase tomorrow morning? | |
| 21:00:54 | dansmith | cdent: as of now, I think we're just hoping to land the time-sensitive stuff before tomorrow | |
| 21:01:04 | dansmith | cdent: but someone can send a mail to the list if there's anything | |
| 21:01:39 | cdent | dansmith: I can move stuff around stick around now if there's something specific that I could help with? | |
| 21:01:40 | mriedem | i'm looking through vladikr's vlan metadata stuff again | |
| 21:01:54 | mriedem | after that it's jaypipes' resource class ironic stuff | |
| 21:02:03 | mriedem | cdent: i think we're mostly just watching the gate | |
| 21:02:04 | mriedem | and rechecking | |
| 21:02:05 | dansmith | cdent: unless you staring at the gate will make it go faster than me staring at the gate. otherwise, I'd go | |
| 21:02:07 | dansmith | heh | |
| 21:02:55 | cdent | my experience is that when I look at the gate it fails, so I'm going to go. but yeah, if there's utc morning stuff to do, leave another of those public handoff emails | |
| 21:03:09 | cdent | good luck, break a leg or whatever it takes | |
| 21:03:44 | bauzas | meh | |
| 21:04:06 | bauzas | the old compute looks good | |
| 21:04:31 | _gryf | do we need a saw | |
| 21:04:33 | _gryf | ? | |
| 21:04:33 | bauzas | but I still don't understand why we're not updating its hostState on the sheduler side | |
| 21:04:55 | _gryf | oh man. wrong channel | |
| 21:09:08 | bauzas | dansmith: the only possibilty I see with the grenade-multi issue is that we don't have the right service version | |
| 21:10:19 | bauzas | checking whether we are calling placement | |
| 21:13:29 | mriedem | +W on that bottom server groups test patch | |
| 21:14:36 | bauzas | erf, we don't follow-up passing the req-id to the placement API | |
| 21:14:45 | bauzas | we regenerate a new one | |
| 21:14:58 | bauzas | so good luck for guessing | |
| 21:15:11 | mriedem | bauzas: we want a reno on this right? https://review.openstack.org/#/c/425806/ | |
| 21:15:22 | mriedem | heads up - your computes won't start if you upgrade to ocata but don't have placement configured | |
| 21:15:33 | mriedem | bauzas: yeah cdent is working on that | |
| 21:15:35 | bauzas | mriedem: I did that in the top patch | |
| 21:15:46 | mriedem | bauzas: ok | |
| 21:15:47 | bauzas | mriedem: but I should have splitted the note | |
| 21:16:00 | mriedem | for shame | |
| 21:16:21 | bauzas | mriedem: that said, if we are unable to merge the top one in time, we can just add the note in a follow-up before RC1 | |
| 21:16:33 | bauzas | unless you strongly disagree | |
| 21:16:55 | mriedem | i think we're ok | |
| 21:16:58 | mriedem | we'll have a reno either way before rc1 | |
| 21:17:09 | bauzas | ok | |
| 21:18:01 | claudiub | mriedem: hi. just an fyi, both moshele and sfinucan gave their blessings on the Hyper-V PCI passthrough patch. :D https://review.openstack.org/#/c/420614/ | |
| 21:18:16 | mriedem | yeah i saw | |
| 21:18:33 | mriedem | it's in yon tab now | |
| 21:18:56 | mriedem | bauzas: fyi v | |
| 21:18:57 | mriedem | https://review.openstack.org/#/c/414230/ | |
| 21:19:16 | bauzas | dansmith: mriedem: okay, I think we're still calling the placement API even if we have an old node | |
| 21:19:20 | bauzas | proof here http://logs.openstack.org/61/417961/31/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/cd57fce/logs/apache/placement-api.txt.gz#_2017-01-26_20_06_58_531 | |
| 21:19:42 | bauzas | the timings are quite the same, vs. http://logs.openstack.org/61/417961/31/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/cd57fce/logs/new/screen-n-sch.txt.gz?#_2017-01-26_20_06_58_556 | |
| 21:20:29 | bauzas | dansmith: I think we need to instrument that and add more logs | |
| 21:20:44 | bauzas | dansmith: you okay with me testing that? | |
| 21:20:59 | dansmith | bauzas: do we have any other choice? so... sure :) | |
| 21:21:39 | mriedem | vladikr: are you sure we're getting an int back in the port details for vlan? | |
| 21:21:49 | mriedem | or could it be a string? | |
| 21:22:39 | bauzas | when I think I'm missing the semi-finals of the handball WC because of that... | |
| 21:24:10 | mriedem | WC = bathroom? | |
| 21:24:16 | mriedem | handball in the men's room? | |
| 21:24:17 | mriedem | kinky | |
| 21:24:52 | mriedem | bauzas: https://review.openstack.org/#/c/425822/ is merged, i'll update the novaclient 7.1.0 patch while you instrument things | |
| 21:27:17 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 21:28:25 | bauzas | mriedem: cool | |
| 21:28:45 | bauzas | mriedem: the logging is done, I'm just now waiting the results... | |
| 21:29:41 | mriedem | si capitan | |
| 21:30:45 | bauzas | France won and goes to final, one good thing at least | |
| 21:32:37 | dansmith | mriedem: I made dash show starred changes in blue, which makes it handy for following stuff through the gate | |
| 21:33:05 | mriedem | cool | |
| 21:38:52 | dansmith | I have been sitting in the same place since this morning, so I'm going to get up for a bit now that my change is in the gate again, and will be back in a bit | |
| 21:39:03 | dansmith | looks like bauzas' change is failing pep8 | |
| 21:39:11 | bauzas | fk | |
| 21:39:17 | bauzas | I edited inline | |
| 21:40:10 | bauzas | http://logs.openstack.org/61/417961/32/check/gate-nova-pep8-ubuntu-xenial/b7bc39a/console.html#_2017-01-26_21_33_30_781456 | |
| 21:40:15 | bauzas | o_O | |
| 21:40:35 | mriedem | bauzas: the wifi is so terrible here i can't even git review -d your novaclient release change | |
| 21:40:44 | mriedem | so i guess it's on you to update that | |
| 21:41:12 | bauzas | mriedem: roger | |
| 21:42:19 | bauzas | mriedem: FWIW, will use inline gerrit | |
| 21:45:01 | bauzas | mriedem: dansmith: do you guys think the pep8 issue would prevent us to see the logs correctly? | |
| 21:45:18 | dansmith | eh? | |
| 21:45:26 | bauzas | I don't give a shit to pep8 since I don't want to merge that like this, but I do care about whether the logs are good | |
| 21:45:39 | bauzas | dansmith: http://logs.openstack.org/61/417961/32/check/gate-nova-pep8-ubuntu-xenial/b7bc39a/console.html#_2017-01-26_21_33_30_781456 | |
| 21:45:40 | dansmith | so you want to let it run? that's fine | |
| 21:45:47 | bauzas | yeah | |
| 21:45:52 | dansmith | biab | |
| 21:46:10 | bauzas | I'll have to bail out in 1h15 so I prefer leave it running | |
| 22:07:26 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata https://review.openstack.org/405954 | |
| 22:07:26 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: Adding vlans field to Device tagging metadata https://review.openstack.org/405951 | |
| 22:07:44 | mriedem | vladikr: -1 on https://review.openstack.org/#/c/405954 because that's not handling that we are getting strings back from neutron for the vlan id in the port detials | |
| 22:07:46 | mriedem | *details | |
| 22:08:25 | vladikr | mriedem, just fixed it | |
| 22:08:30 | vladikr | ^^ | |
| 22:08:59 | mriedem | ah good timing :) | |
| 22:09:02 | vladikr | :) | |
| 22:10:35 | mriedem | jroll: do we have anything in CI that sets the node.resource_class field? | |
| 22:11:25 | edleafe | recheck finally worked on https://review.openstack.org/#/c/404472/ \o/ | |
| 22:13:35 | mriedem | edleafe: just wrapping up some comments in there | |
| 22:13:57 | edleafe | mriedem: okie dokie | |
| 22:13:57 | mriedem | edleafe: what sucks is we have nothing in CI to test this new flow http://logs.openstack.org/72/404472/21/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/d12f128/logs/screen-n-cpu.txt.gz#_2017-01-26_18_59_36_215 | |
| 22:14:00 | mriedem | we're still just doing the old thing | |