| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-01 | |||
| 22:13:24 | jroll | else do the old crap | |
| 22:13:30 | mriedem | yeah basically | |
| 22:13:48 | bauzas | something like that indeed | |
| 22:14:01 | jroll | and in queens we make the extra spec thing mandatory and maybe even fix flavor to not need that | |
| 22:14:28 | mriedem | maybe | |
| 22:14:33 | mriedem | i can't think n+2 | |
| 22:14:39 | jroll | fair | |
| 22:14:42 | jroll | s/queens/future | |
| 22:14:59 | mriedem | by at least pike-eol :) | |
| 22:15:04 | jroll | regardless, pike looks like progress | |
| 22:15:06 | jroll | heh | |
| 22:15:13 | jroll | sounds good to me, thanks | |
| 22:15:27 | mriedem | solid progress either way i think | |
| 22:45:40 | mriedem | this is wrong right? https://review.openstack.org/#/c/408571/41/nova/api/openstack/compute/schemas/servers.py@277 | |
| 22:45:45 | mriedem | should be terminated_at? | |
| 22:51:18 | melwitt | mriedem: I think so. that's what the db column is called, anyway 'terminated_at' | |
| 22:51:34 | mriedem | yeah i'm going through https://review.openstack.org/#/c/421760/10/api-ref/source/parameters.yaml | |
| 22:58:34 | mriedem | fixing | |
| 23:03:03 | openstackgerrit | Ed Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | |
| 23:03:14 | edleafe | jaypipes: mriedem: ^^ tests should be fixed now | |
| 23:03:24 | jaypipes | edleafe: ty ed! | |
| 23:06:40 | mriedem | can you list servers with a query parameter with a 'none' value, to make it equate to NULL in the db query, i.e. GET /servers/?terminated_at=none | |
| 23:06:41 | mriedem | ? | |
| 23:07:27 | mriedem | i guess let's find out | |
| 23:18:15 | tonyb | cdent, mriedem: who did the placement stuff in devstack? and is the placement implementation/upgrade from mitaka -> newton -> ocata in devref documented | |
| 23:19:04 | mriedem | tonyb: cdent did the devstack patches, | |
| 23:19:12 | tonyb | mriedem: okay | |
| 23:19:34 | tonyb | cdent: shoudl expect strange questions in his future | |
| 23:19:40 | mriedem | and http://docs-draft.openstack.org/67/427367/3/check/gate-nova-docs-ubuntu-xenial/de76755//doc/build/html/placement.html | |
| 23:19:43 | mriedem | for devref | |
| 23:19:56 | mriedem | see 'deployment' and 'upgrade notes' | |
| 23:20:37 | tonyb | Thanks! | |
| 23:21:29 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the terminated_at field in the server query params schema https://review.openstack.org/427964 | |
| 23:21:31 | mriedem | oomichi: gmann: alex_xu: Kevin_Zheng: ^ | |
| 23:21:33 | mriedem | sdague: ^ | |
| 23:21:53 | mriedem | fwiw, filtering instances by those datetime fields doesn't seem to work anyway | |
| 23:22:18 | mriedem | i have an instance created at 2017-02-01T23:14:26Z according to nova show, but nothing is returned from: | |
| 23:22:25 | mriedem | curl -s -H "X-Auth-Token: $OS_TOKEN" http://9.5.124.95:8774/v2.1/servers?created_at=2017-02-01T23:14:26Z | |
| 23:24:10 | mriedem | oh interesting it's a regex | |
| 23:24:11 | mriedem | curl -s -H "X-Auth-Token: $OS_TOKEN" http://9.5.124.95:8774/v2.1/servers?created_at=^2017-02-01.* | |
| 23:24:13 | mriedem | ^ works | |
| 23:29:50 | armax | mriedem: is the large ops job running for nova changes? | |
| 23:29:59 | mriedem | armax: the large ops job doesn't run anymore | |
| 23:30:03 | mriedem | as far as i know | |
| 23:30:09 | armax | mriedem: ok, thanks for confirming this | |
| 23:30:13 | mriedem | which reminds me, i was going to kill that stuff from d-g | |
| 23:30:18 | armax | you mean https://review.openstack.org/#/c/228636/2/stackrc? | |
| 23:30:37 | mriedem | nope | |
| 23:30:47 | mriedem | there is some stuff in d-g about the large ops job | |
| 23:30:51 | armax | mriedem: ok | |
| 23:30:53 | mriedem | which i was going to clean out since we don't run that job anymore | |
| 23:31:15 | armax | what about that one? do we want to go back and keep a simpler API_WORKERS expression? | |
| 23:31:29 | armax | I am thinking of setting the minimum num of workers to 1 | |
| 23:31:39 | Kevin_Zheng | mriedem: so you are suggesting separate Reno with API ref patch? | |
| 23:31:46 | armax | that cuts the amount of RSS memory per service quite a bit | |
| 23:32:00 | armax | and should give us some headroom at least for now | |
| 23:32:01 | mriedem | Kevin_Zheng: at this rate, i think we'll probably want to decouple the release note from the rest of that change | |
| 23:32:04 | armax | what do you reckon? | |
| 23:32:27 | mriedem | armax: maximum you mean? | |
| 23:32:51 | mriedem | right now default in devstack is nproc / 2 right? | |
| 23:33:10 | Kevin_Zheng | mriedem: ok, will do it, holiday is over :) | |
| 23:33:22 | mriedem | armax: btw, i'm pretty sure that https://review.openstack.org/#/c/228636/ was all changed after that | |
| 23:33:53 | mriedem | although that was a long time ago so i might be mixed up on when those changes all happened, mtreinish might remember | |
| 23:34:29 | mriedem | Kevin_Zheng: cool. you'll probably want to check the comments/questions in parameters.yaml because it looks like some things are missing, but i might be confused about what's supposed to be in the server query parameter list | |
| 23:34:34 | mriedem | and if that's just sort keys or also filter keys | |
| 23:35:04 | mriedem | i assume that maybe some things were left out of the list in the api-ref if they are things that technically work but we don't want people using, like image_ref | |
| 23:35:08 | mriedem | gmann: ^ | |
| 23:35:40 | armax | mriedem: the outcome of the expression really depends on the number of processors devstack runs in | |
| 23:36:07 | mriedem | armax: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086232.html | |
| 23:36:16 | mriedem | for your historical enjoyment | |
| 23:36:20 | Kevin_Zheng | mriedem: ok | |
| 23:36:30 | armax | mriedem: thanks :) | |
| 23:37:01 | armax | mriedem: I am trying to figure out what’s the best strategy to set the API_WORKERS variable | |
| 23:37:26 | mriedem | armax: you should probably talk to sdague and mtreinish about that | |
| 23:37:35 | mriedem | since they were dealing with that heavily the last time we had an OOM issue | |
| 23:37:54 | armax | indeed, I was hoping I’d have a concrete proposal first | |
| 23:38:47 | armax | I don’t particularly fancy the idea of reducing the number of workers deliberately, but we’re in a bit of a bind | |
| 23:38:48 | mriedem | dude, | |
| 23:38:55 | mriedem | just wave your hands in the ML | |
| 23:39:14 | mriedem | maybe you'll hook someone :) | |
| 23:39:15 | armax | we did, but maybe not high enough | |
| 23:39:22 | mriedem | jump and wave at the same time | |
| 23:39:24 | mriedem | and scream | |
| 23:41:05 | armax | I’ll give it a shot | |
| #openstack-nova - 2017-02-02 | |||
| 00:00:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the terminated_at field in the server query params schema https://review.openstack.org/427964 | |
| 00:05:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: Handle NotImplementedError in _process_instance_vif_deleted_event https://review.openstack.org/427974 | |
| 01:22:09 | oomichi | mriedem: cool, nice catch | |
| 01:31:22 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 01:37:32 | mriedem | https://review.openstack.org/#/c/427782/ is merged | |
| 01:37:32 | mriedem | woot | |
| 01:37:41 | mriedem | hopefully we see that bug drop off | |
| 01:38:03 | mriedem | the postgres fix is merged too https://review.openstack.org/#/c/427667/ | |
| 01:40:08 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 01:46:12 | melwitt | mriedem: I had forgotten I said I'd write up step-by-step examples for setting up cells v2, so I did it just now. do you think it's too late for that? | |
| 01:53:24 | mriedem | no | |
| 01:53:37 | mriedem | link? | |
| 01:53:43 | mriedem | we really needed some upgrade steps | |
| 01:53:46 | mriedem | but i'll take anything | |
| 01:53:52 | melwitt | yay good. one sec, getting some sphinx warnings | |
| 01:54:28 | melwitt | "Title level inconsistent" I'm trying to figure out what level is one step smaller than ~~~~~ for a section | |