Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-01
21:03:11 melwitt thanks
21:03:41 mriedem so in this test, we boot from volume and then create a volume-backed server snapshot, so we have an image with bdmv2 metadata pointing at the volume snapshot,
21:03:56 mriedem we then create a 2nd server from that image-defined bdm (the volume snapshot)
21:04:11 mriedem because delete_on_termination=True,
21:04:29 mriedem when we go to delete the 2nd server, the compute service will attempt to delete the 2nd volume which is the root disk for the 2nd server, created from the volume snapshot,
21:04:35 mriedem but cinder won't let you delete a volume that has snapshots,
21:04:47 mriedem so nova-compute will get an error from cinder and ignore it
21:05:14 melwitt hmm okay
21:05:35 mriedem in here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2422
21:05:53 mriedem you should see "Failed to delete volume" in the n-cpu logs for that test
21:06:12 melwitt it's weird because what's happening is it's the test trying to cleanup the volumes it had created,
21:06:31 melwitt and it can't delete the snapshot because of dependent things
21:06:31 mriedem http://logs.openstack.org/49/479949/71/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/94fbc69/logs/screen-n-cpu.txt.gz#_May_01_15_45_20_004599
21:06:45 melwitt it's not failing because of anything about n-cpu as far as I can tell
21:07:31 mriedem i believe jbernard had a really old patch for dealing with a failure to delete snapshots because they were busy with the cinder rbd driver
21:08:05 mriedem https://review.openstack.org/#/c/281550/
21:08:42 melwitt I remember this patch
21:09:28 melwitt I wasn't sure if this is an rbd-specific SnapshotIsBusy or if it's just because of how the test itself doesn't delete the volume first
21:11:50 openstackgerrit Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722
21:13:11 openstackgerrit Merged openstack/nova master: Use os.rename, not mv. https://review.openstack.org/562463
21:23:45 Swami I am trying to debug a PCI-Passthrough issue, what I am seeing is there are two pci-requests that are being sent to the compute to claim even though there is only one VM that is created. This happens when the first attempt to create the VM errors out and then when we retry creating an instance. [InstancePCIRequest(alias_name='intel10fb',count=1,is_new=False,request_id=None,spec=[{dev_type='type-PF',product_id='10fb',vendor_id='808
21:23:45 Swami 6'}]), InstancePCIRequest(alias_name=None,count=1,is_new=False,request_id=13befe5f-478f-4f4c-aa72-78cce84d942d,spec=[{dev_type='type-PF',physical_network='physnet2'}])]. The odd thing that I see between the two requests is one has a 'request_id' as None, but the other has a 'request_id' uuid.
22:11:52 openstackgerrit Merged openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043
22:13:26 melwitt so we have to backport that all the way to ocata? ^
22:14:01 openstackgerrit Jay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565
22:19:37 dansmith melwitt: I don't really think we do
22:19:46 dansmith it's pretty minor
22:19:47 dansmith IMHO
22:19:54 melwitt okay
22:19:56 dansmith that reminds me I need to look at the second patch there
22:21:27 melwitt that'll close out the blueprint, yeah?
22:22:14 melwitt (after it merges)
22:37:40 openstackgerrit Eric Fried proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573
22:39:08 efried mriedem: Quick re+A on this one ^
22:40:16 openstackgerrit Jay Pipes proposed openstack/nova master: WIP: Add a microversion for consumer generation https://review.openstack.org/565604
23:49:26 mriedem melwitt: would have to ask kashyap because there is a note in the patch about future patches
23:49:34 mriedem "(A future Nova patch will also allow ability to remove CPU flags.)"
#openstack-nova - 2018-05-02
00:26:13 openstackgerrit Merged openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926
00:37:57 openstackgerrit karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620
01:25:58 openstackgerrit jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689
01:47:37 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
02:00:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
02:00:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
02:08:10 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
02:17:20 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
02:22:15 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP new migration threads control https://review.openstack.org/563505
02:54:49 openstackgerrit Merged openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573
05:23:07 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
05:23:08 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
05:23:09 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
05:48:01 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
06:28:19 openstackgerrit jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897
06:39:35 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150
06:39:36 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572
06:52:14 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773
06:59:20 openstackgerrit inspurericzhang proposed openstack/nova master: Replace Chinese punctuation with English punctuation https://review.openstack.org/565655
07:04:24 openstackgerrit jichenjc proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.openstack.org/541152
07:22:51 gibi good morning nova
08:09:38 lyarwood randomhack: https://review.openstack.org/#/c/460243/ fixed this during Queens
08:10:28 lyarwood randomhack: it's the same underlying failure as this swap volume bug https://bugs.launchpad.net/nova/+bug/1739593
08:10:28 openstack Launchpad bug 1739593 in OpenStack Compute (nova) pike "[OSSA-2018-001] Swapping encrypted volumes can lead to data loss and a possible compute host DOS attack (CVE-2017-18191)" [High,Fix committed] - Assigned to Lee Yarwood (lyarwood)
08:11:52 lyarwood I can't mark it as a dup for some reason
08:19:18 kashyap stephenfin: Yes, I intend to handle those two clean-ups
08:19:26 kashyap Noted here: https://review.openstack.org/#/c/564012/4/nova/virt/libvirt/driver.py
08:20:36 openstackgerrit Kashyap Chamarthy proposed openstack/nova stable/queens: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565668
08:21:37 openstackgerrit Kashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565670
08:22:11 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
08:24:04 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
08:25:53 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
08:27:09 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
08:27:24 openstackgerrit Kashyap Chamarthy proposed openstack/nova stable/ocata: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565672
08:29:36 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
08:43:07 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
08:43:08 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
08:43:09 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
08:43:10 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
08:43:11 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
08:55:52 kashyap stephenfin: Ah, you've already posted those two
08:56:37 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
09:01:38 kashyap dansmith: We need to backport "libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043" back to Ocata, or else the existing documention would be a "lie".
09:02:56 lyarwood kashyap: argh my bad, I did mean to review these on Monday
09:03:12 kashyap lyarwood: No problem; I only _just_ backported that
09:03:22 kashyap As the main change got merged while I was asleep
09:03:28 kashyap So you're right on time
09:03:33 kashyap Thanks!
09:03:43 lyarwood kashyap: huh, maybe you just said they'd be coming soon then :)
09:03:51 lyarwood kashyap: and ack, I'll get to them today
09:04:36 kashyap lyarwood: Thanks; you'll see the changes needing reviews here: https://review.openstack.org/#/q/topic:bug/1750829+(status:open+OR+status:merged)
09:07:32 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041
09:39:20 sahid jaypipes: if you have a moment, I have a fix here, it's about how we compute memory we could be in situation where guests get oomkilled
09:39:25 sahid https://review.openstack.org/#/c/532168/
09:48:33 openstackgerrit Michael Still proposed openstack/nova master: Convert fping API to processutils.execute() https://review.openstack.org/564144
09:48:34 openstackgerrit Michael Still proposed openstack/nova master: Convert xenapi's xvp console to processutils. https://review.openstack.org/564145
09:48:36 openstackgerrit Michael Still proposed openstack/nova master: Convert certificate generation to processutils. https://review.openstack.org/564146
10:15:21 openstackgerrit sahid proposed openstack/nova master: compute: introduce cpu_shared_set option https://review.openstack.org/561912
10:15:22 openstackgerrit sahid proposed openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913
10:15:23 openstackgerrit sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set https://review.openstack.org/510897
10:35:21 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
10:35:22 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
10:35:23 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438

Earlier   Later