| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-24 | |||
| 19:37:37 | dansmith | one of those | |
| 19:37:58 | mriedem | which would also impact the build_and_run_instance method in conductor rpc api | |
| 19:38:11 | mriedem | *build_instances | |
| 19:38:35 | dansmith | if we put it into something like reqspec, we won't be able to send them to older computes | |
| 19:38:41 | dansmith | which will break our upgrade process | |
| 19:38:46 | dansmith | because they'll kick the newer version back | |
| 19:39:00 | mriedem | personally i think it's cleaner as a new parameter on the rpc api | |
| 19:39:08 | dansmith | which I guess is the same for the new param approach | |
| 19:39:26 | dansmith | we just need to handle the case in the retry logic, | |
| 19:39:35 | dansmith | if we didn't get these new things, assume we can talk to the scheduler and do a reschedule | |
| 19:40:18 | mriedem | but if your cell conductor is blocked from up calls to the scheduler, how would that work? | |
| 19:40:31 | dansmith | if you have old computes, | |
| 19:40:39 | dansmith | then you don't have a multi-tier cellsv2 environment, | |
| 19:40:43 | dansmith | because we didn't support it before, | |
| 19:40:46 | openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 19:40:46 | dansmith | thus it must be fine | |
| 19:41:11 | mriedem | idk, we don't really have anything doc'ed for this | |
| 19:41:21 | dansmith | eh? | |
| 19:41:24 | dansmith | it can't work right now | |
| 19:41:29 | dansmith | no need to doc it :) | |
| 19:41:40 | mriedem | what does old computes have to do with multi-tier cells v2? | |
| 19:42:13 | dansmith | can we discuss on a hangout? I'm about out of steam and have to move onto something else soon | |
| 19:42:14 | dansmith | be quicker if we can hash it out like that I think | |
| 19:42:25 | mriedem | i'm fine with a hangout | |
| 19:42:36 | jaypipes | me too. | |
| 19:43:04 | dansmith | https://hangouts.google.com/call/pjno3bssgba33a343nf47d357iu | |
| 19:52:38 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711 | |
| 19:53:47 | openstackgerrit | Merged openstack/nova master: Remove an unnecessary argument in _prep_resize https://review.openstack.org/486521 | |
| 19:54:35 | openstackgerrit | Merged openstack/python-novaclient master: Expect id and disabled_reason in GET /os-services response https://review.openstack.org/485409 | |
| 20:09:13 | openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257 | |
| 20:10:51 | efried | mriedem mordred os-service-types didn't get released last week. If it gets released soon, is it too late to get it through and use it in nova for pike? | |
| 20:11:56 | mriedem | efried: i haven't reviewed the nova code so can't really say what the risk is, or if this is all disabled by default and then people have to opt-in, or what | |
| 20:12:49 | efried | mriedem I was about to crank back up on the nova side, now that all the ksa stuff has landed. But one of the main pieces I need is os-service-types. If I can't have that, I'm going to have to basically inline a bunch of it. | |
| 20:13:06 | efried | mriedem os-service-types is brand new, so there's no risk of breaking backward compatibility or whatever. | |
| 20:13:22 | efried | mriedem It'll be a brand new dep, for whatever that means. | |
| 20:27:50 | mriedem | edleafe: so let's hold off on the alternatives stuff for pike, it's too high risk at this point in the schedule, we'll work on that for queens. it means you can't do multi-tier multi-cell with retries, but if you don't care about retries, like CERN, then you're still golden for pike. | |
| 20:28:00 | mriedem | edleafe: so focus on the ironic flavor migration stuff for pike | |
| 20:30:15 | edleafe | mriedem: ok | |
| 21:04:32 | openstackgerrit | Jeroen van Bemmel proposed openstack/nova master: Closes-Bug: 1702475 https://review.openstack.org/486753 | |
| 21:04:33 | openstack | bug 1702475 in OpenStack Compute (nova) "IPv6 data missing from latest/metadata info" [Medium,Confirmed] https://launchpad.net/bugs/1702475 | |
| 21:05:22 | openstackgerrit | Jeroen van Bemmel proposed openstack/nova master: Closes-Bug: 1702475 https://review.openstack.org/486753 | |
| 21:31:41 | openstackgerrit | Merged openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 21:32:24 | mriedem | cdent: the thing that your wsgi-intercept patch failed on failed in another unrelated change http://logs.openstack.org/02/485602/6/check/gate-nova-tox-functional-ubuntu-xenial/edf4c41/testr_results.html.gz | |
| 21:32:29 | mriedem | so probably just some new fun | |
| 21:32:45 | cdent | le sigh | |
| 21:32:58 | cdent | i have the OSAPIFixture using wsgi-intercept in progress | |
| 21:33:10 | cdent | I might stack them and see if that gets us anywhere | |
| 21:42:45 | openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 21:51:37 | mriedem | think i know what's causing the spike in the functional tests failing | |
| 21:51:46 | mriedem | https://review.openstack.org/#/c/484154/2/nova/tests/functional/api/openstack/placement/gabbits/resource-class-in-use.yaml | |
| 21:52:00 | mriedem | i think we have gabbits racing that work against the same custom resource class | |
| 21:52:46 | dansmith | they use the same db instance in parallel? | |
| 21:52:57 | mriedem | creating the same custom resource class, | |
| 21:53:17 | mriedem | in one case it already exists i think so it returns 204 | |
| 21:53:26 | mriedem | which fails the assertion for a 201 | |
| 21:53:51 | mriedem | https://bugs.launchpad.net/nova/+bug/1706207 | |
| 21:53:52 | openstack | Launchpad bug 1706207 in OpenStack Compute (nova) "resource-class-in-use_delete_resource_class fails with "AssertionError: '404' not found in ['204']" since 7/22" [High,Confirmed] | |
| 21:54:33 | dansmith | oh, the same placement fixture I guess? | |
| 21:54:38 | mriedem | yeah | |
| 21:54:41 | dansmith | I see | |
| 21:57:31 | mriedem | although these tests should be using isolated sqlite dbs | |
| 21:57:35 | mriedem | so i'm not sure why this should happen | |
| 22:00:23 | mriedem | exists." | |
| 22:00:23 | mriedem | "On a successful create return 201. Return 204 if the class already | |
| 22:00:37 | mriedem | not sure why the test is doing a PUT instead of a POST | |
| 22:01:16 | openstackgerrit | Dan Smith proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 22:01:16 | mriedem | i guess because the PUT handles both cases | |
| 22:01:46 | dansmith | jaypipes: can haz: https://review.openstack.org/#/c/484949 | |
| 22:02:16 | dansmith | mriedem: well, I thought you meant we're hitting the wrong fixture | |
| 22:02:29 | dansmith | like starting two but always hitting the first one from the tests or something | |
| 22:05:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | |
| 22:06:13 | dansmith | mriedem: I'll fix that since I have it open | |
| 22:06:46 | openstackgerrit | Dan Smith proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 22:07:39 | jaypipes | dansmith: done | |
| 22:07:54 | dansmith | jaypipes: thanks | |
| 22:08:00 | jaypipes | pas de probleme | |
| 22:08:13 | dansmith | "done" about sums it up for me too | |
| 22:08:18 | mriedem | https://review.openstack.org/#/c/459493/ closes out the searchlight notifications bp | |
| 22:08:19 | jaypipes | heh | |
| 22:12:11 | openstackgerrit | Merged openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473 | |
| 22:12:42 | openstackgerrit | Merged openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474 | |
| 22:13:11 | openstackgerrit | Merged openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484 | |
| 22:14:52 | mriedem | so there is one other set of gabbits that create a CUSTOM_GOLD resource class | |
| 22:15:10 | mriedem | but they both use the APIFixture which creates separate db fixtures | |
| 22:15:14 | mriedem | so i'm not sure how those would overlap | |
| 22:15:20 | mriedem | unless it's eventlet again | |
| 22:18:34 | mriedem | when this failed, both tests were running on the same process | |
| 22:18:35 | mriedem | so maybe | |
| 22:22:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests https://review.openstack.org/486782 | |
| 22:22:10 | mriedem | see if you can stomach this ^ | |
| 22:23:02 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests https://review.openstack.org/486782 | |
| 22:25:11 | openstackgerrit | Merged openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475 | |
| 22:25:46 | openstackgerrit | Merged openstack/nova master: Update ironic feature matrix https://review.openstack.org/486148 | |
| 22:31:41 | mriedem | melwitt: simple issue in https://review.openstack.org/#/c/410946/ | |
| 22:31:55 | mriedem | and then i'll +2 before dansmith is completely out | |
| 22:32:38 | dansmith | mriedem: worth trying that rc change right? | |
| 22:33:35 | dansmith | mriedem: jay is out so unless melwitt responds soon and is willing, we might as well slam that in | |
| 22:34:06 | mriedem | i'll update it | |
| 22:34:34 | dansmith | mriedem: I mean your resource class gabbit thing | |
| 22:34:38 | mriedem | oh | |