| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-01 | |||
| 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 | |
| 02:29:08 | mriedem | yeah, | |
| 02:29:45 | mriedem | i don't have a great solution to not be redundant but at the same time, have everything anyone would ever want or need to know in a single section and not make that so overly complicate with if/else conditions that no one can understand it | |
| 02:29:48 | mriedem | can't win really | |
| 02:30:07 | mriedem | i've always failed at process/guide docs like that | |
| 02:30:25 | mriedem | the balance between too much and too little info, because no matter how you write it, someone won't be able to follow it | |
| 02:30:34 | mriedem | lived with that for years in my last project before openstack | |
| 02:31:22 | melwitt | yeah. what's there already is a thorough explanation of all the things, but for the sort of roll up or quick reference, we don't have anything | |
| 02:31:41 | melwitt | maybe that page isn't the best place for it but I wasn't sure where else we would put it | |
| 02:33:19 | openstackgerrit | Hidekazu Nakamura proposed openstack/nova master: Do not attempt volume swap when guest is stopped https://review.openstack.org/389798 | |
| 02:41:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle NotImplementedError in _process_instance_vif_deleted_event https://review.openstack.org/427974 | |
| 02:51:21 | mriedem | melwitt: so i thought about this on the way home for some reason https://review.openstack.org/#/c/355091/ | |
| 02:51:34 | mriedem | i don't really want to open a can of worms, | |
| 02:51:56 | mriedem | but i'm not sure if what we have for placement right now resolves that with reporting 0 DISK_GB for boot from volume or not | |
| 02:52:03 | mriedem | i think it does | |
| 02:53:33 | melwitt | mriedem: yeah, I skimmed some ML threads on that and it wasn't clear to me if the boot from volume disk reporting problem is solved yet with placement | |
| 02:53:41 | mriedem | "We're already reporting proper allocation information for instances with no disk (volume-backed and others) and as soon as we look to the placement API in the scheduler we'll stop suffering from this problem." | |
| 02:53:58 | mriedem | ^ as long as the scheduler is using placement, we're filtering on DISK_GB now | |
| 02:54:56 | melwitt | mriedem: is it now? I've been way out of the loop on what's merged | |
| 02:56:01 | mriedem | melwitt: yeah https://review.openstack.org/#/c/417961/ | |
| 02:56:17 | mriedem | if the minimum nova-compute service version in the deployment is new enough, the filter scheduler will use placement now, | |
| 02:56:21 | mriedem | including filtering on DISK_GB | |
| 02:56:40 | mriedem | https://review.openstack.org/#/c/417961/41/nova/scheduler/filter_scheduler.py@139 | |
| 02:57:52 | melwitt | okay, cool | |
| 02:58:19 | melwitt | well, that's a win, glad to see that | |
| 02:58:52 | mriedem | https://review.openstack.org/#/c/427367/ adds some upgrade notes to the placement devref | |