| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 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 | |
| 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: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 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:40 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add prelude section for Ocata https://review.openstack.org/426864 | |
| 22:09:49 | bauzas | mriedem: so, I'm about to leave, but I'll provide the nova-status check tomorrow morning | |
| 22:09:59 | mriedem | ok | |
| 22:10:32 | bauzas | mriedem: just a quick question, do you think I should use the current _check_placement() method or write a new one? | |
| 22:10:32 | mriedem | dansmith: question in https://review.openstack.org/#/c/424737/12/releasenotes/notes/ocata-cellsv2-support-4b3b5e70e76bc756.yaml | |
| 22:10:37 | mriedem | bauzas: new one | |
| 22:10:41 | bauzas | k | |
| 22:10:48 | mriedem | _check_enabled_filters or something | |
| 22:10:53 | diana_clarke | "barber-pole sorting", that's new to me | |
| 22:12:19 | dansmith | mriedem: I guess it doesn't seem important to me, given that none of the cells stuff is relevant before the upgrade is finished, but I'll say something like "once fully upgraded" or something | |
| 22:13:05 | mriedem | wfm, just don't forget to pull the latest version since both myself and bauzas have sullied your original change | |
| 22:14:04 | openstackgerrit | Eric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | |
| 22:22:23 | mriedem | melwitt: you might have thoughts on https://review.openstack.org/#/c/421420/7 | |
| 22:23:00 | melwitt | mriedem: thanks, will look through | |
| 22:30:28 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs: Send service token along with user tokens https://review.openstack.org/426933 | |
| 22:33:22 | mriedem | diana_clarke: were you going to address these comments? https://review.openstack.org/#/c/421444/6/doc/source/man/nova-manage.rst | |
| 22:33:46 | mriedem | the rest of the series lgtm | |
| 22:41:53 | diana_clarke | mriedem: sorry, I missed that one. I'll circle back tonight and address the feedback from melwitt too. | |
| 22:44:24 | mriedem | WHY GOD http://logs.openstack.org/26/426926/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/8b536f3/logs/grenade.sh.txt.gz#_2017-01-30_22_20_29_466 | |
| 22:45:01 | mriedem | oh sorry god, i know why | |
| 22:51:36 | openstackgerrit | Eric Fried proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | |
| 22:55:05 | mriedem | artom: you want to address the nits in https://review.openstack.org/#/c/424759/ or shall i? | |
| 23:42:02 | openstackgerrit | melanie witt proposed openstack/nova: Detach volume after deleting instance with no host https://review.openstack.org/340614 | |
| #openstack-nova - 2017-01-31 | |||
| 00:20:41 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |
| 00:34:43 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759 | |
| 00:39:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759 | |
| 00:40:31 | openstackgerrit | Matt Riedemann proposed openstack/nova: Pass APIVersionRequest to extensions https://review.openstack.org/425876 | |
| 00:43:29 | mriedem | artom: i cleaned up the bottom change, +2 on that one now | |