Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-01
19:08:37 melwitt what I was missing is that there's no saving of state after a stack.sh so there's no way the teardown would "save" the fact that the services were created and enabled by the stack.sh run
19:21:02 openstackgerrit Jay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565
20:00:49 openstackgerrit Oliver Walsh proposed openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573
20:07:39 efried owalsh: Do we need a bug for ^ ?
20:13:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs https://review.openstack.org/565225
20:13:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578
20:15:07 mriedem efried: i thought about that, but it's not even tagged in rc1
20:15:21 mriedem so not really a good chance of someone hitting that besides red hat QE
20:15:54 efried mriedem: Sure, it's only been broken for a week, but still, paperwork.
20:16:03 efried mriedem: If you of all people don't need a bug, I'm sure not gonna push for it :)
20:16:38 owalsh awww but I love paperwork :-(
20:18:03 owalsh efried, mriedem: yea, I'm lazy and it's not released yet but can raise one if you think it's necessary
20:19:20 openstackgerrit Merged openstack/nova master: Add user_id to RequestSpec https://review.openstack.org/565340
20:20:56 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
20:20:57 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:20:58 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
20:34:01 melwitt hm, looks like all of the third party CIs suddenly are failing
20:35:00 mriedem all or just powervm?
20:35:04 melwitt hyperv CI can't resolve a hostname, some others like powervm and virtuozzo, the log files aren't found
20:35:14 mriedem i blame zuul
20:35:16 melwitt and it seems like this has happened starting today
20:43:52 openstack Launchpad bug 1633033 in OpenStack Compute (nova) "live migration with encrypted volume fails" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood)
20:43:52 randomhack_ was this bug ever solved? https://bugs.launchpad.net/nova/+bug/1633033. ran into this today on a Newton in
20:44:54 randomhack_ love migration of an instance with encrypted volume resulted in volume being mounted without decryption and corrupted the file system
20:45:42 randomhack_ s/love/live
20:46:49 randomhack_ known limitation of encrypted iscsi volumes?
20:48:15 mriedem randomhack_: lyarwood would know but it's really late for him (UK)
20:49:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425
20:49:11 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578
20:49:12 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs https://review.openstack.org/565225
20:49:24 randomhack_ mriedem: thanks i’ll check tomorrow earlier
20:49:32 melwitt yeah. IIRC, it will work as of pike when using qemu native luks decryption https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/libvirt-qemu-native-luks.html
20:49:59 randomhack_ melwitt: thanks!
20:50:04 melwitt because I thought lyarwood had a depends-on patch that ran live migration on top of the series
20:50:46 melwitt randomhack_: double check with lyarwood to make sure
20:52:25 melwitt this was one of the live migration test patches https://review.openstack.org/545074
20:53:06 mriedem melwitt: i think that was queens
20:53:28 mriedem yeah
20:53:28 mriedem https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/libvirt-qemu-native-luks.html :)
20:53:30 melwitt ah, you're right, it was queens
20:59:04 melwitt ceph job failing 100% with SnapshotIsBusy during a volume delete now. le sigh http://logs.openstack.org/49/479949/71/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/94fbc69/logs/screen-c-vol.txt.gz#_May_01_15_45_34_845542
21:00:30 melwitt there's even a note from you in the tempest test about this (I think?) and yet it's still happening https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L239
21:00:59 melwitt oh, wait, it's a TODO
21:01:33 melwitt I'll try to do the todo
21:01:38 mriedem it's a glorious todo
21:02:06 melwitt apparently. I don't know how this used to work before if there's an inherent issue with the volume dependencies in the test
21:02:32 melwitt I don't know volumes about volumes
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 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:31 melwitt and it can't delete the snapshot because of dependent things
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

Earlier   Later