| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-01 | |||
| 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 | |
| 01:59:15 | mriedem | ----- ? | |
| 01:59:56 | mriedem | melwitt: ^ | |
| 02:00:02 | mriedem | Cells V2 ======== Manifesto ~~~~~~~~~ Proposal -------- | |
| 02:00:13 | melwitt | yeah, I think you're right. thanks | |
| 02:03:17 | openstackgerrit | melanie witt proposed openstack/nova master: Add step-by-step examples for Cells V2 setup https://review.openstack.org/427981 | |
| 02:04:49 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 02:04:53 | melwitt | mriedem: ^ | |
| 02:05:22 | gmann | Kevin_Zheng: looks like you are back, updating doc patch as per mriedem comments ? | |
| 02:08:13 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Trivial Fix typo in schema https://review.openstack.org/427983 | |
| 02:08:44 | Kevin_Zheng | gmann: yeah I'm trying to | |
| 02:09:33 | gmann | Kevin_Zheng: thanks, please let me know any help. we should get those last bits in this week | |
| 02:10:32 | Kevin_Zheng | gmann: sure | |
| 02:10:35 | Kevin_Zheng | Thanks | |
| 02:15:15 | gmann | Kevin_Zheng: terminated_at is fixed in https://review.openstack.org/#/c/427964/ | |
| 02:15:55 | Kevin_Zheng | Yeah just saw it | |
| 02:16:23 | gmann | ok | |
| 02:20:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/426636 | |
| 02:21:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 02:21:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 02:22:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912 | |
| 02:22:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512 | |
| 02:22:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | |
| 02:22:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549 | |
| 02:22:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 02:24:05 | mriedem | melwitt: comments inline | |
| 02:24:21 | mriedem | melwitt: i worry a little about duplicating some of the stuff that dan wrote above that in the first time setup section | |
| 02:26:03 | melwitt | thanks | |
| 02:28:44 | melwitt | mriedem: I thought about that a little too. I was thinking we should probably have short "do these steps" things for people to look at to help understand the different commands. well, based on the convos we had with operators in the channel about it | |