Posted	Nick	Remark
#openstack-nova - 2018-05-08
04:21:43	openstackgerrit	Merged openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367
04:53:46	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645
05:17:32	openstackgerrit	Merged openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346
05:24:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658
05:25:18	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659
05:46:17	openstackgerrit	jichenjc proposed openstack/nova master: [doc] merge quota document https://review.openstack.org/566783
05:54:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528
06:13:23	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP] Add queued for delete to instance_mappings table. https://review.openstack.org/566788
07:06:49	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795
07:14:03	openstackgerrit	Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795
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 resource providers without inventories https://review.openstack.org/559554
07:35:26	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
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

Earlier   Later