| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-07 | |||
| 16:03:58 | openstackgerrit | Merged openstack/nova master: Send request_id on neutron calls https://review.openstack.org/468356 | |
| 16:06:44 | openstackgerrit | Merged openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923 | |
| 16:13:15 | cfriesen_ | mriedem: regarding your live migration uri stuff...don't we have another issue too? we pass the unmodified "migrate_uri" to guest.migrate(), and that gets written to the 'migrate_uri' parameter that gets passed to self._domain.migrateToURI3() | |
| 16:14:21 | mriedem | cfriesen_: no | |
| 16:15:11 | mriedem | cfriesen_: you're talking about this https://github.com/openstack/nova/blob/9dde4fe0eacc01c135401da3313c493caa99a30c/nova/virt/libvirt/driver.py#L6256 | |
| 16:15:25 | mriedem | migrate_uri might be None https://github.com/openstack/nova/blob/9dde4fe0eacc01c135401da3313c493caa99a30c/nova/virt/libvirt/driver.py#L6210 | |
| 16:15:35 | mriedem | if live_migration_inbound_addr is not set | |
| 16:15:40 | mriedem | or you're tunneling | |
| 16:16:10 | mriedem | or, jupiter is aligned with saturn | |
| 16:16:24 | cfriesen_ | mriedem: right...but if we need to add usernames and such to the first parameter, would we ever need to add it to the second? | |
| 16:16:38 | mriedem | it would get added either way | |
| 16:16:50 | mriedem | because they both go through the _live_migration_uri method | |
| 16:17:07 | mriedem | ooops sorry, | |
| 16:17:11 | mriedem | there is a _migrate_uri method | |
| 16:17:19 | cfriesen_ | mriedem: exactly | |
| 16:17:19 | mriedem | which != _live_migration_uri | |
| 16:17:20 | mriedem | nice | |
| 16:17:53 | mriedem | https://github.com/openstack/nova/blob/9dde4fe0eacc01c135401da3313c493caa99a30c/nova/virt/libvirt/driver.py#L681 | |
| 16:17:59 | mriedem | "QEMU accept two schemes: tcp and rdma" | |
| 16:18:02 | mriedem | that's wrong, it also takes ssh | |
| 16:18:06 | cfriesen_ | I didn't see the patch adding CONF.libvirt.live_migration_scheme, but arguably it should have tweaked _migrate_uri as well I think | |
| 16:32:26 | melwitt | mriedem: on the config option move thing, it is part of the [compute] group but it's also in the [DEFAULT] group. it looks like all of the compute options are in both groups | |
| 16:35:22 | mriedem | melwitt: are you sure about that? | |
| 16:35:30 | mriedem | i think i sorted out that it wasn't in the [compute] group | |
| 16:35:37 | mriedem | the variables for the groups in nova/conf/compute.py were confusing me | |
| 16:35:58 | mriedem | if it were in both groups and we moved it out of one, w/o deprecating, that would be a bug | |
| 16:35:58 | melwitt | mriedem: I could be wrong, but I thought I saw they are added to both | |
| 16:36:30 | melwitt | hm, yeah | |
| 16:37:54 | melwitt | oh, 'compute_opts' vs 'compute_group_opts', I think I confused them together before | |
| 16:38:43 | mriedem | me too | |
| 16:40:43 | melwitt | phew | |
| 16:41:01 | mriedem | melwitt: while you're here https://review.openstack.org/#/c/446240/ | |
| 16:41:25 | melwitt | oh, thanks | |
| 16:44:34 | sfinucan | gabor_antal: Looks like https://review.openstack.org/#/c/451789 is failing now? | |
| 16:45:45 | sfinucan | mriedem: I did some work around libvirt live migration options, because what was there wasn't enough | |
| 16:45:50 | bauzas | oh, weird bug https://bugs.launchpad.net/nova/+bug/1696417 | |
| 16:45:50 | openstack | Launchpad bug 1696417 in OpenStack Compute (nova) "nova-manage db online_data_migrations can fail when upgrading to newton under certain conditions" [Undecided,New] | |
| 16:46:16 | sfinucan | mriedem, cfriesen_: tada https://review.openstack.org/#/c/456571/ | |
| 16:46:27 | sfinucan | (ignore the spurious build failure) | |
| 16:47:00 | gabor_antal | sfinucan: :O it should not imho but it looks like | |
| 16:47:24 | mriedem | sfinucan: cool that looks like what we need in devstack | |
| 16:47:26 | mriedem | the username one | |
| 16:48:29 | mriedem | sfinucan: i'll test that out with my devstack change | |
| 16:48:42 | sfinucan | (y) | |
| 16:48:51 | sfinucan | I'll reply to your ML post accordingly | |
| 16:50:42 | sfinucan | mriedem, melwitt: I'm knocking off for the night but if ye could take another (final??) look at the PCIWeigher stuff today, that'd be great https://goo.gl/qC5Rfy | |
| 16:51:43 | openstackgerrit | Baodong (Robert) Li proposed openstack/nova-specs master: Expose vlan trunking in metadata/configdrive https://review.openstack.org/471815 | |
| 16:51:46 | mriedem | gdi lumberg | |
| 16:51:56 | mriedem | he's great in veep btw | |
| 16:52:37 | mriedem | also, the beastmaster | |
| 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 | |