| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-31 | |||
| 17:19:13 | cdent | avolkov: yeah, I think the change in topic made them harder to find | |
| 17:20:25 | openstackgerrit | Ed Leafe proposed openstack/nova master: Pass the number of hosts to return explicitly https://review.openstack.org/465767 | |
| 17:20:25 | openstackgerrit | Ed Leafe proposed openstack/nova master: Have scheduler claim resources https://review.openstack.org/463993 | |
| 17:20:26 | openstackgerrit | Ed Leafe proposed openstack/nova master: Return multiple hosts from select_destinations https://review.openstack.org/465768 | |
| 17:20:26 | openstackgerrit | Ed Leafe proposed openstack/nova master: Pass a list of instance UUIDs to scheduler https://review.openstack.org/465171 | |
| 17:27:00 | openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database https://review.openstack.org/469578 | |
| 17:28:57 | cdent | edleafe: did you see my comments on ps 11 of https://review.openstack.org/#/c/463993/ ? don't want them lost now that you've rebased | |
| 17:50:29 | edleafe | cdent: yes, I just wanted to clean things up first before making changes | |
| 17:51:03 | cdent | roger | |
| 17:52:25 | edleafe | cdent: I removed the patch that got rid of host_subset_size, so there was that | |
| 18:13:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use microversions for new style volume attachments https://review.openstack.org/469579 | |
| 18:13:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use 3.27 microversion when creating new style volume attachments https://review.openstack.org/469609 | |
| 18:13:51 | mriedem | ildikov: jgriffith: stvnoyes: ^ ok should be ready for review with tests | |
| 18:14:23 | mriedem | i'm going to push a DNM change on top of that to request something like 3.99 for attachment_create and see it raise the expected error so we can fallback to the old style attach | |
| 18:19:17 | cfriesen | where's the nova api code for boot-from-volume? | |
| 18:19:58 | mriedem | cfriesen: everywhere | |
| 18:21:49 | cfriesen | mriedem: okay let me rephrase that. I see that in the current release it's using server_controller. Is that basically true for earlier releases as well? | |
| 18:22:49 | cfriesen | mriedem: I'm looking at kilo code and only see one two mentions of 'os-volumes_boot' | |
| 18:23:09 | cfriesen | make that "two" | |
| 18:23:22 | mriedem | if POST /servers body has a "block_device_mapping_v2" entry in it, it's boot from volume | |
| 18:23:51 | mriedem | the compute api code translates and validates that to BlockDeviceMappingList | |
| 18:24:00 | mriedem | those get created in the cell in conductor along with the instance(s) | |
| 18:24:38 | mriedem | eventually _prep_block_devices in the compute manager create the volumes in cinder (if a volume id wasn't provided), waits for it to be available and then attaches it to the guest | |
| 18:25:14 | mriedem | the code for bfv and bdms is *everywhere* though | |
| 18:25:33 | cfriesen | and POST /os-volumes_boot ? I see that the ResourceExtension inherits 'servers', does that mean it uses api.openstack.compute.servers.Controller.create | |
| 18:28:30 | mriedem | for v2 | |
| 18:28:30 | mriedem | i think os-volumes_boot was some old extension thing | |
| 18:28:35 | mriedem | not a thing in v2.1 anymore | |
| 18:28:49 | mriedem | btw, bfv is right...behind....you! | |
| 18:29:14 | cfriesen | yeah, deprecated now. silly customers running old code | |
| 18:29:31 | mriedem | i guess i'm not following what your issue is | |
| 18:30:01 | mriedem | you have a user doing POST /os-volume_boot ? | |
| 18:30:24 | cfriesen | I'm trying to find the actual API code used for the POST /os-volumes_boot case. I think it ends up using the same create() routine as POST /servers with "block_device_mapping_v2" | |
| 18:30:38 | cfriesen | yeah, kilo code, they're doing POST /os-volume_boot | |
| 18:31:03 | mriedem | yes it's an extension on POST /servers | |
| 18:31:07 | mriedem | https://developer.openstack.org/api-ref/compute/ | |
| 18:31:07 | mriedem | you can't post to os-volumes_boot | |
| 18:31:21 | cfriesen | you could in Kilo apparently | |
| 18:32:09 | mriedem | ok looking at the route list in master you can too | |
| 18:32:40 | cfriesen | it's just explicitly using server_controller in master, which is a lot easier to follow | |
| 18:32:47 | mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/routes.py#L383 | |
| 18:32:59 | cfriesen | cool, I'm sorted out. thanks | |
| 18:33:14 | mriedem | i'm pretty sure we don't test that alias anywhere | |
| 18:33:16 | mriedem | btw | |
| 18:33:50 | cfriesen | I'll let them know they shouldn't be using it | |
| 18:34:01 | cfriesen | it's documented as deprecated | |
| 18:35:52 | mriedem | cfriesen: documented as deprecated in your product? | |
| 18:35:55 | mriedem | b/c it's not even mentioned in the api-ref | |
| 18:37:03 | cfriesen | mriedem: right, I was misremembering. It's os-volumes that is documented as deprecated | |
| 18:37:12 | mriedem | yeah that's the proxy api | |
| 18:43:46 | jaypipes | sfinucan: hmmm. "ListOfDictOfNullableStringsOrListField". you like that? :) | |
| 18:43:59 | dansmith | wat | |
| 18:44:16 | mriedem | List<Object> | |
| 18:44:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test requesting microversion for attachment_create and falling back https://review.openstack.org/469618 | |
| 18:44:20 | mriedem | just do that ^ | |
| 18:44:25 | mriedem | List<?> is even better | |
| 18:49:18 | jaypipes | mriedem: boost::variant<std::map<std::string, boost::optional<std::string>>, std::list<std::string>> | |
| 18:49:23 | jaypipes | mriedem: how's that? :) | |
| 18:51:22 | jaypipes | hehe | |
| 18:53:29 | mriedem | ship it | |
| 18:57:04 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 18:57:05 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 18:57:06 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | |
| 18:57:06 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | |
| 18:57:07 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 18:57:08 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 18:59:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test requesting microversion for attachment_create and falling back https://review.openstack.org/469618 | |
| 18:59:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use 3.27 microversion when creating new style volume attachments https://review.openstack.org/469609 | |
| 19:00:36 | jaypipes | mriedem, dansmith, melwitt: your reviews on https://review.openstack.org/#/c/461574/ and the remainder of the series would be awesome. tia. | |
| 19:01:13 | dansmith | jaypipes: I'll trade you for https://review.openstack.org/#/c/464059 | |
| 19:01:27 | jaypipes | dansmith: accepted. | |
| 19:04:41 | cdent | jaypipes, edleafe: sort of security fix for os-traits https://review.openstack.org/#/c/469625/ | |
| 19:04:47 | ildikov | mriedem: jgriffith: if that's the desired microversion direction we could try to put the attach PoC on top of that chain | |
| 19:04:53 | mriedem | ildikov: that's the idea | |
| 19:05:11 | mriedem | ildikov: https://review.openstack.org/#/c/469618/ is an example | |
| 19:05:19 | mriedem | but would eventually be jgriffith's poc | |
| 19:06:23 | jaypipes | cdent: ty ty ty! | |
| 19:06:49 | ildikov | mriedem: ah ok, missed that one, but that's what I had in mind too | |
| 19:07:35 | mriedem | jaypipes: cdent: i think we can test that with a devstack wip change before it's released | |
| 19:07:35 | ildikov | mriedem: glad we're on the same page :) | |
| 19:07:40 | mriedem | give me a few | |
| 19:07:46 | mriedem | jaypipes: which was your change that was failing on that again? | |
| 19:07:48 | jaypipes | mriedem: cool, that would be awesomesauce. | |
| 19:07:55 | jaypipes | mriedem: sec | |
| 19:08:06 | mriedem | bbq sauce will suffice | |
| 19:08:13 | jaypipes | mriedem: https://review.openstack.org/#/c/462770/ | |
| 19:08:17 | mriedem | ack | |
| 19:08:36 | mriedem | can we rebase that quick? | |
| 19:08:42 | mriedem | i'm going to use it as a depends-on | |
| 19:08:48 | mriedem | by we i mean you | |
| 19:08:57 | openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database https://review.openstack.org/469578 | |
| 19:09:18 | cdent | mriedem: my version of the same functionality has the same problem but is smaller ^ | |
| 19:10:27 | mriedem | cdent: ok | |
| 19:10:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell on local delete https://review.openstack.org/464059 | |
| 19:10:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment https://review.openstack.org/458537 | |
| 19:13:46 | edleafe | cdent: reviewed the pkgutil fix. One minor issue and I think it's good | |
| 19:13:54 | cdent | thanks edleafe | |
| 19:15:22 | cdent | dood, can I claim "unrelated change" and "didn't you write that?"? | |
| 19:15:56 | edleafe | cdent: sure, but since you're messing with the code... | |
| 19:16:22 | cdent | perhaps a follow up? | |