Posted	Nick	Remark
#openstack-nova - 2019-06-28
00:14:36	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
00:35:30	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add placement request filter for disabled computes https://review.opendev.org/654596
00:51:11	openstackgerrit	Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061
00:51:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove 'MultiattachSupportNotYetAvailable' exception https://review.opendev.org/651315
00:51:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Follow-up for I6a777b4b7a5729488f939df8c40e49bd40aec3dd https://review.opendev.org/664967
01:06:15	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.opendev.org/648686
01:17:48	bhagyashris	mriedem, Sean-k-mooney, alex_xu: Hi, I would like your insights on https://review.opendev.org/#/c/612626/
01:26:57	openstackgerrit	Merged openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.opendev.org/648686
01:41:19	adriant	Hello! Out of curiosity, was there ever a bug in Nova that stopped delete-on-terminate working specifically for volumes created from an instance snapshot?
01:42:04	adriant	^ we have a case where a volume made from a volume snapshot, delete-on-terminate works. The volume just goes away with the instance
01:42:41	adriant	but a volume made from an instance snapshot, delete-on-terminate doesn't work. And the volume must be manually delete afterwards.
01:43:34	adriant	we are running an old version of nova, so just curious if this was a known bug, and if it ever got fixed, or if this is just some weird combination of failures somewhere on our deployment
01:44:50	sean-k-mooney	not that i see in launchpad
01:44:50	openstackgerrit	Merged openstack/nova stable/stein: Noop CantStartEngineError in targets_cell if API DB not configured https://review.opendev.org/663030
01:44:57	openstackgerrit	Merged openstack/nova stable/stein: Fix python3 compatibility of rbd get_fsid https://review.opendev.org/664512
01:45:02	adriant	yeah, I didn't find anything either
01:46:08	sean-k-mooney	to be clear your workflow is you boot a vm. create a snapshot then later boot another vm from the instnace shapshot with delete-on-terminate
01:46:14	adriant	yes
01:46:39	adriant	and if you instance make a volume snapshot of that instances root disk, and boot from that, it works as expected
01:46:42	adriant	your*
01:46:43	sean-k-mooney	that should be the same as a boot form a standard image really
01:47:29	adriant	I know :P
01:47:33	adriant	but somehow it isn't
01:50:40	adriant	I'm just confirming that I can replicate it with a new instance
01:50:45	adriant	but I've seen a client do it
01:58:48	yonglihe	mriedem, sean-k-mooney: what i got from discuss, 1) use UUID instead of name for querying orphans instances 2) checking metadata (which is already done per Sean's suggest), 3) clean up if possible , if db record gone, i don't how to do that. for reap/poweroff/delete, we already had configuration options. 4) re-use/extending current period task instead of create new one.
01:59:46	yonglihe	mriedem, sean-k-mooney: these changes seems quite a lot for this 2 patches. i want to make sure i'm on right track before i start it.
02:09:00	sean-k-mooney	yonglihe: you likely do not need to have 2 seperate patches.
02:09:32	sean-k-mooney	most of the chagnes in the first patch are not required.
02:10:04	sean-k-mooney	yonglihe: melwitt: was suggeting we might be able to jsut create a fake instance object and reuse the existing destroy method in the driver
02:10:24	sean-k-mooney	that occured to me als but i was not sure it that would be valid in all cases
02:12:29	sean-k-mooney	but yes 1.) we would prefer to use uuids instead of names, 2.) for the libvirt dirver check the metadata(which you have already done) 3.) determin if we can clean up local resoces like disk or ports on a vswitch. 4.) combine you chagnes with the existing periodic task as a new confg option reap_unknown
02:27:09	yonglihe	sean-k-mooney: sure, seems we cloud move on, it's gonna drop a lot and come up with new one, though. about clean up local resources, i might need reach for help, might trouble you -:)
02:31:30	sean-k-mooney	the concern is if we jsut delete the domain we would leave the root disk and vswitch ports behind. and in that case the operator would not know what instance they belonged or if it wa safe to remvoe them
02:32:39	sean-k-mooney	we might decide that is ok but if we can sanely clean up that would be nice to do
02:38:32	yonglihe	we might need to add power_off_unknow in case admin want to keep them for investigating.
02:40:11	sean-k-mooney	yes i think that would be fine and in that case no cleanup is needed
02:47:29	yonglihe	I gonna add them both.
03:45:39	openstackgerrit	sean mooney proposed openstack/nova master: [WIP] Libvirt: report storage bus traits https://review.opendev.org/666914
03:45:40	openstackgerrit	sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
03:45:40	openstackgerrit	sean mooney proposed openstack/nova master: [WIP] use domain capablites to get supported device models https://review.opendev.org/666915
04:09:43	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix test_flavors to run with correct microversion https://review.opendev.org/666536
04:10:13	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add missing tests for flavor extra_specs mv 2.61 https://review.opendev.org/667600
04:10:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
05:39:53	openstackgerrit	Merged openstack/nova master: Fix test_flavors to run with correct microversion https://review.opendev.org/666536
06:51:10	gmann	efried: can you re-approve this. it was rebased - https://review.opendev.org/#/c/667600/4
07:03:20	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Multiple API cleanup changes https://review.opendev.org/666889
07:49:45	kashyap	lyarwood: Morning; stable/stein has merged, if you want to ACK its stable/rocky counterpart: https://review.opendev.org/#/c/667436/1
08:00:38	lyarwood	kashyap: the same rules apply to stable as master fwiw so I can't just +2+W this, I'll recheck and leave it to others to +W.
08:01:40	kashyap	lyarwood: Okay, thought I was under the illusion that johnthetubaguy had also ACKed the stable/rocky one
08:03:02	lyarwood	kashyap: I think that was a different one no?
08:03:38	kashyap	lyarwood: Which one was it? Strangely I recall there were 2 ACKs from John on this same change
08:03:49	kashyap	Maybe he ACKed the Git/master one and I confused with it; /me goes to look
08:04:17	kashyap	(No, never mind; I'll chalk it up to poor memory)
08:06:05	kashyap	I find it dispiriting that even for multiple backports there's more burden of getting *two* "ACKs" from separate companies.
08:07:49	openstackgerrit	Kashyap Chamarthy proposed openstack/nova stable/queens: libvirt: Rework 'EBUSY' (SIGKILL) error handling code path https://review.opendev.org/668111
09:07:49	openstackgerrit	Lee Yarwood proposed openstack/nova stable/stein: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/668118
09:08:10	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/668119
09:22:29	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/668123
09:30:25	openstackgerrit	Lee Yarwood proposed openstack/nova stable/stein: DNM - Test rbd unshelve fix on stable stein https://review.opendev.org/668126
09:43:48	openstackgerrit	zhufl proposed openstack/nova master: Fix invalid assertIsNone states https://review.opendev.org/668132
10:36:44	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Add flavor group https://review.opendev.org/663563
10:38:00	openstackgerrit	Merged openstack/python-novaclient stable/rocky: Use SHA256 instead of MD5 in completion cache https://review.opendev.org/664025
10:45:26	openstackgerrit	Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
11:03:19	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: DNM - Test rbd unshelve fix on stable rocky https://review.opendev.org/668141
11:06:49	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: DNM - Test rbd unshelve fix on stable queens https://review.opendev.org/668142
11:21:29	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660363
11:21:30	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: DNM - Test rbd unshelve fix on stable queens https://review.opendev.org/668142
11:21:30	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/668123
12:17:09	efried	gmann: done
13:04:31	openstackgerrit	Merged openstack/nova master: Fix invalid assertIsNone states https://review.opendev.org/668132
13:26:48	kashyap	johnthetubaguy: Can you please put this stable/rocky backport through? https://review.opendev.org/#/c/667436/
13:26:58	kashyap	johnthetubaguy: You've ACKed the stable/stein one
13:27:11	kashyap	(And this, stable/rocky version, already has one +2)
13:27:31	kashyap	Or anyone else, if John is not around
13:32:13	KeithMnemonic	is there anyone here that is maintaining or familiar with https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/ds_util.py
13:33:31	kashyap	KeithMnemonic: Your best bet is GKotton (who is not here on IRC)
13:34:04	efried	mriedem, dansmith: placement agg sync is automatic now, right? No need to run nova-manage placement sync-aggregates? https://review.opendev.org/#/c/667952/1/doc/source/reference/forbidden-aggregates.rst@44
13:34:16	kashyap	KeithMnemonic: You might want to try his e-mail (gkotton@vmware.com).
13:34:36	dansmith	efried: yeah, the manage sync is for fixup and upgrades at this point I think
13:34:37	kashyap	[And of course, Cc the list, so others could learn, too.]
13:34:45	efried	thanks dansmith
13:35:32	KeithMnemonic	yes i was looking for him, we know each other so hopefully i can find him next week
13:35:43	KeithMnemonic	thanks kashyap
13:36:14	kashyap	KeithMnemonic: By "the list" I meant: openstack-discuss@lists.openstack.org
13:36:20	mriedem	efried: right what dan said,
13:36:32	mriedem	except if the api fails to remove a provider from an aggregate, sync_aggregates won't fix that
13:36:42	mriedem	sync_aggregates is only additive
13:36:53	KeithMnemonic	ok, i thought you meant wait for him to show up back here. i will send an email today
13:38:02	efried	also, mriedem, /me Remind / ping / harass re https://review.opendev.org/662881 (sdk spec)
13:39:26	mriedem	d'oh!
13:39:39	mriedem	can i hit snooze on that until after i take my kid to camp?
13:39:57	efried	of course. I can hit you up hourly
13:40:20	dansmith	cron
13:40:34	dansmith	it's they only way he'll learn.
13:46:57	openstackgerrit	Merged openstack/nova master: Add missing tests for flavor extra_specs mv 2.61 https://review.opendev.org/667600
14:08:28	efried	ugh, do we not have `openstack resource provider trait add` ?
14:24:29	mriedem	efried: https://docs.openstack.org/osc-placement/latest/cli/index.html#trait-create
14:24:46	mriedem	osc verbs are create/set/unset/delete/list
14:24:52	mriedem	and /show

Earlier   Later