| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-07 | |||
| 16:53:05 | mriedem | oh wait maybe that's not him | |
| 16:53:55 | melwitt | sfinucan wants us to come in on saturday | |
| 16:54:11 | gbarros | mriedem, could you clarify workflow for me? looks like we have all the + on the vif thing, now jenkins runs a set of tests and then merges automatically after they pass? | |
| 16:54:12 | mriedem | sfinucan doesn't work past 5 | |
| 16:54:20 | mriedem | gbarros: yes | |
| 16:54:30 | gbarros | ty | |
| 16:54:53 | openstackgerrit | Merged openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore https://review.openstack.org/459235 | |
| 16:55:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test removing live_migration_uri from devstack https://review.openstack.org/471019 | |
| 16:56:26 | melwitt | cells meeting in 4 minutes | |
| 16:57:56 | dtp | melwitt dansmith - i have a conflicting meeting this morning, so i'm going to miss cells v2, but i'll be around after | |
| 16:58:13 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 16:58:14 | melwitt | k | |
| 16:58:16 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 16:58:23 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 16:58:40 | dansmith | dtp: me too maybe we should just cancel | |
| 16:58:41 | dansmith | melwitt: ^ | |
| 16:58:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 16:58:54 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 16:59:27 | melwitt | hm, yeah, if it's only going to be me and mriedem, maybe we cancel this week. what do you think mriedem? | |
| 16:59:39 | mriedem | fine | |
| 16:59:52 | melwitt | cool | |
| 17:06:30 | mriedem | sfinucan: confused, why does the PCIWeigher depend on the NUMATopologyFilter rather than the PciPassthroughFilter? | |
| 17:08:12 | mriedem | i guess the numa filter also looks at pci requests | |
| 17:08:52 | openstackgerrit | Merged openstack/nova master: use plain routes list for os-virtual-interfaces https://review.openstack.org/462173 | |
| 17:11:43 | mriedem | cfriesen_: ^ would you agree on https://review.openstack.org/#/c/379524/22/doc/source/filter_scheduler.rst that if we mention the numa filter we should also mention the pci passthrough filter? | |
| 17:21:27 | openstackgerrit | Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 17:22:13 | cfriesen_ | mriedem: checking | |
| 17:34:09 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | |
| 17:36:51 | openstackgerrit | Merged openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818 | |
| 17:38:58 | openstackgerrit | Merged openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 17:48:21 | openstackgerrit | Baodong (Robert) Li proposed openstack/nova master: [WIP] Expose vlan trunking in metadata/configdrive https://review.openstack.org/471874 | |
| 17:49:16 | rm_work | Anyone here that could help me with getting nova-lxd up and working in our gate? (Octavia project) | |
| 17:49:28 | rm_work | http://logs.openstack.org/37/453237/9/check/gate-octavia-v1-dsvm-scenario-lxd-ubuntu-xenial-nv/83d816f/logs/screen-n-cpu.txt.gz#_Jun_07_08_40_43_066936 | |
| 17:49:49 | rm_work | Trying to figure out what we need to set up, I'm a complete nova-lxd nub | |
| 17:50:51 | rm_work | looks like it's set to the right compute driver, so we just need to get the LXD service installed properly in the OS maybe? http://logs.openstack.org/37/453237/9/check/gate-octavia-v1-dsvm-scenario-lxd-ubuntu-xenial-nv/83d816f/logs/etc/nova/nova.conf.txt.gz | |
| 17:51:48 | openstackgerrit | melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635 | |
| 17:52:23 | openstackgerrit | Baodong (Robert) Li proposed openstack/nova master: [WIP] Expose vlan trunking in metadata/configdrive https://review.openstack.org/471874 | |
| 18:03:06 | openstackgerrit | melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635 | |
| 18:04:32 | mriedem | rm_work: that's likely another channel somewhere | |
| 18:04:37 | rm_work | hmmmm k | |
| 18:04:39 | mriedem | i'd say zigo but i'm not sure he's working on that anymore | |
| 18:04:54 | rm_work | maybe zigo at least knows what channel it is then? :P | |
| 18:05:00 | mriedem | that's my hope | |
| 18:05:11 | mriedem | "If you prefer live discussions, some of us also hang out in #lxcontainers on irc.freenode.net." | |
| 18:05:14 | mriedem | https://github.com/openstack/nova-lxd | |
| 18:05:43 | rm_work | ah, missed that | |
| 18:05:44 | rm_work | thanks | |
| 18:30:58 | elady | looking for soft affinity filter code | |
| 18:36:08 | mriedem | elady: nova/scheduler/filters | |
| 18:40:06 | elady | The affinity_filter.py has only affinity + anti-affinity the soft-affinity and soft-anti-affinity are not there and missing in code | |
| 18:41:47 | elady | meriedm : The affinity_filter.py has only affinity + anti-affinity the soft-affinity and soft-anti-affinity are not there and missing in code | |
| 18:53:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | |
| 18:53:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Use uuid for id in Services API https://review.openstack.org/464280 | |
| 18:54:48 | mriedem | elady: the soft affinity stuff is in the weigher https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py | |
| 18:55:19 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/soft-affinity-for-server-group.html | |
| 18:56:34 | openstackgerrit | Merged openstack/nova master: libvirt: update LibvirtConfigGuestDeviceAddress to provide XML https://review.openstack.org/459737 | |
| 19:02:21 | elady | mriedem: I created server group with soft-affinity policy and set soft-affinity weight to 1.0 and still not working as expected | |
| 19:08:00 | dansmith | jaypipes: melwitt see this on that bottom patch? http://logs.openstack.org/45/470645/6/check/gate-nova-tox-functional-py35-ubuntu-xenial/c97c7cb/testr_results.html.gz | |
| 19:09:54 | dansmith | I guess that can't be related since the bottom patch is just db stuff | |
| 19:10:25 | melwitt | dansmith: yeah. I've seen that before several times in the past. some "authentication error" that's transient in functional tests. I don't get that | |
| 19:12:36 | dansmith | ack | |
| 19:13:33 | melwitt | hm, though I just noticed this: | |
| 19:13:35 | melwitt | KeyError: 'allocations' | |
| 19:13:35 | melwitt | return resp.json()['allocations'] | |
| 19:13:35 | melwitt | nova/scheduler/client/report.py", line 914, in get_allocations_for_resource_provider | |
| 19:14:33 | melwitt | it's not part of the patch in question but it's something that seems wrong? | |
| 19:14:43 | dansmith | that was my point | |
| 19:14:47 | dansmith | but this patch is only db stuff | |
| 19:14:59 | melwitt | yeah | |
| 19:15:22 | dansmith | recheck-and-chill, I say | |
| 19:15:57 | melwitt | hah | |
| 19:16:58 | openstackgerrit | Merged openstack/nova master: libvirt: set full description of the controller used by disk https://review.openstack.org/459738 | |
| 19:17:42 | openstackgerrit | Merged openstack/nova master: libvirt: rearange how scsi controller is defined https://review.openstack.org/459739 | |
| 19:19:18 | mriedem | dansmith: do you remember why we didn't do a ComputeHostNotUnique thing here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4568 like we did for services? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4371 | |
| 19:19:49 | mriedem | dansmith: i've seen that functional test fail before | |
| 19:19:51 | mriedem | it's random | |
| 19:19:53 | dansmith | mriedem: because theres no delete | |
| 19:20:02 | mriedem | dansmith: but there is show | |
| 19:20:04 | dansmith | mriedem: you can delete a service by id, and you might delete the wrong one | |
| 19:20:06 | dansmith | sure | |
| 19:20:36 | dansmith | you were going to fix it too, but just not worry about changing the behavior if you get the wrong compute node I think, because it's not destructive | |
| 19:20:44 | dansmith | fix it == make it use uuids | |
| 19:20:56 | mriedem | yeah that's what i'm working on now | |
| 19:21:14 | dansmith | doesn't hurt to make it behave the same if you want, it's just not critical I think | |
| 19:21:33 | mriedem | yeah it's definitely not as bad as delete | |
| 19:25:24 | openstackgerrit | Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 19:36:23 | mriedem | uuuuuuuuuuuuuuggggggggggggggghhhhhhhhhhhhhhhh | |
| 19:36:27 | mriedem | so, | |
| 19:36:41 | mriedem | returning compute nodes from the api with uuids means we're also going to need to change the paging links | |
| 19:36:54 | mriedem | the paging stuff plumbed all the way down to the database is using id | |
| 19:37:51 | dansmith | gross | |
| 19:37:54 | mriedem | dansmith: so if the marker in https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L402 can be an id or uuid, i assume that means i'll need a new remotable method? | |
| 19:37:57 | dansmith | same with services? | |
| 19:38:02 | mriedem | we don't have paging on services | |
| 19:38:04 | mriedem | thank god | |
| 19:38:36 | dansmith | mriedem: well, yeah, you should have a new remotable | |
| 19:38:40 | mriedem | also, | |
| 19:38:52 | mriedem | this likely means we aren't handling paging correctly right now with multiple cells | |
| 19:38:56 | dansmith | because otherwise we have no way to know that the other end doesn't support it | |
| 19:38:57 | dansmith | also, | |