| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 20:33:03 | bauzas | mriedem: so I think operators would see that by the prelude | |
| 20:33:57 | dtp | anyone need anything? cup of coffee, etc | |
| 20:34:05 | mriedem | just had one thanks | |
| 20:34:14 | mriedem | bauzas: ok i guess i can rebase my reno on top of yours | |
| 20:34:51 | mriedem | bauzas: and your reno needs a rebase too because your change and dan's change below it are both adding something to the prelude about cells v2 | |
| 20:35:23 | bauzas | mriedem: fine by me, which rebase? | |
| 20:35:58 | mriedem | your change isn't on top of the latest version of https://review.openstack.org/#/c/424737/ | |
| 20:36:53 | bauzas | mriedem: okay | |
| 20:38:16 | dtp | ok. let me know if there's anything i can do | |
| 20:38:54 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911 | |
| 20:43:56 | mriedem | bauzas: i suppose we still want to get something into nova-status for warning if CoreFilter/RamFilter/DiskFilter aren't in filter_scheduler.enabled_filters... | |
| 20:44:31 | bauzas | mriedem: mmm, it could be nice, agreed | |
| 20:44:40 | bauzas | mriedem: I can write that if you want | |
| 20:45:09 | bauzas | mriedem: you mean, adverting to modify the ratios then ? | |
| 20:45:24 | mriedem | yeah something along those lines, | |
| 20:45:31 | mriedem | (1) is the filter_scheduler being used, | |
| 20:45:43 | mriedem | (2) are any of those things NOT in enabled_filters | |
| 20:46:07 | mriedem | (3) for each filter that's not enabled, check it's related allocation ratio and see if it's the default value and if so, issue a warning | |
| 20:46:08 | mriedem | something likethat | |
| 20:46:22 | bauzas | yeah, like I wrote in the note | |
| 20:46:33 | bauzas | very good point | |
| 20:46:52 | bauzas | mriedem: just one point, where operators are running the CLI ? | |
| 20:47:14 | mriedem | we don't know yet | |
| 20:47:23 | bauzas | mriedem: they need to do that in not in a computer, right? rather close to the other services, I guess | |
| 20:47:26 | mriedem | as we haven't integrated nova-status into the CI system yet | |
| 20:47:40 | mriedem | well, it's really, give me your full config | |
| 20:47:57 | mriedem | so api db, main db, compute config, scheduler config, all of it | |
| 20:48:02 | mriedem | and probably run it in a venv or container | |
| 20:48:23 | mriedem | so you can install and run it in isolation from the newton deployment code (which doesn't have the CLI) | |
| 20:49:16 | bauzas | ok | |
| 20:49:19 | mriedem | i'm thinking about writing a wip patch to grenade to run nova-status in the from-netwon script, after we'd done our necessary setup for ocata | |
| 20:50:30 | dansmith | mriedem: did you abandon that devstack patch that sets the db host to the primary? | |
| 20:50:39 | mriedem | yeah | |
| 20:50:42 | mriedem | i can find it | |
| 20:51:06 | mriedem | dansmith: this? https://review.openstack.org/#/c/421399/ | |
| 20:51:48 | dansmith | mriedem: yeah, can you un-abandon? I'll need that to start up a second cell on the subnode | |
| 20:52:48 | mriedem | done | |
| 20:53:03 | dansmith | thanks | |
| 20:53:05 | mriedem | this was my patch for creating a 2nd cell https://review.openstack.org/#/c/420976/ | |
| 20:53:08 | mriedem | which is out of date now | |
| 20:53:24 | mriedem | which also needed https://review.openstack.org/#/c/420973/ | |
| 20:53:50 | openstackgerrit | Ed Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | |
| 20:53:57 | dansmith | ah, okay, I hadn't seen that one | |
| 20:54:18 | edleafe | jaypipes: cdent: johnthetubaguy: ^^ fixed the functional test | |
| 20:55:05 | jaypipes | edleafe: danke | |
| 20:55:11 | bauzas | mriedem: dansmith: FYI, merging notes with both prelude sections are most of the time very difficult because they could be misgenerated | |
| 20:55:43 | bauzas | that's why we add the ">" section header | |
| 20:55:50 | mriedem | bauzas: so i guess take the prelude out of dan's and move it into yours | |
| 20:56:04 | dansmith | bauzas: okay, I added prelude text at mriedem's request | |
| 20:56:09 | dansmith | but yeah, feel free to take it | |
| 20:56:23 | bauzas | okay, just wanted to ask you folks before | |
| 20:56:34 | bauzas | and explain why reno is giving you -1 on its job | |
| 20:56:46 | bauzas | (well, guessing why) | |
| 20:57:15 | bauzas | yeah that http://logs.openstack.org/37/424737/10/check/gate-nova-releasenotes/20a3d9a/console.html#_2017-01-30_18_28_37_031390 | |
| 20:57:50 | dansmith | ack | |
| 20:59:13 | bauzas | dansmith: I'll push a new rev by deleting your prelude section and putting it in my patch | |
| 20:59:49 | dansmith | bauzas: s/deleting/stealing/ | |
| 20:59:50 | dansmith | sure | |
| 21:00:12 | dansmith | bauzas: you keep stealing american work and trump will put france on "the list" | |
| 21:00:30 | bauzas | dansmith: yeah, I'm stealing a job | |
| 21:00:44 | bauzas | but | |
| 21:01:06 | bauzas | our French beauty queen won a Trump meetup, so... | |
| 21:01:12 | dansmith | and it's after 1700 in france, so you're a criminal too! | |
| 21:01:22 | bauzas | poor me | |
| 21:02:05 | cburgess | lol | |
| 21:06:09 | diana_clarke | mriedem: Sorry, I meant linking to http://docs.openstack.org/developer/nova/cells.html for more information. And my internet keeps dropping on me, so I may have missed your response. | |
| 21:11:38 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 21:11:39 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add prelude section for Ocata https://review.openstack.org/426864 | |
| 21:12:14 | bauzas | there | |
| 21:12:16 | mriedem | diana_clarke: sorry i'm not following | |
| 21:12:39 | bauzas | I added a Co-Authored-By tag, so I won't be in jail when I am in ATL | |
| 21:13:07 | diana_clarke | mriedem: No doubt; that wasn't very clear of me ;) | |
| 21:15:42 | diana_clarke | mriedem: I'm wondering if I should add a few opening paragraphs to the cells v2 commands. Making it clearer which commands should be run when and in what order. | |
| 21:16:02 | diana_clarke | mriedem: Perhaps adding a high level bucket overview: http://paste.openstack.org/show/596936/ | |
| 21:16:17 | diana_clarke | linking to http://docs.openstack.org/developer/nova/cells.html etc | |
| 21:25:43 | mriedem | diana_clarke: oh, | |
| 21:25:56 | mriedem | diana_clarke: from the man page or from http://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v2 ? | |
| 21:26:15 | mriedem | i think we definitely want ^ pointing at the man pages | |
| 21:28:02 | diana_clarke | mriedem: I was talking about a few overview paragraphs at the top of the cells v2 sections here: doc/source/man/nova-manage.rst | |
| 21:28:13 | diana_clarke | section | |
| 21:28:34 | mriedem | diana_clarke: i'm a bit torn on that, not wanting to duplicate something there with what we have in http://docs.openstack.org/developer/nova/cells.html | |
| 21:29:08 | mriedem | diana_clarke: i think i'd prefer the more detailed big picture stuff in http://docs.openstack.org/developer/nova/cells.html and then link to the man pages for details on the individual commands | |
| 21:30:03 | mriedem | i've noticed the see also in http://docs.openstack.org/developer/nova/man/nova-manage.html#see-also - that, or something like it, should probably refer to http://docs.openstack.org/developer/nova/cells.html | |
| 21:30:27 | diana_clarke | mriedem: Cool, perhaps I'll work on updates to developer/nova/cells.html instead. Thanks! | |
| 21:31:03 | mriedem | sure, np. i'm not docs writer so i won't claim to know the right answer | |
| 21:31:09 | mriedem | s/not/no/ | |
| 21:31:58 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911 | |
| 21:32:55 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs: Send service token along with user tokens https://review.openstack.org/426933 | |
| 21:36:39 | bauzas | mriedem: dansmith: mmm, I had a thought related to cellsV2 | |
| 21:37:26 | bauzas | mriedem: dansmith: given we get the list of compute nodes in the scheduler when not having all the computes being Ocata, it could be a problem if we have more than 1 cell, right? | |
| 21:38:09 | bauzas | mriedem: dansmith: I mean, a general ComputeNodeList.get_all() | |
| 21:41:15 | mriedem | first thing that comes to mind is we aren't supporting multiple cells really in ocata | |
| 21:41:18 | mriedem | we don't test it | |
| 21:41:45 | mriedem | the scheduler will pull from whichever main database is setup in the nova.conf for the scheduler node | |
| 21:42:40 | bauzas | mriedem: right, that I know | |
| 21:42:58 | mriedem | dansmith: scheduler is global in cells v2 right? | |
| 21:42:58 | bauzas | mriedem: I just wanted to explain that we could needing to work on that for Pike | |
| 21:43:11 | bauzas | mriedem: that's something we agreed | |
| 21:43:46 | bauzas | mriedem: but for example, I'm writing the nova-status patch, and I need to lookup at the compute nodes to see their ratios | |
| 21:43:57 | mriedem | having a list of cellsv2-related things to work on in pike would be good for the ptg etherpad | |