Posted	Nick	Remark
#openstack-nova - 2019-01-29
20:02:53	mriedem	:(
20:03:47	ioni	so is mostly about driver
20:04:00	ioni	or about the storage backend
20:04:06	ioni	qcow, raw or other
20:05:26	mriedem	well, libvirt is by far the most widely used compute driver,
20:05:49	mriedem	if you were using shared instance storage across all of your computes, then the image might be cached and you don't need to fetch it from glance,
20:05:58	mriedem	but that's not a deployment strategy that everyone uses
20:07:27	mriedem	ioni: btw i'm asking in #openstack-operators
20:07:52	ioni	mriedem, yes, i saw, i'm going to continue there
20:11:04	mriedem	jaypipes: the answer was change the image status to "community" which hides it from the image list for all users except the owner, but you can reach it by uuid if you know it
20:11:13	mriedem	so like shared but less private
20:18:30	jaypipes	mriedem: ugh
20:18:52	jaypipes	mriedem: that is so very non-obvious.
20:20:15	mriedem	at least there is a solution
20:26:52	cdent	"at least there is a solution" is the first album from my band "so very non-obvious"
20:35:26	artom	I thought My Chemical Romance already did that?
20:43:53	cdent	mriedem, jaypipes : can either of you remember the details of what's left on the nova side to make shared disk a possibility?
20:44:29	cdent	The reason I ask is because I've been modelling various disk set ups in placement with the usual gabbits things I do, and it works very nicely from that side of things.
20:52:29	mriedem	the bug about it has details
20:52:50	jaypipes	cdent: mostly I think it's just that nova and the virt drivers simply have no idea that instances use shared storage or not. There needs to be something added to the resource tracker (I had proposed that provider YAML config file for this type of purpose) that communicates to the nova-compute service that the compute node has access to shared storage.
20:52:51	mriedem	https://bugs.launchpad.net/nova/+bug/1784020
20:52:52	openstack	Launchpad bug 1784020 in OpenStack Compute (nova) "Shared storage providers are not supported and will break things if used" [Medium,Fix released]
20:53:18	mriedem	there are lots of TODOs and FIXMEs in the code with things like "this will need to change to support sharing providers"
20:53:24	mriedem	like in the cold migration flow
20:53:30	cdent	thanks, I had lost track of that bug
20:53:43	mriedem	also L220 https://etherpad.openstack.org/p/nova-ptg-stein
20:53:53	jaypipes	cdent: along with a giant-ass reshape-the-world-for-shared-storage function that would "correct" the allocation of resources.
20:57:02	cdent	It's looking like shared disk could make multiple datastores in one vsphere cluster way more sensible. nested could also work, but shared will work in both the "this cluster owns this store" and "this store is used by multiple clusters" set ups
21:05:21	mriedem	fyi, new pip release of something is destroying pep8 in the gate
21:06:41	mriedem	this https://pypi.org/project/pycodestyle/2.5.0/
21:11:06	cdent	joy!
21:11:20	cdent	this is the disk playing about I was talking about ^^ https://github.com/cdent/placement-disk-models
22:04:45	mriedem	good lord i really dislike the PciDevice* object interfaces
22:04:51	mriedem	so different from everything else we have for versioned objects
22:06:32	mriedem	burying too much business logic in the data model
22:06:42	mriedem	it's that some kind of anti-pattern...
22:06:45	mriedem	*isn't that
22:38:28	mriedem	dansmith: interesting fun fact of the day: you can't load 'fault' using expected_attrs with Instance.get_by_uuid
22:40:41	mriedem	http://paste.openstack.org/show/744212/
22:40:47	mriedem	something everyone cares about
23:24:12	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove deprecated 'os-server-groups' policy https://review.openstack.org/633672
#openstack-nova - 2019-01-30
00:51:20	openstackgerrit	Matt Riedemann proposed openstack/nova master: Isolate cell-targeting code in MigrationTask https://review.openstack.org/621310
00:51:21	openstackgerrit	Matt Riedemann proposed openstack/nova master: Move resize.prep.start/end notifications to helper method https://review.openstack.org/633298
00:51:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Extract compute API _create_image to compute.utils https://review.openstack.org/621311
00:51:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
00:51:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035
00:51:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
00:51:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
00:51:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
00:51:25	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123
00:51:25	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.openstack.org/627892
00:51:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581
00:51:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313
00:51:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add prep_snapshot_based_resize_at_dest compute method https://review.openstack.org/633293
00:51:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
00:51:28	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
00:51:28	openstackgerrit	Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.openstack.org/633853
01:14:00	openstackgerrit	Adam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts https://review.openstack.org/633855
02:16:42	openstackgerrit	Merged openstack/nova master: Raise 403 instead of 500 error from attach volume API https://review.openstack.org/632904
02:16:51	openstackgerrit	Merged openstack/nova stable/pike: Default embedded instance.flavor.is_public attribute https://review.openstack.org/619352
02:16:58	openstackgerrit	Merged openstack/nova master: Fix port dns_name reset https://review.openstack.org/631684
08:20:07	openstackgerrit	Yongli He proposed openstack/nova master: Add server subresouce toplogy API https://review.openstack.org/621476
10:08:15	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
10:08:15	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
10:29:55	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Fup for the bandwidth resource provider series https://review.openstack.org/633776
10:30:06	openstackgerrit	Merged openstack/nova master: Fix string interpolations in logging calls https://review.openstack.org/633657
10:40:03	openstackgerrit	Merged openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325
10:42:44	openstackgerrit	melanie witt proposed openstack/nova stable/rocky: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/633910
13:20:42	openstackgerrit	Merged openstack/nova master: Don't call begin_detaching when detaching volume from shelved vm https://review.openstack.org/624959
13:28:01	openstackgerrit	Merged openstack/nova master: Reject interface attach with QoS aware port https://review.openstack.org/570078
13:53:32	mriedem	Kevin_Zheng: should be able to cherry-pick https://review.openstack.org/#/c/624959/ to stable/rocky in the gerrit UI, I don't expect there to be conflicts
13:54:38	Kevin_Zheng	mriedem: yeah, I was planning to, my computer is updating...again
13:59:12	mnaser	not that i run or care much about ocata but, all previous serious patches have merged -- https://review.openstack.org/#/c/619366/
13:59:26	mnaser	(i just want to get it off my review list :)
13:59:52	mnaser	(cc melwitt smcginnis as they've merged the stable/pike)
14:05:50	smcginnis	Oh yeah, I had looked earlier but was waiting for later stable branches to merge.
14:12:15	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Fix port dns_name reset https://review.openstack.org/633806
14:15:53	sean-k-mooney	stephenfin: jaypipes care to re +w https://review.openstack.org/#/c/591607/? it still can merge until https://review.openstack.org/#/c/614167/20 does but if ye are still happy with it it would be nice to have that lined up for wehn the bottom patch is merged
14:17:49	jaypipes	sean-k-mooney: done. I need to final review the bottom patch anyway, thx for the reminder
14:19:27	sean-k-mooney	jaypipes: this fixes a downstream customer issue which has be blocked on my trello board since last year so i am both lokking forward to it mergenign and not since i will have to backport it to newton...
14:20:31	maciejjozefczyk	\o/
14:25:48	openstackgerrit	Zhenyu Zheng proposed openstack/nova stable/rocky: Don't call begin_detaching when detaching volume from shelved vm https://review.openstack.org/633960
14:25:54	cdent	all the way to newton? ugh
14:27:06	openstackgerrit	Zhenyu Zheng proposed openstack/nova stable/queens: Don't call begin_detaching when detaching volume from shelved vm https://review.openstack.org/633962
14:29:24	mriedem	jangutter: a few questions in https://review.openstack.org/#/c/572082/
14:30:25	mriedem	sean-k-mooney: if it makes you feel better huawei will need to take it to mitaka
14:32:40	jangutter	mriedem: thanks, checking!
14:34:05	mdbooth	lyarwood: Just looking at your patch https://review.openstack.org/#/c/626952/1/nova/api/openstack/wsgi_app.py. IIUC that's basically what we do for all non-wsgi services here: https://github.com/openstack/nova/blob/master/nova/cmd/__init__.py
14:34:49	jaypipes	sean-k-mooney: yikes. (on newton). I just backported shit to Ocata and know how painful that is.
14:35:10	mdbooth	I'm pretty sure that means that we monkey patch while looking up the entry point, so before anything else.
14:35:57	sean-k-mooney	mriedem: well if it was not for the fact that osp 7 went endo f life on august 5 2018 we would have had to backport to kilo
14:36:18	mdbooth	'end of life'
14:38:24	mriedem	tssurya: regarding https://review.openstack.org/#/c/614167/ cern might want to investigate that up front before running it on all cells since it iterates all cells but is intended to migrate cell-specific data
14:38:40	mriedem	which is kind of different from how we do other online data migrations which only hit the cell db
14:38:41	sean-k-mooney	jaypipes: osp 10/netwon is an lts release which will be supported until decmber 15 2021 alther it will move to extend maintant at the end of the year but ya we have to backport to newton faily commonly at present
14:39:03	mriedem	tssurya: it uses a marker though so once it runs for all cells on the first run it should be a noop if run on each individual cell,
14:39:20	mriedem	which i'm assuming cern does as part of upgrades - run online_data_migrations per cell db config via config management
14:39:43	mdbooth	And, on occasion, to Kilo still, despite being 'end of life' :/
14:41:00	sean-k-mooney	mdbooth: well there is always the $$$ factor that makes "end of life" more flexible then we might like :)
14:41:04	mdbooth	lyarwood: Ah, your patch touches that code too. And.... I'll bet the reason is that nova.utils imports keystoneauth1, which means we've already imported it before running the monkey patching.

Earlier   Later