| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-01 | |||
| 19:01:21 | dansmith | but what I said is what I remember off the top of my head for that stuff | |
| 19:02:45 | melwitt | yeah, np, thanks for that. not looking for a deep answer, wish I had written a comment on here from back when someone explained it | |
| 19:05:18 | openstackgerrit | Eric Fried proposed openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 19:05:27 | melwitt | I think it might be because start_* and stop_* aren't going to happen in the same run (stack.sh vs unstack.sh) and there's not going to be any saving of state about the cell services between those runs | |
| 19:06:20 | melwitt | so during the unstack, you can't stop the per cell processes unless you enable them first | |
| 19:07:41 | dansmith | that's what I'm saying, | |
| 19:07:49 | dansmith | because we generate their service names from the cell we're in | |
| 19:07:52 | dansmith | n-cond-cell1, etc | |
| 19:07:55 | melwitt | right | |
| 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 | randomhack_ | was this bug ever solved? https://bugs.launchpad.net/nova/+bug/1633033. ran into this today on a Newton in | |
| 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: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 | https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/libvirt-qemu-native-luks.html :) | |
| 20:53:28 | mriedem | yeah | |
| 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 | 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 | |