Posted	Nick	Remark
#openstack-nova - 2019-11-14
01:17:09	openstackgerrit	Merged openstack/nova stable/rocky: doc: fix and clarify --block-device usage in user docs https://review.opendev.org/693554
01:17:16	openstackgerrit	Merged openstack/nova stable/rocky: Revert "openstack server create" to "nova boot" in nova docs https://review.opendev.org/693240
01:46:16	openstackgerrit	Merged openstack/nova master: Fix review link. https://review.opendev.org/689612
03:08:07	openstackgerrit	Merged openstack/nova master: Use named kwargs in compute.API.resize https://review.opendev.org/693425
03:08:13	openstackgerrit	Merged openstack/nova master: "SUSPENDED" description changed in server_concepts guide and API REF https://review.opendev.org/663590
05:24:24	alex_xu	stephenfin: yea, Intel have proxy.
05:24:53	alex_xu	stephenfin: let me know if you come to Beijing, I can treat you local food :)
05:25:29	alex_xu	bauzas: nice, one more place catch you for review ;)
05:49:06	openstackgerrit	Abhishek Kekane proposed openstack/nova-specs master: Support multiple store of Glance https://review.opendev.org/641210
06:02:34	openstackgerrit	Abhishek Kekane proposed openstack/nova-specs master: Support multiple store of Glance https://review.opendev.org/641210
08:00:02	openstackgerrit	Guo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
08:29:40	openstackgerrit	Merged openstack/nova master: Require Migration object arg to migrate_instance_finish method https://review.opendev.org/690106
08:29:46	openstackgerrit	Merged openstack/nova master: Consolidate [image_cache] conf options https://review.opendev.org/690723
08:29:54	openstackgerrit	Merged openstack/nova master: Add image caching to the support matrix https://review.opendev.org/690748
08:30:00	openstackgerrit	Merged openstack/nova master: Stop converting Migration objects to dicts for migrate_instance_start https://review.opendev.org/690115
08:49:51	openstackgerrit	Merged openstack/nova master: Make API always RPC cast to conductor for resize/migrate https://review.opendev.org/693937
08:55:58	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Add flavor extra specs group https://review.opendev.org/663563
09:46:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: Convert legacy nova-live-migration and nova-multinode-grenade to py3 https://review.opendev.org/692374
09:46:38	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop testing Python 2 https://review.opendev.org/687954
09:46:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: zuul: Make functional job inherit from openstack parents https://review.opendev.org/688425
10:28:04	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/pike: Only nil az during shelve offload https://review.opendev.org/693839
10:28:47	openstackgerrit	Merged openstack/nova master: Remove dead set_admin_password code to generate password https://review.opendev.org/691873
10:29:01	openstackgerrit	Merged openstack/nova master: doc: add troubleshooting guide for cleaning up orphaned allocations https://review.opendev.org/691427
10:31:41	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/pike: Only nil az during shelve offload https://review.opendev.org/693839
11:13:55	brinzhang	gibi_off: Thanks for your review https://review.opendev.org/#/c/663563/, how about rename the bp as "resource metadata of instance"? and change the new db table's name as "resource_metadata"?
11:20:41	brinzhang	gibi_off: I think we also should establish the association for the "instance_uuid" and "resource_metadata_id". here, a) create a new table, contain the columns of "instance_uuid" and "resource_metadata_id". b) add "resource_metadata_id" to the "instance_metadata" db table, and mapping the choosed (key, value) to the instance.(I think this is compatible with current usage.)
11:36:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Change order of two classes https://review.opendev.org/689178
11:36:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Rework '_delete_server' https://review.opendev.org/689179
11:36:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Make '_wait_for_state_change' behave consistently https://review.opendev.org/689180
11:36:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Unify '_wait_until_deleted' implementations https://review.opendev.org/689181
11:36:08	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Make 'ServerTestBase' subclass 'InstanceHelperMixin' https://review.opendev.org/689182
11:42:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: DNM: Try to figure out why rbd import fails https://review.opendev.org/694260
12:22:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: DNM: python3 experiments https://review.opendev.org/694281
13:26:05	openstackgerrit	Lee Yarwood proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices https://review.opendev.org/693849
13:26:05	openstackgerrit	Lee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063
14:19:27	efried	o/ nova
14:19:39	efried	Is anyone around for the meeting?
14:20:00	efried	First meeting since PTG+USADST, so an hour earlier for many of us.
14:23:48	gibi_off	efried: I have to skip it today
14:24:16	efried	Okay. You and bauzas and stephenfin were going to be the stars of the show, so...
14:24:35	efried	and next week's (at 2100 utc) is going to be inconvenient for all of you.
14:24:41	bauzas	sorry, on 1:1 meeting now
14:26:16	gibi_off	efried: yeah 2100 is bad, but today (and yesterday) I have to be present on an internal conference demoing the bandwidth stuff repeatedly
14:26:37	efried	that sounds like more fun than a nova meeting anyway.
14:26:59	gibi_off	efried: and as this is done in an open demo area I don't know when somebody pops up at our booth
14:27:37	gibi_off	I can imagine there is a way to define fun that applies to me now :)
14:27:38	efried	Heh, hang an "out to lunch" placard :P
14:27:46	gibi_off	:)
14:34:39	stephenfin	efried: I got confused with the timezones. Drat /o\
14:34:46	stephenfin	DST--
14:43:45	mriedem	huh, i forgot to change my calendar as well :)
14:48:52	mriedem	efried: was there anything you wanted to go over? my thing in open discussion is in the ML and i'm just going to do a quick PoC I think
14:49:02	mriedem	there were a few stable branch releases
14:49:08	mriedem	and elod was added to the nova-stable-maint team
14:52:22	efried	mriedem: The other open (which luyao hadn't put on the agenda as of meeting start time) was whether the LM bits of vPMEM need a BP/spec or not. What are your thoughts?
14:55:20	mriedem	my gut reaction is vPMEM hasn't even baked to the point of thinking about supporting live migration
14:55:40	mriedem	i.e. i don't think we even support live migration with vgpus do we?
14:55:48	mriedem	and people might actually be using that by now
14:56:12	mriedem	vpmem and live migration is likely complicated that a short spec probably doesn't hurt
14:56:20	mriedem	*complicated enough
14:56:45	mriedem	stephenfin: i commented on https://review.opendev.org/#/c/692374/
14:57:02	mriedem	tl;dr i don't think USE_PYTHON3=True is making it to the devstack-plugin-ceph run when the post-test script runs it
14:57:36	mriedem	stephenfin: also, in trying to convert the nova-live-migration job to zuulv3 here https://review.opendev.org/#/c/693364/ oi
14:57:53	mriedem	i'm coming to the conclusion that it would be a ton simpler to just split the nova-live-migration job in two,
14:58:06	mriedem	one for block migration and local disk and one with ceph + shared disk
14:58:14	efried	mriedem: Pretty sure the impl is already done, it just didn't make Train. Anyway, I agree it could do with a short spec, as I think it needs new object fields, so we should at least be able to bikeshed the names of those. <== luyao
14:58:35	mriedem	efried: is the vpmem 3rd party ci stable?
14:58:46	mriedem	and if it's stable, can they run it multinode to test live migration?
14:58:50	efried	dunno. alex_xu luyao ^ ?
14:58:51	mriedem	at least periodically?
14:59:18	efried	I thought the CI was stable at the end of train, for single node. Don't know the status of LM CI.
14:59:38	mriedem	i just don't have a ton of energy to invest in building feature functionality on top of a thing that i don't know if anyone is using it
14:59:50	mriedem	like SEV in that way...
15:00:21	mriedem	having said all that, i think dansmith would be totally on board with building in live migration support for vpmems, that's right up his alley
15:01:01	efried	I think of it less as "tack a feature onto this thing" and more as "piece of the puzzle that didn't quite get finished with the rest".
15:02:16	efried	oh, maybe I'm wrong. I see the patch wasn't proposed until Oct.
15:02:32	mriedem	so looking at a recent pmem ci run http://52.27.155.124/79/694179/2/check/pmem-tempest-plugin-filtered/1123b99/job-output.txt
15:02:36	mriedem	it runs 3 tests in less than a minute
15:02:43	mriedem	the entire job takes about 30 minutes
15:03:05	mriedem	surely there is more testing that could be done with operations on a server with pmems, though i'm not sure what test_server_basic_ops does
15:03:24	mriedem	e.g. if resize works, cold migration should work, so maybe first steps are making that ci job multinode and testing cold migrate
15:04:05	sean-k-mooney	looking at http://ciwatch.mmedvede.net/project?project=nova it seams somewhat statble although it is missing some patches
15:04:41	mriedem	ok here is the basic ops test https://github.com/LuyaoZhong/vpmem-tempest-plugin/blob/master/vpmem_tempest_plugin/tests/scenario/test_server_basic_ops.py#L127
15:04:56	mriedem	create a server, ssh into it, delete the server...
15:05:03	mriedem	so reboot + pmem isn't being tested
15:05:03	stephenfin	mriedem: https://review.opendev.org/#/c/694281/1
15:05:06	mriedem	shelve/unshelve
15:05:16	stephenfin	I think that fixes the Python 3 issues, right?
15:05:23	stephenfin	Or have I missed something obvious?
15:06:16	sean-k-mooney	mriedem: i suspect pause/unpause and suspend/resume are also missing
15:06:37	stephenfin	obviously converting things to zuul v3 native would be better though, of course, but maybe I should squash that back so we can continue working on the Python 3 only'ification in parallel
15:07:17	mriedem	stephenfin: looks like it does yeah, py3 is being used here https://zuul.opendev.org/t/openstack/build/2850353a84da4ec08b4f7cda7a08a781/log/logs/subnode-2/screen-n-cpu.txt.gz#3
15:07:20	sean-k-mooney	mriedem: they have a resize test https://github.com/LuyaoZhong/vpmem-tempest-plugin/blob/master/vpmem_tempest_plugin/tests/scenario/test_server_pmem_ops.py#L105
15:07:27	mriedem	sean-k-mooney: that would be resize to the same host though
15:07:32	stephenfin	\o/
15:07:38	sean-k-mooney	yes
15:07:54	mriedem	stephenfin: so you want to squash that into https://review.opendev.org/#/c/692374/ ?
15:08:05	stephenfin	yeah
15:08:08	sean-k-mooney	but if resize on same host work then they should be able to resize to a different host no? if it was run multinode
15:08:08	mriedem	go for it

Earlier   Later