| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-31 | |||
| 15:55:22 | dansmith | because of all the switchin' during service setup | |
| 15:55:25 | dansmith | api thinks api is old | |
| 15:55:41 | dansmith | which is why we have fixtures.AllServicesCurrent() | |
| 15:56:13 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2386 ? | |
| 15:56:27 | dansmith | mriedem: yar | |
| 15:56:28 | mriedem | is service_version 0 or soething? | |
| 15:56:30 | mriedem | something | |
| 15:56:43 | mriedem | this reminds me of something i've been meaning to talk to you about, | |
| 15:56:46 | dansmith | it is in the db that is getting consulted | |
| 15:57:20 | mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L437 | |
| 15:57:36 | dansmith | HAH | |
| 15:57:39 | mriedem | in the api we check the minimum compute service version for things like get-me-a-network and device tags | |
| 15:57:45 | mriedem | that's not going to work anymore with multicell | |
| 15:57:55 | kashyap | mdbooth: One last on that -- in your scenario of speeding up a remote boot while provisioning, it is still a single QEMU process that is handling COR, and live commit, isn't it... | |
| 15:57:59 | mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L583 | |
| 15:58:09 | dansmith | mriedem: well (a) we need to remove those now anyway, but (b) there's an all_cells worker to get that | |
| 15:58:15 | mdbooth | kashyap: Not if you've got 2 instances using the same image backing file | |
| 15:58:29 | kashyap | mdbooth: Ah, right! | |
| 15:58:30 | mdbooth | Both before the backing file download has finished | |
| 15:58:46 | kashyap | mdbooth: But yes, as it stands, upstream doesn't support multiple writers | |
| 15:59:06 | mriedem | dansmith: yeah https://github.com/openstack/nova/blob/master/nova/cmd/api_os_compute.py#L43 | |
| 15:59:12 | kashyap | Probably you want to write an RFE to qemu-{devel, block} lists :-) | |
| 15:59:14 | mdbooth | COR == implicitly writeable backing file | |
| 15:59:25 | mriedem | device tagging was newton too right? | |
| 15:59:28 | mriedem | yeah, i know it wasn't ocat | |
| 15:59:29 | mriedem | *ocata | |
| 15:59:45 | dansmith | mriedem: well, it was fixed in ocata api, but yeah | |
| 15:59:56 | mriedem | however | |
| 16:00:03 | mriedem | we have something else using that now in nova.compute.api | |
| 16:00:14 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1404 | |
| 16:00:17 | mriedem | ^ was added in pike | |
| 16:00:34 | dansmith | ack | |
| 16:00:46 | mriedem | we could remove the checks for get me a network and device tags though probably | |
| 16:01:34 | mriedem | i can't find your nova-status patch | |
| 16:01:41 | mriedem | ah https://review.openstack.org/#/c/442787/ | |
| 16:03:25 | openstackgerrit | Dan Smith proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | |
| 16:09:21 | mriedem | dansmith: that looks much better | |
| 16:09:41 | dansmith | I'm thrilled | |
| 16:11:45 | leakypipes | _gryf: lol at Michal's April Fool's spec :) | |
| 16:12:27 | leakypipes | dulek: :) | |
| 16:33:53 | ildikov | mriedem: lyarwood: would the Cinder-Nova meeting work for you an hour earlier (1600 UTC) on Thursdays? | |
| 16:33:55 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_password.py https://review.openstack.org/452259 | |
| 16:35:55 | lyarwood | ildikov: yes | |
| 16:36:53 | ildikov | lyarwood: cool, tnx | |
| 16:43:26 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: [WIP] Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | |
| 17:05:06 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_external_events.py https://review.openstack.org/452263 | |
| 17:26:02 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in server_diagnostics.py https://review.openstack.org/452267 | |
| 17:26:43 | JayF | mriedem: https://review.openstack.org/#/c/416487/ mind dropping your procedural -2 from last cycle? | |
| 17:26:59 | JayF | mriedem: we're tying all the patches together now to get it all tested up in tempest and ready to merge (in ironic, then nova) | |
| 17:49:58 | jroll | oh we should approve that blueprint | |
| 17:51:35 | openstackgerrit | Mario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | |
| 17:52:27 | JayF | mriedem: \o/ thanks | |
| 17:53:02 | mriedem | yw | |
| 17:53:17 | mriedem | ildikov: works for me | |
| 17:55:03 | sdague | mriedem: so... clarkb has an interesting though on the libvirt malloc crashes, which is that we start doing UCA by default | |
| 17:55:13 | sdague | which would mean jumping up to libvirt 2.5 in master | |
| 17:55:36 | sdague | it has a lot of other implications, he's going to write the whole thing up to share on the ML | |
| 17:55:46 | mriedem | ok | |
| 17:55:50 | sdague | but, I wanted to get that seed out there for everyone to ponder | |
| 17:56:18 | sdague | it would mean we'd start testing some of the pathways that aren't otherwise tested | |
| 17:56:35 | mriedem | well, some of those are things like numa/pci right? | |
| 17:56:45 | mriedem | which i don't think we'd be testing regardless | |
| 17:57:24 | clarkb | I think there are newer live migration things too | |
| 17:57:36 | clarkb | at least my tests tripped over that with bad wheel builds against old libvirt | |
| 17:58:03 | sdague | yeh, postcopy | |
| 17:58:23 | sdague | and virtlogd | |
| 17:58:39 | mriedem | what are we right now? 1.3.1? | |
| 17:58:43 | sdague | yep | |
| 17:59:15 | sdague | and perf_events? | |
| 17:59:33 | mriedem | yeah, so the only thing of those that would get tested by tempest are probably postcopy | |
| 17:59:42 | mriedem | unless virtlogd ties in with any console-related tests | |
| 17:59:45 | sdague | that seems like everything with higher versions bring in | |
| 18:00:35 | mriedem | yeah the perf events stuff won't be tested by tempest directly | |
| 18:00:36 | sdague | yeh, I think there might be console tests there | |
| 18:00:52 | sdague | anyway, a thing to ponder, details comming. | |
| 18:02:19 | mriedem | https://www.youtube.com/watch?v=09kuOAr_wkc | |
| 18:02:24 | mriedem | i'll ponder it on the tree of woe | |
| 18:02:58 | openstackgerrit | David Moreau Simard proposed openstack/nova master: Be honest with users when no valid host was found https://review.openstack.org/452279 | |
| 18:03:12 | sdague | heh | |
| 18:11:04 | leakypipes | sdague, sfinucan: another easy one: https://review.openstack.org/#/c/447401/5 | |
| 18:11:39 | sdague | bam | |
| 18:11:53 | leakypipes | heh | |
| 18:20:23 | mriedem | i need another specs core on https://review.openstack.org/#/c/441692/ | |
| 18:20:35 | mriedem | i think i addressed all of johnthetubaguy's comments | |
| 18:20:43 | mriedem | i'm not sure if there is something i need to update in there, so i haven't | |
| 18:21:02 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | |
| 18:24:26 | sdague | mriedem: have you tried to POC any of that yet? | |
| 18:24:31 | mriedem | sdague: no | |
| 18:24:49 | mriedem | i started by just trying to get devstack setup with searchlight in a ci job | |
| 18:25:17 | mriedem | https://review.openstack.org/#/c/441696/ | |
| 18:25:18 | sdague | how did that go? | |
| 18:25:21 | mriedem | not well | |
| 18:25:35 | mriedem | i didn't spend much time on it | |
| 18:25:45 | mriedem | i was hoping the searchlight devstack plugin would configure nova, but it doesn't | |
| 18:26:10 | mriedem | and now the logs are gone so i don't remember what was failing, i'll recheck it | |
| 18:26:26 | mriedem | also, searchlight doesn't have a dsvm job, so they didn't have something i could copy really | |
| 18:26:28 | sdague | so, it feels like one of those things were could define a direction here as we're going to try, but there should be a bit more POC before we commit (and especially punt on doing any of this in nova) | |
| 18:26:34 | sdague | mriedem: ah, bummer | |
| 18:26:59 | mriedem | sdague: the spec is about the general plan | |
| 18:27:12 | mriedem | just like the original resource providers spec was about the general plan, and changed a lot over time | |