Posted	Nick	Remark
#openstack-nova - 2019-11-20
22:28:19	efried	and (I think) the driver can't tell if it's being asked to snapshot as part of a shelve or backup or...
22:28:47	sean-k-mooney	am im not 100% sure about that last part
22:29:04	efried	so I guess it could edit the sysmeta itself... but if it's not a shelve and the compute mgr is about to blow away the instance, the compute mgr needs a way to get that information so it can stuff it into the image meta.
22:29:13	efried	I guess it can pull it from the instance
22:29:43	efried	so the contract with the virt driver's snapshot method has to spell out where & how this info is stored in the sysmeta.
22:30:29	sean-k-mooney	we create snapshot for shelve but i dont think the compute magener actully calls snapshot as part of that flow i think its donw by the driver
22:30:47	efried	eh?
22:30:49	sean-k-mooney	that said i could be way off base there so im checking
22:30:56	efried	compute manager's shelve calls driver's snapshot
22:31:24	sean-k-mooney	oh ok
22:31:31	efried	https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L5911
22:32:32	sean-k-mooney	ya your right i though we had a shelve fucntion on the driver interface that we called but this is handeled in the compute manager instead
22:33:30	sean-k-mooney	so snapshot is passed the instace so it can get the flavor and image metadata
22:33:58	sean-k-mooney	so the snapshot function can check if the isntace has a vtpm and directly update the system metadata as you said
22:34:14	sean-k-mooney	oh right but it would not know if its a shleve or not
22:36:00	sean-k-mooney	oh it can jsut check the instance.task_state right?
22:36:38	efried	could, but compute mgr has to be able to peel it out anyway, so it's simpler if snapshot just always behaves the same and compute mgr pulls the info out of the sysmeta if it needs it.
22:37:14	sean-k-mooney	ok
22:37:53	sean-k-mooney	anyway night all o/
22:46:34	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Integrating with unified limits https://review.opendev.org/615180
23:09:16	openstackgerrit	John Garbutt proposed openstack/nova master: Better policy unit tests https://review.opendev.org/657697
23:17:55	efried	argh, destroy_disks=True is no longer sufficient grounds to delete the barbican key :(
23:18:28	efried	In fact, I'm not sure we can "ever" do that.
23:51:05	openstackgerrit	Eric Fried proposed openstack/nova-specs master: Spec: Ussuri: Encrypted Emulated Virtual TPM https://review.opendev.org/686804
23:51:38	efried	phew. dansmith sean-k-mooney TxGirlGeek ^
23:51:45	efried	and jroll
#openstack-nova - 2019-11-21
00:06:38	openstackgerrit	Dustin Cowles proposed openstack/nova-specs master: Update provider config spec for identification conflicts https://review.opendev.org/693414
00:42:50	openstackgerrit	Merged openstack/nova stable/train: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695145
01:11:11	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591
01:11:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
01:11:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
01:11:13	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
01:11:13	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013
01:11:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058
01:11:14	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
01:11:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075
01:11:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
01:11:16	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
01:11:16	openstackgerrit	Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453
01:11:17	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647
01:11:17	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add RevertResizeTask https://review.opendev.org/638046
01:11:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047
01:11:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
01:11:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.opendev.org/638268
01:11:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add archive_deleted_rows wrinkle to cross-cell functional test https://review.opendev.org/651650
01:11:20	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.opendev.org/614353
01:16:35	gmann	efried: any idea or have you seen 'openstack:' as resource provider name in https://zuul.opendev.org/t/openstack/build/b26bd7fc82d94d4cac97b5481623b629/log/logs/grenade.sh.txt.gz#30397
01:16:55	gmann	this is grenade job on octavia which is failing due to that when moving to py3.
05:23:13	openstackgerrit	Merged openstack/nova stable/pike: Only nil az during shelve offload https://review.opendev.org/693839
06:36:10	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
08:10:09	bauzas	good morning Nova
08:16:32	gibi	bauzas: good morning
08:16:51	bauzas	:)
08:50:26	openstackgerrit	ya.wang proposed openstack/nova master: Handle instance crash event in libvirt driver https://review.opendev.org/695369
09:25:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
09:26:36	stephenfin	bauzas: Morning. Any chance you could look at one or two of the "die, nova-network, die" patches I have up today? Like, say, this really easy one https://review.opendev.org/#/c/684345/ ?
09:26:47	bauzas	lol ok
09:28:32	openstackgerrit	Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.opendev.org/650188
09:34:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: zuul: Remove unnecessary 'USE_PYTHON3' https://review.opendev.org/695380
09:45:22	openstackgerrit	Sylvain Bauza proposed openstack/nova stable/stein: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695381
09:49:08	openstackgerrit	Sylvain Bauza proposed openstack/nova stable/rocky: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695382
09:51:44	openstackgerrit	Sylvain Bauza proposed openstack/nova stable/queens: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695383
10:32:34	yaawang	mdbooth: Hello, can you review the spec again, I'd updated it :) https://review.opendev.org/#/c/693655/
10:52:05	kashyap	yaawang: Hi, on the "no_performance_impact" name -- I still find it too broad and sweeping
10:52:15	kashyap	yaawang: Thanks for addressing my feedback, though!
10:53:14	kashyap	I agree, we don't want to expose "hypervisor"-specific features. (BTW, we are abusing the term "hyerpvisor" here to include QEMU; the term normally includes KVM/Kernel area only.)
10:53:22	kashyap	Maybe "no_migration_perf_impact"
10:57:01	kashyap	But just plain "no_performance_impact" is just _awful_ name (Cc: mdbooth.) At least make it: "no_migration_perf_impact"
11:10:15	yaawang	kashyap: Agree, "no_performance_impact" sounds include live-migrate and migrate. How about "no_perf_impact_live_migration"?
11:12:09	kashyap	yaawang: Yeah, I thought of including "live" as well; almost good - but reverse it: "no_live_emigration_perf_impact"
11:12:55	kashyap	Typo: s/emigration/migration/
11:14:22	kashyap	yaawang: Commented on the change. I'd say, let's go with the above ("no_live_migration_perf_impact").
11:17:24	yaawang	kashyap: Thanks, need mdbooth to post his comment.
12:03:41	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/pike: Explicitly fail if trying to attach SR-IOV port https://review.opendev.org/695408
12:16:39	gibi	bauzas: you can send this to the age too https://review.opendev.org/#/c/686808
12:34:50	openstackgerrit	Wei Hui proposed openstack/nova master: bugfix device_type=type-PCI passthrough failed https://review.opendev.org/695416
12:39:19	gibi	mriedem, stephenfin: thanks for the laugh https://review.opendev.org/#/c/489267/2/nova/network/neutronv2/api.py@181
12:41:09	gibi	stephenfin: btw you have now a lot of +2s on your nova-net removal series :)
12:42:29	efried	gmann: looking...
12:49:22	mriedem	gibi: heh, anytime
12:51:07	efried	gmann: that log file isn't loading up for me :(
12:51:45	efried	It would be weird for a provider managed by nova to have 'openstack:' in its name. If it was made by some other service, maybe, but I've never heard of it.
12:53:28	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Include removal of ephemeral backing files in the image cache manager https://review.opendev.org/689422
12:58:15	johnthetubaguy	stephenfin: I just had a look at the network API removal... are we not adding a microversion to signal when we removed the API, I think that would be more consistent with our rules?
12:58:45	johnthetubaguy	I am guessing I missed about a year of conversation, so probably missed the reasoning
13:00:05	mriedem	fun gate bug i finally wrote up after rechecking several times https://bugs.launchpad.net/tempest/+bug/1853453
13:00:05	openstack	Launchpad bug 1853453 in tempest "test_shelve_volume_backed_instance intermittently fails guest ssh with dhcp lease fail" [Undecided,New]
13:00:11	mriedem	looks like this only hits in multinode jobs,
13:00:21	mriedem	and looking at a recent failure, we shelve from the primary node and unshelve on the subnode
13:00:45	mriedem	i wonder if the snapshot is no good in some cases and unshelving on another node causes some kind of issue
13:06:33	mriedem	oh nvm there is no snapshot, it's volume-backed
13:06:34	mriedem	derp
13:12:08	gibi	efried: do you know if provider_tree in the report client will contain RPs associated to the compute RP via aggregate? this comment states it does https://github.com/openstack/nova/blob/1cd5563f2dd2b218db2422397c8aab394d484626/nova/scheduler/client/report.py#L593
13:12:36	gibi	efried: but I did not find in the impl where we query for those RPs from placement
13:12:51	efried	gibi: it should, yes; the code has been written that way for a while, but we've never had a real scenario that uses it.
13:13:07	efried	hold on, let me find you the code (It's somewhere under _refresh_associations...)
13:13:52	efried	ah, it looks like we specifically target only sharing providers (we specifically filter on MISC_SHARES)
13:13:55	efried	which makes sense
13:14:04	efried	we wouldn't want to just arbitrarily pick up anything in an aggregate.
13:14:37	gibi	efried: cool, MISC_SHARES... is what I'm looking for. Do you have a link?

Earlier   Later