| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-11 | |||
| 16:28:12 | jgwentworth | we have a review for increasing the max number of attached volumes to a single instance from 26 https://review.openstack.org/567472 | |
| 16:29:15 | jgwentworth | mdbooth or lyarwood: would appreciate your review on that ^ | |
| 16:30:29 | lyarwood | jgwentworth: ack yeah will have to be early next week now | |
| 16:30:47 | jgwentworth | np, no rush. just a heads up | |
| 16:31:30 | mdbooth | jgwentworth: Have you tested it, btw? What's the limit on the default storage controller? | |
| 16:32:55 | jgwentworth | mdbooth: uh, I have no idea? | |
| 16:34:13 | jgwentworth | I need to do some research to find out what makes sense there. I didn't know we currently limit to 26 until that patch | |
| 16:35:11 | jgwentworth | and I'm aware that sahid did some work with virtio-scsi to allow 256 devices. but that must be somehow separate from this, maybe those aren't volumes | |
| 16:35:57 | sean-k-mooney | finucannot: still around? | |
| 16:36:09 | finucannot | yup | |
| 16:36:25 | openstackgerrit | Merged openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | |
| 16:36:34 | sean-k-mooney | notice anything wrong with this https://github.com/openstack/os-vif/blob/master/tox.ini#L26 | |
| 16:37:33 | sean-k-mooney | ill give you a hint look at the python 3 tests below it | |
| 16:37:36 | finucannot | I'm assuming '.tests' shouldn't be in the regex? | |
| 16:37:47 | finucannot | (or we should drop 'test-path') | |
| 16:38:07 | sean-k-mooney | yep we are not running the unit test for the plugins in python 2 | |
| 16:38:16 | sean-k-mooney | for the last 10 months... | |
| 16:38:38 | finucannot | Boo, ralonsoh. Booooo | |
| 16:38:45 | sean-k-mooney | im going to fix this. this is is why i was not able to run the tests for the noop plugin | |
| 16:39:31 | sean-k-mooney | speaking of which ill finish that before i leave today but im guess you will be gone | |
| 16:39:44 | finucannot | I'll be around for another while yet | |
| 16:39:57 | finucannot | But if I am, I can review Tuesday for sure | |
| 16:40:02 | finucannot | (I'm gone Monday) | |
| 16:40:31 | sean-k-mooney | cool no worries i should have the review up in the next 15 mins or so | |
| 16:41:22 | sean-k-mooney | it was driving nuts trying to figure out why my test were not running lol i should start testing with py35 by default | |
| 16:52:15 | openstackgerrit | sean mooney proposed openstack/os-vif master: fix tox py27 job https://review.openstack.org/567895 | |
| 16:58:36 | PaulN | Hi, quick question, I'm trying to create a stack via Heat Templates containing a OS::Nova::ServerGroup in which I want to use the soft-anti-affinity policy which was added in nova compute api microversion 2.15. I can create it via the CLI by specifying --os-compute-api-version 2.15, but I can't create it via heat. I can't seem to find any information on how one specifies the nova api microversion from a Heat Template. | |
| 17:00:26 | PaulN | This in on an older OpenStack cluster that is only running Newton. | |
| 17:03:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899 | |
| 17:06:43 | jgwentworth | PaulN: I think you'll need to try the heat channel or tripleo channel. not sure anyone here will know how to specify microversions via heat templates | |
| 17:06:44 | PaulN | Yeah I have the Heat API version set to the value appropriate for Newton, but thats just the Heat Template version, doesn't really seem to have any correlation to what Nova API version is used. | |
| 17:06:57 | cfriesen_ | PaulN: based on https://bugzilla.redhat.com/show_bug.cgi?id=1447798 it seems heat support for nova microversions was only added in Ocata | |
| 17:07:06 | jgwentworth | nvm | |
| 17:07:08 | PaulN | Okay, I can try over in the heat channel. | |
| 17:07:51 | PaulN | Ugh, based on that seems I may be unable to use it via Heat on this cluster. | |
| 17:08:18 | openstack | bugzilla.redhat.com bug 1447798 in openstack-heat "soft-anti-affinity is not accepted by python-openstackclient but is with python-novaclient" [High,Closed: duplicate] - Assigned to zbitter | |
| 17:11:38 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899 | |
| 17:20:00 | prometheanfire | mriedem_lunch: the new keystonemiddleware breaks nova tests | |
| 17:20:03 | prometheanfire | http://logs.openstack.org/49/567449/1/check/cross-nova-py27/c2d43ef/testr_results.html.gz | |
| 17:35:10 | jgwentworth | cdent ^ you might understand what's going on there. only difference is the string quotes? | |
| 17:38:02 | cdent | jgwentworth: huh, wacky. | |
| 17:38:10 | jgwentworth | yeah | |
| 17:41:07 | cdent | jgwentworth: it's this https://review.openstack.org/#/c/559925/ | |
| 17:41:24 | jgwentworth | aaahhh | |
| 17:42:00 | cdent | so the test was relying on apparently not quite correct behavior by the middleware | |
| 17:42:20 | cdent | so it failing now makes sense | |
| 17:42:24 | jgwentworth | good find | |
| 17:42:26 | cdent | I guess we can just change the test to be a bit more flexible | |
| 17:42:44 | cdent | the format of the output isn't really the important part | |
| 17:42:54 | jgwentworth | true | |
| 17:43:06 | cdent | i'm sort of in the middle of the something at the moment, but can do it a bit later if someone reminds me? | |
| 17:43:19 | jgwentworth | yeah, np | |
| 17:45:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | |
| 17:50:34 | prometheanfire | mriedem: I'd like that :D | |
| 17:50:52 | mriedem | like what? | |
| 17:51:10 | openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | |
| 17:51:11 | mriedem | https://www.youtube.com/embed/bsB7UQ8BlE0 | |
| 17:51:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | |
| 17:51:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | |
| 17:51:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | |
| 17:51:29 | prometheanfire | mriedem: the config drive live migration | |
| 17:51:39 | mriedem | leakypipes: i want to see you yell that at your dogs ^ | |
| 17:51:42 | mriedem | cousins style | |
| 17:53:40 | leakypipes | mriedem: :) | |
| 17:56:27 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Fix HTTP500 error of changes-since on v2.0 API https://review.openstack.org/567916 | |
| 17:56:57 | cdent | jgwentworth: that thing I was in the midst of has utterly failed to do the right thing so needs to think about what it did, so I can fix the keystone thing now. Should I make a bug or just fix it or something else? | |
| 17:58:02 | jgwentworth | prometheanfire: do you have a bug for the cross-nova-py27 fail? cdent is gonna fix it | |
| 18:00:11 | prometheanfire | jgwentworth: I don't have a bug yet, it's just part of the release review | |
| 18:00:19 | prometheanfire | https://review.openstack.org/567449 | |
| 18:00:36 | jgwentworth | k | |
| 18:01:54 | jgwentworth | cdent: I'll open a bug | |
| 18:02:05 | prometheanfire | thanks | |
| 18:02:28 | cdent | thanks jgwentworth and prometheanfire . should have something pretty quick | |
| 18:03:56 | prometheanfire | ya, it didn't look bad | |
| 18:06:18 | jgwentworth | cdent: https://bugs.launchpad.net/nova/+bug/1770718 | |
| 18:06:20 | openstack | Launchpad bug 1770718 in OpenStack Compute (nova) "test_auth_middleware_factory fails with new keystonemiddleware 5.1.0" [Undecided,New] - Assigned to Chris Dent (cdent) | |
| 18:06:32 | cdent | thanks | |
| 18:06:46 | jgwentworth | thanks for fixing it :) | |
| 18:18:43 | openstackgerrit | Chris Dent proposed openstack/nova master: Flexibly test keystonmiddleware in placement stack https://review.openstack.org/567923 | |
| 18:18:57 | cdent | jgwentworth, prometheanfire ^ that oughta do the trick (and work for old and new) | |
| 18:19:46 | jgwentworth | saweet | |
| 18:20:47 | openstackgerrit | sean mooney proposed openstack/os-vif master: add noop plugin https://review.openstack.org/567924 | |
| 18:20:58 | prometheanfire | nice | |
| 18:32:18 | mriedem | blarg, if the libvirt driver raises RequestedVRamTooHigh during spawn we'll reschedule to at least 2 other nodes and fail on the same thign | |
| 18:32:19 | mriedem | *thing | |
| 18:33:59 | mriedem | also, double blarg, this is something we could validate in the API | |
| 18:34:36 | jgwentworth | yeah, I was going to ask, do we validate the other extra specs in the API or where | |
| 18:36:13 | openstackgerrit | Merged openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913 | |
| 18:39:14 | openstackgerrit | Merged openstack/nova master: Add connection_parameters to list of items copied from database https://review.openstack.org/567248 | |
| 18:42:26 | mriedem | jgwentworth: validating this in the api for server create and rebuild would be trivial, | |
| 18:42:51 | mriedem | only question about that is currently only the libvirt driver supports the image property and flavor extra spec, but i guess that just means - you shouldn't be using these if yo'ure not using libvirt, | |
| 18:43:09 | mriedem | and if you're doing mixed compute, make sure you use host aggregates to pin the image/flavor to libvirt hosts | |
| 18:45:03 | jgwentworth | you mean as far as where the checks belong in the api since they're driver-specific? | |
| 18:45:14 | jgwentworth | s/where/whether/ | |
| 18:47:11 | mriedem | kind of yeah | |
| 18:47:20 | artom | Expose it as traits? | |
| 18:47:21 | mriedem | my point being, the admin controls the flavors, | |
| 18:47:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't reschedule on RequestedVRamTooHigh errors https://review.openstack.org/567929 | |
| 18:47:38 | mriedem | and if the cloud doesn't have libvirt hosts to support this image property, then don't put that extra spec in the flavor | |
| 18:47:48 | mriedem | and if you have mixed compute, you would need to isolate using aggregates | |