| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-08 | |||
| 07:35:21 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | |
| 07:35:22 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | |
| 07:35:23 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | |
| 07:35:24 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | |
| 07:35:26 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | |
| 07:35:26 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | |
| 07:35:27 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487 | |
| 07:58:14 | kashyap | dansmith: Sorry, had to be AFK a bit early last evening. Just catching up with the scroll | |
| 08:02:14 | kashyap | dansmith: melwitt: Okido, let's go with the exhibit (a) which is: "deprecate in Rocky and hard-fail in Stein" | |
| 08:29:48 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP/POC] Updating queued_for_delete from instance_destroy() https://review.openstack.org/566813 | |
| 08:45:58 | kashyap | bauzas: When you are about, can you merge this simple Ocata backport? Already has a +2 from Sean McGinnis: https://review.openstack.org/#/c/565672/ | |
| 08:46:09 | kashyap | bauzas: (Pinging you, as you ACKed the other branch backports) | |
| 08:46:21 | kashyap | Or even lyarwood ^ | |
| 08:46:38 | kashyap | That's the outstanding backport for that bug. | |
| 08:56:57 | lyarwood | kashyap: looking | |
| 08:57:46 | lyarwood | kashyap: done | |
| 08:57:57 | kashyap | lyarwood: Splendid! | |
| 08:58:01 | kashyap | Thank you | |
| 09:13:32 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212 | |
| 09:16:35 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | |
| 09:16:36 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | |
| 09:16:37 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | |
| 09:16:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | |
| 09:16:39 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | |
| 09:16:40 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | |
| 09:16:41 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487 | |
| 09:23:34 | openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | |
| 09:44:26 | jmccarthy | mriedem: jgwentworth: Thanks for your help :) I tested the review, and in my setup it fixed the issue - https://review.openstack.org/#/c/566367 | |
| 10:17:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add tests for sharing rps in test_common_rc() https://review.openstack.org/566842 | |
| 10:19:25 | tetsuro | efried, jaypipes, bauzas: ^ found a new bug in sharing case. might be related to https://review.openstack.org/#/c/560459/3? | |
| 10:40:50 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565672 | |
| 10:50:14 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | |
| 10:51:26 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | |
| 10:52:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571 | |
| 10:52:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | |
| 10:52:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | |
| 10:52:43 | stephenfin | owalsh: ^ | |
| 10:53:14 | owalsh | stephenfin: ack, thanks | |
| 10:59:09 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add scheduling notification https://review.openstack.org/566470 | |
| 11:02:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unused methods in nova/compute/utils.py https://review.openstack.org/566855 | |
| 11:06:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924 | |
| 11:30:00 | tetsuro | Hmm... this makes me change my mind that this is a good opportunity to rework the allocation candidates logic in a totally different way. So it is fair that you ask that if we can amend nested alloc cands series without considering the existing path for sharing providers. | |
| 11:43:42 | artom | sean-k-m1, sean-k-mooney[m], one of you around? Have some questions about the Intel NFV CI | |
| 11:50:50 | sean-k-mooney | artom: hi | |
| 11:51:35 | artom | sean-k-mooney, heya - live migration with NUMA (either pinning or actual NUMA topologies) can be tested in the Intel NFV CI, right? | |
| 11:51:42 | sean-k-mooney | sorry i have been on vacation since last tuseday. my mum ha a minor operation so i was being a good son and makeing her tea and mining the house all week | |
| 11:51:52 | artom | sean-k-mooney, awww :) | |
| 11:51:57 | sean-k-mooney | artom: so yes nfv ci | |
| 11:52:11 | artom | Hope she's doing well | |
| 11:53:03 | sean-k-mooney | artom: she is doing fine. am yes you could test numa + livemigration in the intel nfv ci but i think the livemigration tests are not in cluded in teh tempest regex in the multinode job | |
| 11:53:49 | theanalyst | t | |
| 11:54:19 | artom | sean-k-mooney, could you point me to the code? | |
| 11:55:04 | artom | https://github.com/openstack/intel-nfv-ci-tests I assume | |
| 11:55:12 | sean-k-mooney | artom: the ci is currently configured using an internal git repo but im going to take a look at the tempest.conf | |
| 11:55:25 | sean-k-mooney | artom: that is a part of it but not the ci config | |
| 11:55:41 | sean-k-mooney | artom: that host our tempest plugin | |
| 11:56:28 | sean-k-mooney | artom: tempest-dsvm-intel-nfv-xenial/ uses that plugin but its not multinode | |
| 11:56:47 | artom | sean-k-mooney, does it go through gerrit? My plan is to write a live migration NUMA test (unless those already exist? A quick look at the code doesn't turn anything up), and make it Depends-On: my own patches for that | |
| 11:57:49 | sean-k-mooney | artom: the statdard livemigration test would partly test this. the tempest-dsvm-ovsdpdk-nfv-networking-xenial replaces the default flvors with ones that use cpu pinning + hugepages + 2 numa nodes | |
| 11:58:27 | sean-k-mooney | artom: this is one of the tempest.confs from a run on the ci http://52.27.155.124/portland/2018-04-27/553072/6/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/ff9bb78/logs/tempest_conf.txt.gz | |
| 11:59:00 | sean-k-mooney | so yes live_migration = False is set | |
| 11:59:45 | sean-k-mooney | we could modify that via a patch to networking-ovs-dpdk to enable livemigration via its devstack plugin. | |
| 11:59:51 | artom | sean-k-mooney, so at least as a first step I could need to get live_migration set to True? | |
| 12:00:04 | artom | *would need | |
| 12:00:39 | sean-k-mooney | artom: yep, we could also do that via changing the internal repo but i dont have access to it because i dont know where it is now | |
| 12:01:04 | artom | sean-k-mooney, heh, are the contacts listed on https://wiki.openstack.org/wiki/ThirdPartySystems/Intel_NFV_CI still good? | |
| 12:01:22 | sean-k-mooney | networking-ovs-dpdk however i have full contol over. yes the contacts are correct | |
| 12:02:10 | artom | sean-k-mooney, wait, so which one do you *not* have access to? | |
| 12:03:48 | sean-k-mooney | artom: i do not have acess to the zuul 2.5 configure repo that stores the job config. i am the maintainer of networking-ovs-dpdk and have full aceess to modify the job via that repo using the devstack plugin | |
| 12:04:12 | artom | sean-k-mooney, ah, gotcha | |
| 12:04:20 | artom | And those are both internal Intel, right? | |
| 12:05:01 | sean-k-mooney | networking-ovs-dpdk is hosted by openstack infra but is not technically part of openstack. https://github.com/openstack/networking-ovs-dpdk | |
| 12:05:29 | sean-k-mooney | artom: we should be able to use the test-config section of the devstack plugin to enable live migration https://github.com/openstack-dev/devstack/blob/master/doc/source/plugins.rst#pluginsh-contract | |
| 12:05:39 | artom | Also, http://52.27.155.124/portland/2018-04-20/563037/2/check/tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial/3e525a4/logs/tempest_conf.txt.gz seems to enable live_migration? | |
| 12:06:08 | artom | Which woujld | |
| 12:06:22 | artom | Which wou45ld | |
| 12:06:41 | artom | Which would make sense since it's multinode, and the other one isn't | |
| 12:06:51 | sean-k-mooney | tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial used to be the same job just run against neutron | |
| 12:07:26 | sean-k-mooney | sorry yes i ment tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial | |
| 12:07:29 | artom | (Excuse the random characters, daughter decided my laptop is a toy) | |
| 12:07:54 | sean-k-mooney | artom: no worries | |
| 12:08:09 | sean-k-mooney | still more understandable then my spelling at times | |
| 12:11:29 | sean-k-mooney | artom: do you know if devstack-gate sets any env varibles that we could use to enable "ci testing" config changes within the devstack plugin. | |
| 12:11:57 | artom | sean-k-mooney, I'm not sure I understand the question | |
| 12:13:09 | sean-k-mooney | well if we add the sections to enable livemirgation in the networking-ovs-dpdk devstack plugin i am wondering is there a way to enable it automatically in a ci env and not on a normal install | |
| 12:14:00 | sean-k-mooney | if we are just setting tempest settings then there is no real harm always doing it i was just wondering if there was a way to detect that we are running in a ci | |
| 12:15:11 | mriedem | sean-k-mooney: https://github.com/openstack-infra/devstack-gate/blob/b3da8a393c68cc62924ee2f752f442d5c85ea8ed/devstack-vm-gate.sh#L566 | |
| 12:15:45 | mriedem | all non-grenade multinode jobs run live migration by default | |
| 12:15:55 | mriedem | grenade doesn't because grenade by default only runs smoke tests, and live migration tests aren't smoke tests | |
| 12:16:14 | mriedem | we do have a grenade job specifically that tests just live migration back and forth from n-1 to n and back | |
| 12:17:07 | artom | mriedem, I suppose I could just test numa-aware live migration with cpu pinning since we don't need special hardware for that, but there'd be no way to assert anything about the instance XML, which is kinda what we want | |
| 12:17:28 | sean-k-mooney | mriedem: ah you are right looking at http://52.27.155.124/portland/2018-04-27/553072/6/check/tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial/943bdb9/logs/tempest_conf.txt.gz livemigration is turned on the quest then is why is it not running those tests | |
| 12:18:08 | mriedem | sean-k-mooney: blacklist? | |
| 12:18:22 | sean-k-mooney | mriedem: proably ill have to look into that | |
| 12:18:47 | mriedem | [[ ^(?!.*(?:tempest\.scenario\.test_network_advanced_server_ops\.TestNetworkAdvancedServerOps\.test_server_connectivity_suspend_resume.*)|(?:tempest\.scenario\.test_network_advanced_server_ops\.TestNetworkAdvancedServerOps\.test_server_connectivity_pause_unpause.*)|(?:.*\.admin\.test_live_migration.*)|(?:.*\.TestNetworkAdvancedServerOps.test_server_connectivity_cold_migration_revert.*)|(?:.*\.TestNetworkAdvancedServerOps.test | |
| 12:18:47 | mriedem | ver_connectivity_cold_migration.*)|(?:.*\.TestNetworkAdvancedServerOps.test_server_connectivity_resize.*)|(?:.*\.TestNetworkBasicOps\.test_update_router_admin_state.*)|(?:.*\.TestNetworkBasicOps\.test_network_basic_op.*)|(?:.*\.TestNetworkBasicOps\.test_update_instance_port_admin_state.*))((?:tempest\.scenario\.test_network_basic_ops.*)|(?:tempest\.scenario\.test_network_advanced_server_ops.*)|(?:tempest\.api\.compute\.admin\ | |
| 12:18:47 | mriedem | t_live_migration.*)|(?:tempest\.api\.network.*)).*$ != '' ]] | |
| 12:19:02 | mriedem | yes it's blacklisted live migration | |
| 12:19:08 | sean-k-mooney | artom: you need a multi numa host vm as we have a restiction in the libvirt drivver that maps each guest numa node to a different host numa node | |
| 12:19:12 | artom | Which makes sense, because it's broken | |