Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-20
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: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
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: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:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
01:11:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
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:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
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: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:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
01:11:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add RevertResizeTask https://review.opendev.org/638046
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:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
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: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:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.opendev.org/638268
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 openstack Launchpad bug 1853453 in tempest "test_shelve_volume_backed_instance intermittently fails guest ssh with dhcp lease fail" [Undecided,New]
13:00:05 mriedem fun gate bug i finally wrote up after rechecking several times https://bugs.launchpad.net/tempest/+bug/1853453
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?
13:15:09 efried gibi: https://opendev.org/openstack/nova/src/branch/master/nova/scheduler/client/report.py#L787-L806 ish
13:15:22 gibi efried: thanks a lot!
13:15:42 gibi shilpasd: ^^

Earlier   Later