| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-18 | |||
| 18:36:42 | jaypipes | lol | |
| 18:36:44 | mriedem | so now it's just whatever the last thing i said | |
| 18:42:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: sort parameters for limits, quotas and quota classes https://review.openstack.org/528786 | |
| 18:44:47 | jaypipes | mriedem: where is the latest microversion for the compute API found? | |
| 18:45:06 | jaypipes | mriedem: in placement, it's a constant in the nova/api/openstack/placement/microversion.py file. | |
| 18:47:06 | artom | jaypipes, nova/api/openstack/api_version_request.py has _MAX_API_VERSION | |
| 18:47:14 | jaypipes | artom: ty sir! | |
| 19:02:15 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 19:22:31 | openstackgerrit | Chris Dent proposed openstack/nova master: Remove microversion fallback code from report client https://review.openstack.org/528794 | |
| 19:25:14 | cdent | that ^ may have some holes | |
| 19:30:50 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 19:43:15 | kumarmn | Could these changes by mriedem get merged? https://review.openstack.org/#/c/527228/ and https://review.openstack.org/#/c/527468/ | |
| 19:43:31 | kumarmn | They are holding me from tagging trove for queens-2. | |
| 19:43:56 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Make request_spec.spec MediumText https://review.openstack.org/528332 | |
| 19:44:34 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Make request_spec.spec MediumText https://review.openstack.org/528338 | |
| 19:45:03 | mriedem | kumarmn: need to find another core to +W | |
| 19:45:11 | mriedem | slim pickins right now | |
| 19:45:14 | mriedem | jaypipes: ? ^ | |
| 19:45:20 | mriedem | oomichi: ^ | |
| 19:45:55 | melwitt | oh, stable cores. I was like, what about me!! | |
| 19:45:58 | jaypipes | mriedem: I'm not a stable core. | |
| 19:46:08 | mriedem | artom: i rev'ed the newton and ocata backports to fix the git commit hash to point at the updated ocata/pike commits, respectively | |
| 19:46:16 | mriedem | jaypipes: no, not the stable patches | |
| 19:46:26 | mriedem | https://review.openstack.org/#/c/527228/ and https://review.openstack.org/#/c/527468/ | |
| 19:46:34 | mriedem | melwitt: you already +2ed them :) | |
| 19:46:53 | jaypipes | ah, ok. yeah, just a sec. | |
| 19:47:24 | melwitt | I too, thought they were the stable patches. too many links :D | |
| 19:48:57 | mriedem | melwitt: what if you were blue wagyu chopped liver? | |
| 19:49:02 | kumarmn | thanks mriedem, jaypipes, melwitt | |
| 19:49:39 | melwitt | that is a conundrum | |
| 19:52:54 | artom | mriedem, ah, thanks | |
| 19:54:34 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128 | |
| 19:55:45 | openstackgerrit | melanie witt proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance https://review.openstack.org/457886 | |
| 20:00:01 | jaypipes | mriedem: done | |
| 20:02:27 | mriedem | thanks | |
| 20:05:03 | mriedem | it the scheduler. i wonder if there is an easy way to make the cell conductor fail to hit the scheduler | |
| 20:05:03 | mriedem | edleafe: melwitt: i was thinking about something with the alternate hosts change during create server - https://review.openstack.org/#/c/511358/ - there is no functional test in that change that shows we're actually using the alternate hosts. we already have at least a couple functional tests which use 2 hosts and make the first fail to trigger a reschedule. but that worked before this because the compute and cell conductor c | |
| 20:05:12 | mriedem | in our functional tests we're just running one conductor service | |
| 20:06:18 | windsn | Hi, Im having issues with trying to use nova with ironic.... I see this error in the nova-compute logs, https://thepasteb.in/p/pghQo0OnW5vcR, and when i create an ironic node, i dont see the nova-hypervisor-list getting populated with an entry for the ironic node... I'm not too sure what seems to be the issue | |
| 20:06:55 | mriedem | windsn: ocata, pike, master? | |
| 20:07:01 | windsn | Pike | |
| 20:07:36 | mriedem | windsn: have you run "nova-manage cell_v2 discover_hosts"? | |
| 20:08:09 | mriedem | edleafe: melwitt: an easy think we could do is just make a select_destinations fixture which blows up if it's called more than once... | |
| 20:08:50 | windsn | mriedem: No I haven't run that... I'm not sure what that does | |
| 20:08:51 | mriedem | otherwise i think we'd need to do something where we start a 2nd conductor service with modified transport_url/db connection, and modify the compute service to point at the cell transport_url also | |
| 20:09:17 | mriedem | windsn: https://docs.openstack.org/nova/latest/cli/nova-manage.html#nova-cells-v2 | |
| 20:09:45 | edleafe | mriedem: I did think about doing something like that, but got tangled up in the multiple cell/conductor creation | |
| 20:10:14 | mriedem | edleafe: by default the tests are running with 2 cells, cell0 and cell1 (unless the SimpleCellFixture or whatever it's called is used) | |
| 20:10:36 | mriedem | so the trick would be running the cell conductor service and the compute service configured for the cell1 transport_url i think | |
| 20:10:49 | mriedem | but we use a fake mq so i'm not sure it would actually help anything | |
| 20:10:59 | edleafe | mriedem: ah, then I was trying for a third cell to show that it didn't touch the other cell | |
| 20:11:15 | edleafe | I guess I don't think of cell0 as a "real" cell | |
| 20:11:42 | mriedem | RPCFixture i think | |
| 20:12:23 | edleafe | mriedem: ok, I'll start playing around with that in a bit | |
| 20:13:13 | mriedem | looks like dan left a breadcrumb https://review.openstack.org/#/c/396417/23/nova/tests/fixtures.py | |
| 20:13:36 | cdent | dansel | |
| 20:14:10 | jaypipes | gdi, these server group functional tests are inscrutable. :( | |
| 20:14:17 | jaypipes | make me want to stab something. | |
| 20:14:23 | mriedem | anyway, just thought about this before digging into the ironic multinode reschedule failures | |
| 20:16:54 | jaypipes | AAAARRRRGGGGGGGGGGGGGGHHHHHHHHHHH I frigggin hate the integrate test helpers. | |
| 20:17:19 | melwitt | jaypipes: one thing I learned about those tests is if you want to have scheduling consider > 1 compute host, you need a compute service and set_nodes per host | |
| 20:18:05 | jaypipes | melwitt: just doing a simple copy/paste of one of the "negative" server group func tests results in completely random NotFound errors from the faked-out image service... | |
| 20:18:13 | jaypipes | melwitt: makes no sense whatsoever. | |
| 20:18:25 | jaypipes | melwitt: and the tests are completely unintelligible. | |
| 20:18:28 | melwitt | hm, yeah I would not have expected that | |
| 20:18:45 | melwitt | that, I do know from having used them as examples for other tests | |
| 20:19:02 | melwitt | (the multiple compute host part I mean) | |
| 20:20:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - Support aggregate affinity filters https://review.openstack.org/528804 | |
| 20:20:49 | jaypipes | jroll: they are stowed away in padded rooms. not to worry. | |
| 20:20:57 | jroll | excellent. | |
| 20:21:52 | jaypipes | melwitt: see the patch above... it will fail with this for no discernable reason whatsoever: http://paste.openstack.org/show/629242/ | |
| 20:22:01 | jaypipes | edleafe: very good question... | |
| 20:22:13 | jroll | ahem, projects | |
| 20:22:17 | jaypipes | lol | |
| 20:23:19 | jaypipes | jroll: well, disregarding my current functional test woes, your eyeballs on https://review.openstack.org/#/c/528804/1/nova/scheduler/filters/affinity_filter.py would be useful. | |
| 20:23:41 | jroll | jaypipes: sure thing, on my list | |
| 20:23:49 | jaypipes | ya, no rush.. | |
| 20:24:44 | windsn | mriedem: I ran the command ""nova-manage cell_v2 discover_hosts", but i still dont see the hypervisor-list getting populated with the ironic node | |
| 20:26:31 | windsn | mriedem: is the "nova-manage cell v2 discover_host" supposed to create a "host_mapping" entry in the nova db ? I don't see that happening | |
| 20:28:01 | mriedem | edleafe: ah i think i know why the ironic multinode job is failing to reschedule | |
| 20:28:02 | mriedem | "Re-scheduling is disabled" | |
| 20:28:20 | mriedem | if the retryfilter isn't running, then max_attempts = 0 and we wouldn't get alternates yeah? | |
| 20:29:20 | mriedem | jaypipes: likely using the latest microversion | |
| 20:29:26 | mriedem | so we don't proxy for images | |
| 20:29:36 | mriedem | you have to pass a specific image to _build_minimal_create_server_request i think | |
| 20:29:43 | mriedem | something like that | |
| 20:30:12 | mriedem | windsn: supposed to create a host_mapping entry in the nova_api db | |
| 20:30:42 | mriedem | jaypipes: yeah "GET /v2.1/6f70656e737461636b20342065766572/images/detail" status: 404 len: 78 microversion: 2.59 | |
| 20:31:07 | mriedem | that's capped at 2.35 | |
| 20:31:25 | jaypipes | mriedem: sorry, I'm not following you... | |
| 20:31:31 | melwitt | aha, yup. good thinkin mriedem | |
| 20:31:41 | mriedem | jaypipes: your test is using the latest microversion by default | |
| 20:31:51 | mriedem | or you've set self.microversion = 'latest' | |
| 20:31:59 | mriedem | the /images proxy API is capped at 2.35 | |
| 20:32:00 | jaypipes | mriedem: it's not *my* test :) it's a copy of one of the tests that is already in there... | |
| 20:32:04 | melwitt | as of that microversion, we no longer proxy image API requests to glance, so you're getting not found | |
| 20:32:12 | mriedem | anyway, you'll need to hard-code the image_uuid http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/functional/integrated_helpers.py#n140 | |
| 20:32:19 | mriedem | to one of the images in our fake image service | |
| 20:32:30 | jaypipes | mriedem: ah... since the test I copied from sets microversion to 2.15, it didn't bomb it. | |
| 20:32:44 | edleafe | mriedem: also, alternates are hosts, not nodes, so unless there are multiple computes per cell for ironic, there won't be alternates | |