| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-26 | |||
| 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 | |
| 22:14:47 | edleafe | mriedem: yeah, the extra specs stuff for ironic flavors hasn't been done yet | |
| 22:15:36 | jroll | mriedem: I have some stuff in progress to do that thing | |
| 22:15:42 | bauzas | dansmith: around ? | |
| 22:18:22 | bauzas | dansmith: confirmed that the min service version is 16 http://logs.openstack.org/61/417961/32/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/f07f0e1/logs/new/screen-n-sch.txt.gz#_2017-01-26_22_02_47_735 | |
| 22:18:29 | bauzas | while it should 15 | |
| 22:18:31 | bauzas | ;( | |
| 22:18:33 | bauzas | mriedem: ^ | |
| 22:19:50 | bauzas | potentially the cache isn't working well | |
| 22:24:52 | mriedem | edleafe: comments posted | |
| 22:25:29 | mriedem | mostly simple things | |
| 22:25:31 | mriedem | and some missing tests | |
| 22:26:31 | mriedem | dansmith: you should be able to set a str to an IntegerField right? | |
| 22:27:39 | mriedem | or does coerce() just make that work | |
| 22:32:03 | openstackgerrit | Eric Brown proposed openstack/nova: Add RPC version aliases for Ocata https://review.openstack.org/425958 | |
| 22:34:11 | dansmith | bauzas: no, it shouldn't be the cache, unless we're down long enough to consider the other node down | |
| 22:34:18 | dansmith | and then start before it checks in | |
| 22:34:34 | dansmith | mriedem: I think it will coerce a string to an int yeah | |
| 22:34:46 | mriedem | blarg | |
| 22:34:49 | mriedem | ok lesson learned | |
| 22:35:08 | bauzas | dansmith: any thoughts on why then we have 16? | |
| 22:35:09 | mriedem | vladikr: +2 on your middle change too, even though a couple of the unit tests need tweaks | |
| 22:36:15 | dansmith | bauzas: how long is conductor stopped for? | |
| 22:37:35 | bauzas | dansmith: verifying the tempest test | |
| 22:37:46 | dansmith | service_down_time defaults to 60s so if it is >60s between conductor going down and scheduler coming back up, then it could be excluding the "down" service | |
| 22:39:25 | edleafe | mriedem: working on adding the missing tests | |
| 22:39:32 | bauzas | dansmith: the test doesn't assume stopping anything | |
| 22:39:42 | dansmith | bauzas: what test? | |
| 22:39:50 | bauzas | the tempest test | |
| 22:39:57 | bauzas | oh, you mean, upgrade? | |
| 22:40:02 | dansmith | during the upgrade yeah | |
| 22:40:17 | bauzas | so we try to boot on the old node | |
| 22:40:50 | bauzas | I'm asking the service version for computes | |
| 22:43:07 | dansmith | I'm not getting what you're saying | |
| 22:43:21 | bauzas | dansmith: I'm trying to understand the inner logic | |
| 22:44:01 | bauzas | I'm not getting why the scheduler could be excluding the old node if we never upgraded the latter | |
| 22:44:16 | bauzas | wrt the service down time you mentioned | |
| 22:46:34 | dansmith | because conductor was down, so the subnode couldn't refresh its record | |
| 22:49:29 | dansmith | bauzas: see my change to your grenade patch just now | |
| 22:50:52 | bauzas | dansmith: ah, I see | |
| 22:51:06 | bauzas | dansmith: we're restarting the scheduler too soon | |
| 22:51:17 | dansmith | right, that's what I'm thinking | |
| 22:51:19 | bauzas | without the compute having properly registered back | |
| 22:51:22 | dansmith | right | |
| 22:51:23 | bauzas | gotcha | |
| 22:51:35 | bauzas | okay, let's see | |
| 22:51:37 | dansmith | if it's more than 60s then the scheduler's min version calculus will exclude it | |
| 22:51:43 | bauzas | you gonna recheck ? | |
| 22:51:53 | bauzas | I mean the main change | |
| 22:52:15 | dansmith | jesus I have to do everything around here! | |
| 22:52:26 | bauzas | :p | |
| 22:52:41 | bauzas | well, I can write a gerrit comment :p | |
| 22:54:51 | bauzas | dansmith: it's pretty late now, and I have to pass my whole tomorrow on train | |
| 22:55:02 | bauzas | dansmith: I'll have to handover :( | |
| 22:55:02 | dansmith | ...oh | |
| 22:55:09 | dansmith | hmm, okay | |
| 22:55:41 | bauzas | mriedem: thoughts on me officially asking for a FFE? | |
| 22:56:15 | mriedem | granted | |
| 22:57:29 | bauzas | dansmith: FWIW, do you think I should make a note in the reno file that operators have to make sure they leave a large enough window for having all the computes registered back or nope? | |
| 22:57:59 | dansmith | bauzas: well, this is already the case for auto rpc mode, but sure | |
| 22:58:08 | bauzas | right, my point | |
| 22:58:19 | bauzas | it's not specific to the scheduler | |
| 22:58:36 | bauzas | but I just want to make sure it's written somewhere | |
| 22:59:23 | bauzas | either way, I'll have to bail out, but since I'm gonna pledge for a FFE, I will follow-up tomorrow in the train (with limited internet access, but mriedem proved it works) | |
| 23:00:36 | mriedem | heh it is possible | |
| 23:00:40 | mriedem | despite wanting to gouge your eyes out | |
| 23:02:08 | mriedem | crap do we have a meeting right now or was that htis morning? | |
| 23:02:26 | bauzas | I did this morning | |
| 23:02:41 | bauzas | and it should have been 1 hour before if odd week | |
| 23:02:51 | bauzas | a very short one | |
| 23:03:05 | mriedem | ok you ran it, cool, thanks | |
| 23:03:19 | mriedem | my calendar is in my work email which i don't have up | |
| 23:03:29 | mriedem | i'll work on vacation, just not for my employer | |
| 23:03:37 | bauzas | :) | |
| 23:03:53 | bauzas | hope the weather is good at least | |
| 23:05:14 | openstackgerrit | Matt Riedemann proposed openstack/nova: Adding vlans field to Device tagging metadata https://review.openstack.org/405951 | |
| 23:05:25 | mriedem | cloudy today | |
| 23:05:30 | mriedem | vladikr: ^ updated the reno | |
| 23:05:34 | mriedem | to mention it's libvirt-only right now | |
| 23:06:49 | mriedem | these are ready to go https://review.openstack.org/#/q/topic:bp/sriov-pf-passthrough-neutron-port-vlan+status:open | |
| 23:06:53 | mriedem | +2s on all | |
| 23:07:35 | bauzas | mriedem: you have enough bandwidth for git rev'ing ocata-3 ? | |
| 23:07:45 | mriedem | bauzas: yeah | |
| 23:07:52 | mriedem | there is always the 'business center' | |
| 23:08:07 | bauzas | or do you want me to do a WIP and you just update it with the gerrit UI? | |
| 23:08:14 | mriedem | i've got it | |
| 23:08:18 | bauzas | okay | |
| 23:08:48 | bauzas | mriedem: dansmith: thanks for helping me on that busy day | |
| 23:09:11 | mriedem | no problema | |
| 23:09:16 | mriedem | er, de nada | |
| 23:11:40 | mriedem | 319379 is 38 in the gate? | |
| 23:11:41 | mriedem | crap | |
| 23:12:11 | mriedem | dansmith: so i guess i'm going to just be tagging o-3 without the cells v2 change, but don't think that's a major issue | |
| 23:12:32 | dansmith | mriedem: are you serious? | |
| 23:12:50 | mriedem | umm i guess not? | |
| 23:12:51 | mriedem | what breaks | |
| 23:13:02 | dansmith | my heart | |
| 23:13:04 | mriedem | ha | |
| 23:13:37 | dansmith | and the plan is for FFE? | |