Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-01
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 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.
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
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 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:10:28 lyarwood randomhack: it's the same underlying failure as this swap volume bug https://bugs.launchpad.net/nova/+bug/1739593
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
10:37:41 openstackgerrit Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552
10:37:42 openstackgerrit Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835
10:37:43 openstackgerrit Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487
10:58:06 openstackgerrit Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694
11:09:56 openstackgerrit Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526
11:11:05 jbernard melwitt: re the patch (yesterday, rbd busy snapshots), it's a valid rbd state, but tempest does not tolerate it
11:11:11 moshele stephenfin: hi can you review the pike backport https://review.openstack.org/#/c/563501/ ?
11:12:22 jbernard melwitt: the order of operations during teardown is non-deterministic
11:13:11 openstackgerrit Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921
11:17:32 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
11:24:28 kashyap melwitt: When you're back, mriedem is right about that extraCPU flags blueprint: a future patch needs to add the ability to remove specific CPU flags.
11:30:41 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:30:42 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_REALTIME_VERSION https://review.openstack.org/565707

Earlier   Later