| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 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 | bauzas | mriedem: I just wanted to explain that we could needing to work on that for Pike | |
| 21:42:58 | mriedem | dansmith: scheduler is global in cells v2 right? | |
| 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 | |
| 21:44:08 | bauzas | mriedem: so in a cellsV2 world, I need to lookup all cells | |
| 21:44:11 | mriedem | bauzas: then you do it via the cell mappings | |
| 21:44:20 | mriedem | there is already a check doing that same type of thing in nova-status | |
| 21:44:28 | mriedem | the RPs check | |
| 21:44:32 | bauzas | mriedem: yup, that's how we do currently for nova-status | |
| 21:44:42 | bauzas | mriedem: I saw that, hence my point above | |
| 21:44:48 | bauzas | mriedem: because the scheduler isn't doing that | |
| 21:45:10 | mriedem | yeah | |
| 21:45:14 | bauzas | I mean, we possibly have a lot of things that do ComputeNodeList.get_sth() | |
| 21:45:19 | bauzas | in the API at least | |
| 21:45:23 | bauzas | for hypervisors | |
| 21:45:24 | mriedem | well, this is why we have a caveat in the release notes about things just not working with multiple cells right now | |
| 21:45:48 | bauzas | okay, just wanted to make sure we keep that in mind for Pike | |
| 21:45:55 | mriedem | put it in the ptg etherpad | |
| 21:46:01 | bauzas | ack | |
| 21:46:05 | mriedem | https://etherpad.openstack.org/p/nova-ptg-pike | |
| 21:46:17 | bauzas | already in my tabs | |
| 21:46:23 | bauzas | 50+ | |
| 21:46:32 | mriedem | L55 and L57 are about cells v2 things - we should just throw those into their own cells v2 topics section | |
| 21:46:37 | dansmith | mriedem: yup | |
| 21:48:03 | mriedem | btw, kablammo http://logs.openstack.org/26/426926/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/04aad64/logs/grenade.sh.txt.gz#_2017-01-30_21_41_51_417 | |
| 21:48:14 | mriedem | ah crap | |
| 21:49:02 | mriedem | can we start placement before the nova services in grenade? | |
| 21:49:05 | mriedem | we should be able to right? | |
| 21:49:43 | mriedem | i.e. https://review.openstack.org/#/c/424730/18/projects/60_nova/upgrade.sh@86 before start_nova_api | |
| 21:49:56 | dansmith | should be able to | |
| 21:50:44 | bauzas | where is the gate for that change ? /me looking | |
| 21:51:23 | bauzas | arf, still there after 6 hours, wow | |
| 21:52:00 | mriedem | ok https://review.openstack.org/#/c/426926/2/projects/60_nova/upgrade.sh | |
| 21:52:39 | mriedem | should be fine since that's what the 55_placement was for | |
| 21:59:33 | openstackgerrit | Eric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | |
| 22:05:10 | openstackgerrit | Emilien Macchi proposed openstack/nova: DNM - tripleo CI test https://review.openstack.org/426949 | |
| 22:05:45 | mriedem | sean-k-mooney: do you know of anything that we need to get into os-vif before creating the stable/ocata branch? http://lists.openstack.org/pipermail/openstack-dev/2017-January/111135.html | |
| 22:08:39 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911 | |
| 22:08:39 | openstackgerrit | Matt Riedemann proposed openstack/nova: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 22:08:40 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add prelude section for Ocata https://review.openstack.org/426864 | |