| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-09 | |||
| 21:37:40 | smcginnis | melwitt: Up to you. I'm fine waiting awhile. | |
| 21:37:51 | mriedem | smcginnis doesn't have to go to china tomorrow | |
| 21:37:56 | melwitt | heh | |
| 21:38:23 | smcginnis | :) | |
| 21:40:02 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | |
| 21:40:09 | dansmith | that was a super nasty revert, fyi | |
| 21:40:15 | dansmith | so look at it with critical eyes | |
| 21:40:33 | melwitt | mriedem, get out soul crusher #3 | |
| 21:45:01 | melwitt | smcginnis: am I to include cycle-highlights in the patch? I see that was done for queens | |
| 21:45:08 | melwitt | *in the release patch | |
| 21:46:06 | prometheanfire | dansmith: thanks | |
| 21:48:21 | smcginnis | melwitt: Ideally, yes. Marketing type folks would love to have that. | |
| 21:48:31 | melwitt | ok, I will include them | |
| 21:48:31 | smcginnis | melwitt: It can be a follow up patch too though. | |
| 21:48:36 | melwitt | thanks | |
| 21:56:07 | mriedem | dansmith: ok done | |
| 21:56:17 | mriedem | the params stuff looks like it made that terrible | |
| 21:57:40 | dansmith | yes, yes it did | |
| 21:57:44 | dansmith | mriedem: you want a reno that says what exactly? | |
| 21:58:07 | mriedem | well we can revert this because the original bug is fixed with the new config option right? | |
| 21:58:13 | dansmith | that bug $orig was solved automatically but because of bug $new you must now enable $conf? | |
| 21:58:18 | mriedem | right | |
| 21:58:31 | dansmith | the original bug was arguably less bad than the current state | |
| 21:58:32 | dansmith | okay | |
| 21:59:17 | mriedem | the chances of anyone even having picked up that fix on stable already and be relying on it are pretty slim, at least for upstream, but you guys sound like you had at least one major customer that needed this | |
| 21:59:37 | dansmith | they did, but it never worked for them.. I think I now know why :) | |
| 22:01:15 | dansmith | mriedem: and are you asking me to actually clean up those tests here or just commenting about later reverts? | |
| 22:04:36 | melwitt | mriedem: rc1 release proposed https://review.openstack.org/590574 | |
| 22:05:18 | melwitt | I tried to pick the top highlights, let me know if I should add/remove based on your opinion | |
| 22:05:42 | mriedem | dansmith: just commenting, and that we can clean up that other unused stuff in the later separate revert | |
| 22:07:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | |
| 22:08:36 | dansmith | since we're doing this in rc2, and since melwitt will be up early tomorrow to ping him anyway, I assume we're going to wait for sahid's ack before putting this in? | |
| 22:08:47 | mriedem | melwitt: lgtm | |
| 22:09:07 | mriedem | yeah, also waiting on the recreate in my ci patch | |
| 22:09:12 | dansmith | yeah | |
| 22:09:36 | melwitt | yeah, let's talk to sahid tomorrow | |
| 22:15:08 | mriedem | and, | |
| 22:15:20 | mriedem | just wrapped up my plumbing thing | |
| 22:15:23 | mriedem | it's all coming together | |
| 22:15:44 | melwitt | granite all stars | |
| 22:16:17 | mriedem | well you see the granite tops come with a free sink but i need to know the dimensions for the plumber otherwise we needed to order our own which costs extra obviously and there is a time crunch and just omwoeoweitew | |
| 22:16:49 | melwitt | haha | |
| 22:21:24 | mriedem | https://review.openstack.org/#/c/553608/ | |
| 22:31:50 | openstackgerrit | Merged openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | |
| 22:31:59 | openstackgerrit | Merged openstack/nova master: Update ssh configuration doc https://review.openstack.org/589844 | |
| 22:35:09 | openstackgerrit | Merged openstack/nova stable/queens: [placement] Retry allocation writes server side https://review.openstack.org/588569 | |
| 22:38:59 | openstackgerrit | Merged openstack/nova stable/pike: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589251 | |
| 23:02:53 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for affinity with multiple cells https://review.openstack.org/585073 | |
| 23:02:54 | openstackgerrit | melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258 | |
| 23:06:36 | melwitt | guess I'll be holding off on stable releases because of the slow live migration issue | |
| #openstack-nova - 2018-08-10 | |||
| 00:15:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: wait for live migration vif plug short timeout https://review.openstack.org/590585 | |
| 01:58:54 | openstackgerrit | liuyamin proposed openstack/python-novaclient master: Replace os-client-config to openstacksdk https://review.openstack.org/590141 | |
| 02:05:41 | openstackgerrit | Merged openstack/nova master: Update compute rpc version alias for rocky https://review.openstack.org/589972 | |
| 03:06:10 | openstackgerrit | Chen proposed openstack/nova stable/ocata: Fix host validity check for live-migration https://review.openstack.org/590611 | |
| 03:10:58 | openstackgerrit | Chen proposed openstack/nova stable/ocata: Fix host validity check for live-migration https://review.openstack.org/590611 | |
| 03:21:11 | openstackgerrit | Chen proposed openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243 | |
| 05:04:39 | openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | |
| 05:50:19 | openstackgerrit | Rajesh Tailor proposed openstack/nova stable/ocata: Return 400 when compute host is not found https://review.openstack.org/590649 | |
| 05:56:51 | openstackgerrit | huanhongda proposed openstack/nova master: Destroy evacuated instance while unset nova-compute forced_down https://review.openstack.org/587807 | |
| 07:03:44 | mvpnitesh | hi all, i'm create a vm using Heat, the VM's are not getting created , they are failing with the below errors "ERROR nova.cells.messaging AttributeError: 'NoneType' object has no attribute 'identity'" and "{"message": "'NoneType' object has no attribute 'hosts'", "code": 500, "details": " File | | |
| 07:03:44 | mvpnitesh | | | \"/opt/stack/nova/nova/conductor/manager.py\", line 585, in build_instances | | |
| 07:03:44 | mvpnitesh | | | instance_uuids, | | |
| 07:03:45 | mvpnitesh | | | return_alternates=True) | | |
| 07:03:48 | mvpnitesh | | | File \"/opt/stack/nova/nova/conductor/manager.py\", line 720, in | | |
| 07:03:51 | mvpnitesh | | | _schedule_instances | | |
| 07:03:54 | mvpnitesh | | | scheduler_utils.setup_instance_group(context, request_spec) | | |
| 07:03:57 | mvpnitesh | | | File | | |
| 07:04:00 | mvpnitesh | | | \"/opt/stack/nova/nova/scheduler/utils.py\", line 836, in setup_instance_group " | |
| 07:18:25 | openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | |
| 07:20:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | |
| 08:20:39 | lyarwood | mdbooth: https://review.openstack.org/#/c/590439/ - simple bugfix if you have a second to review. | |
| 08:32:26 | mdbooth | lyarwood: I wonder... what happens if we detach a volume, the detach times out in the guest, then we do roll-detaching. Then the detach succeeds in the guest (after the timeout). If we try to detach again, will the second attempt to detach from the guest fail now that it's already detached? | |
| 08:32:38 | mdbooth | lyarwood: That doesn't impact your patch, btw | |
| 08:32:45 | mdbooth | I just thought of it | |
| 08:33:05 | lyarwood | iirc that raises NoDisk or something | |
| 08:33:10 | lyarwood | I think we catch that above | |
| 08:33:19 | lyarwood | and continue with the cinder removal | |
| 08:33:34 | lyarwood | DiskNotFound | |
| 08:33:36 | lyarwood | even | |
| 08:41:18 | openstackgerrit | huanhongda proposed openstack/nova master: Destroy evacuated instance while unset nova-compute forced_down https://review.openstack.org/587807 | |
| 08:59:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove patching the mock lib https://review.openstack.org/590709 | |
| 09:03:02 | mvpnitesh | hi all, i'm creating a VM using HEAT, i'm getting the below error "{"message": "'NoneType' object has no attribute 'hosts'", "code": 500, "details": " File | |
| 09:03:03 | mvpnitesh | \"/opt/stack/nova/nova/conductor/manager.py\", line 586, in build_instances | |
| 09:03:03 | mvpnitesh | instance_uuids, | |
| 09:03:03 | mvpnitesh | return_alternates=True) | |
| 09:03:05 | mvpnitesh | File \"/opt/stack/nova/nova/conductor/manager.py\", line 714, in | |
| 09:03:08 | mvpnitesh | _schedule_instances | |
| 09:03:13 | mvpnitesh | scheduler_utils.setup_instance_group(context, request_spec) | |
| 09:03:16 | mvpnitesh | File | |
| 09:03:19 | mvpnitesh | \"/opt/stack/nova/nova/scheduler/utils.py\", line 707, in setup_instance_group | |
| 09:03:22 | mvpnitesh | ||
| 09:03:25 | mvpnitesh | request_spec.instance_group.hosts = list(group_info.hosts) " | |
| 09:05:51 | lyaaaaaarwood | mvpnitesh: which version are you using? | |
| 09:06:28 | mvpnitesh | i'm using master and i faced the same issue on Queens also | |
| 09:07:11 | mvpnitesh | <lyaaaaaarwood>: Actually i'm creating a magnum cluster, i'm getting this error. If i create a normal instance using heat, i'm not getting any error | |
| 09:21:56 | lyaaaaaarwood | mvpnitesh: sorry, not sure why but request_spec.instance_group appears to be None | |
| 09:23:01 | lyaaaaaarwood | mvpnitesh: you can workaround this by adding a simple `if request_spec.instance_group and [..]` to the conditional at the end of setup_instance_group | |
| 09:23:11 | lyaaaaaarwood | mvpnitesh: btw can you write this up in a bug please? | |
| 09:24:12 | mvpnitesh | lyaaaaaarwood: Sure. I'm getting this error http://paste.openstack.org/show/727798/ in nova-cell-region and http://paste.openstack.org/show/727799/ in nova-cell-child | |
| 09:25:43 | lyaaaaaarwood | mvpnitesh: that's with the workaround? | |