| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-05 | |||
| 22:56:56 | openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | |
| 22:57:01 | dansmith | mriedem: dtantsur|afk ^ | |
| 22:57:07 | jaypipes | efried: as much as they just inevitably end up reinforcing my annoyance with orchestration. | |
| 22:57:13 | dansmith | needs a reno but I'm out of brain power and the smoke is cutting my oxygen supply | |
| 22:57:49 | efried | cdent jaypipes So what we're talking about here is that we made an architectural call to be able to take a chunk of the placement response and just blat it into a (single) allocation request; but if we've made multiple calls to placement we'll have multiple such chunks, and there's currently no semantic for "combining" them into a single allocation request. | |
| 22:58:00 | efried | Did I get that right? | |
| 22:59:03 | cdent | jaypipes: sadly, somewhere is going to have to have a model for that tosca thing for an instance doign nested rp stuff and it is going to need to land on a compute (so the instance can build correctly). we planned ourselves into this corner, is just the way the world is for now :( | |
| 22:59:40 | cdent | efried: yes, pretty much. we’d need to build that “reassembler” in the scheduler and in a perfect world wouldn’t have to | |
| 23:00:10 | efried | In practical terms, the "opaque" allocation request is just a list of things, and we would just append those lists together and be fine. We just didn't wanna have to do that. | |
| 23:00:12 | cdent | we now need to be smart in at least two spots | |
| 23:00:53 | cdent | efried: not exactly. | |
| 23:01:11 | cdent | If we are lisp coders and are talking about this problem, then yes, we building lists | |
| 23:01:24 | cdent | but the selection of pieces is not just reassambling a sequence | |
| 23:03:53 | efried | jaypipes cdent I gotta run. FYI, I've been assembling notes which I eventually planned to link off of the main PTG etherpad once they were in a state where they were sanely readable by someone other than me. I'm not sure if we've reached that point yet, but... https://etherpad.openstack.org/p/nova-ptg-queens-generic-device-management | |
| 23:04:21 | cdent | thanks for doing that efried, you want annotations in the realm of “nowish” or “laterish”? | |
| 23:06:03 | efried | cdent I guess any-time-ish is fine, thanks. I didn't think I was done with it for sure, but I believe I've at least removed most of my horribly-misinformed early thoughts/ideas. | |
| 23:06:22 | cdent | ✔ | |
| 23:07:06 | efried | Thanks as always for talking through this with me jaypipes cdent dansmith sean-k-mooney | |
| 23:09:46 | jaypipes | ciao | |
| 23:28:41 | gmann | mriedem, +1, i overlooked | |
| 23:30:11 | gmann | mriedem, can we have a specless BP for index schema chages - https://review.openstack.org/#/c/500347/ https://review.openstack.org/#/c/499091/ etc | |
| 23:30:30 | gmann | mriedem, that will be basically continuation of this - https://blueprints.launchpad.net/nova/+spec/consistent-query-parameters-validation | |
| 23:30:54 | gmann | it will be easy to track and capture any accidental API changes | |
| 23:37:00 | gmann | mriedem, created one, check if it looks fine - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-index-query-param | |
| 23:37:06 | gmann | alex_xu, ^^ | |
| 23:39:02 | alex_xu | gmann: thanks, that's great | |
| 23:47:07 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add a spec for POST /allocations in placement https://review.openstack.org/499259 | |
| 23:50:22 | openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | |
| 23:51:57 | openstackgerrit | Merged openstack/nova master: Add recreate test for forced host evacuate not setting dest allocations https://review.openstack.org/499678 | |
| #openstack-nova - 2017-09-06 | |||
| 00:33:33 | mriedem | gmann: these don't require microversion changes, correct? | |
| 00:34:00 | openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling error availiable change to available https://review.openstack.org/501043 | |
| 00:34:58 | gmann | mriedem, yes. only thing i want to confirm from alex_xu about restricting the int convertible string as limit like '1' it used to be valid and converted by utils previously and now it will be 400 | |
| 00:35:26 | gmann | i think we discussed it in original spec but i do not remember the consensus . | |
| 00:36:17 | mriedem | gmann: if a microversion bump is required then i think we need a spec, | |
| 00:36:22 | mriedem | otherwise i'm ok with specless | |
| 00:37:06 | gmann | mriedem, yea, if so it need spec. we will discuss it in today meeting for all cases and ll update you | |
| 00:37:22 | mriedem | thanks | |
| 00:38:10 | openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake availiable change to available https://review.openstack.org/501045 | |
| 00:41:29 | openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake prefered change to preferred https://review.openstack.org/501046 | |
| 00:43:22 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: WIP spelling error availiable change to available https://review.openstack.org/501043 | |
| 00:47:27 | openstackgerrit | wanghongtaozz proposed openstack/nova stable/pike: spelling mistake intergration change to integration https://review.openstack.org/501047 | |
| 00:47:54 | alex_xu | mriedem: gmann it needn't microversion, I think just just add query params validation for the exist API and keep it same behavour for the API | |
| 00:49:02 | gmann | alex_xu, but we are doing non negative integer for limit - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/keypairs.py#L109 | |
| 00:49:21 | gmann | https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/validation/parameter_types.py#L438 | |
| 00:50:00 | gmann | alex_xu, this is only case change the behavior for 'int' 200 -> 400 | |
| 00:50:01 | alex_xu | gmann: that isn't backward-compatible? | |
| 00:50:26 | alex_xu | gmann: I remember there are some validation in the python code for the limits, that should check the non-negative value also | |
| 00:51:06 | gmann | alex_xu, humm | |
| 00:51:15 | gmann | alex_xu, i found the flow like this | |
| 00:51:16 | gmann | 1. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L189 | |
| 00:51:30 | gmann | 2. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L202 | |
| 00:52:10 | gmann | 3. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L883 | |
| 00:52:26 | gmann | 3rd one convert the '1' to 1 | |
| 00:54:23 | alex_xu | gmann: there is parameter 'min_value', that is limited to non-negative? | |
| 01:02:00 | openstackgerrit | Merged openstack/nova master: tests: Remove useless test https://review.openstack.org/483031 | |
| 01:02:53 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certificates to REST API https://review.openstack.org/486204 | |
| 01:06:47 | openstackgerrit | Merged openstack/nova master: Cleanup allocations on invalid dest node during live migration https://review.openstack.org/498861 | |
| 01:08:21 | alex_xu | gmann: I may not clear the question | |
| 01:08:34 | alex_xu | the value of '1' will get 400? | |
| 01:09:12 | openstackgerrit | Merged openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287 | |
| 01:14:59 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | |
| 01:15:00 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | |
| 01:15:28 | cdent | dansmith: some of the jiggery pokery in that ^ is starting to get a bit wack | |
| 01:15:32 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param https://review.openstack.org/500396 | |
| 01:38:20 | openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | |
| 02:05:09 | openstackgerrit | jichenjc proposed openstack/nova master: conf: Remove quota_networks options https://review.openstack.org/501068 | |
| 02:21:10 | openstackgerrit | Chenghui Yu proposed openstack/nova master: Remove usage of kwarg retry_on_request in API https://review.openstack.org/501073 | |
| 04:43:58 | openstackgerrit | Pooja Jadhav proposed openstack/nova master: Fix ValueError if invalid max_rows passed to db purge https://review.openstack.org/500771 | |
| 05:03:05 | itlinux | there are 4 compute nodes.. and those images are cirros.. | |
| 05:11:03 | itlinux | hello team.. | |
| 05:11:29 | itlinux | looks like the scheduler gets stuck when I try to spin up few machines at one time.. | |
| 05:11:33 | itlinux | like 4 or 5 | |
| 05:11:44 | itlinux | I have 4 compute nodes and the images are cirros | |
| 05:11:54 | itlinux | I checked the hypervisors are up | |
| 05:47:02 | openstackgerrit | jichenjc proposed openstack/nova master: Update doc to indicate nova-network deprecated https://review.openstack.org/500654 | |
| 05:49:03 | openstackgerrit | jichenjc proposed openstack/nova master: conf: Remove quota_networks options https://review.openstack.org/501068 | |
| 05:49:31 | gmann | alex_xu, i mean, if we pass any int as string in limit like '1' not 1, it will be 400 in current case of keypair. because of schema | |
| 05:49:44 | itlinux | hello nova team.. anyone has a tweak since when I spin up some vms the compute that gets like two vms takes forever.. | |
| 05:49:53 | itlinux | like 100sec for a cirros image.. | |
| 05:50:00 | itlinux | sounds a little too much! | |
| 05:55:30 | gmann | alex_xu, but when i ran it without schema it is also same(400). cannot this convert '1' -? 1 - https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L883 | |
| 06:04:53 | openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | |
| 06:09:08 | openstackgerrit | jichenjc proposed openstack/nova master: Enhance doc for nova services https://review.openstack.org/499536 | |
| 06:09:30 | openstackgerrit | jichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError https://review.openstack.org/469779 | |
| 06:11:30 | alex_xu | gmann: that schema should work with '1', not sure I missed something | |
| 06:13:43 | gmann | alex_xu, oh, 1 sec | |
| 06:19:22 | openstackgerrit | Merged openstack/nova master: Add uuid to migration object and migrate-on-load https://review.openstack.org/496934 | |
| 06:40:15 | openstackgerrit | Merged openstack/nova master: Add uuid online migration for migrations https://review.openstack.org/498589 | |
| 06:57:30 | openstackgerrit | Gao Zexu proposed openstack/os-traits master: Update the documentation urls https://review.openstack.org/501122 | |
| 07:02:50 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for used_limits function https://review.openstack.org/499091 | |
| 07:09:59 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:Remove quota_networks and enable_network_quota https://review.openstack.org/501068 | |
| 07:11:11 | jichen | gmann: hi, in https://review.openstack.org/#/c/501057/2/nova/tests/unit/api/openstack/compute/test_config_drive.py, you mentioned :i like to have some uuid as project id here instead of removing. | |
| 07:11:49 | jichen | I think in v2.18, we have: | |
| 07:12:04 | jichen | 2.18 Establishes a set of routes that makes project_id an optional construct in v2.1. | |
| 07:12:20 | jichen | why we still need add project id in the test ? | |
| 07:21:40 | gmann | jichen, yea, but we do have tests to test both | |
| 07:22:28 | gmann | jichen, i do not have string opinion on this but i we can discuss whether it is ok to remove the project_id from tests | |
| 07:23:03 | gmann | jichen, its unit test only right? does it matter anywhere. i mean url is really dummy things in unit tests right | |
| 07:28:42 | openstackgerrit | Lajos Katona proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | |
| 07:34:55 | jichen | gmann: yeah, we should have test for both..I will see whether it's possible to at least include some project id into test | |