| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 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 | |
| 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 | mriedem | dansmith: question in https://review.openstack.org/#/c/424737/12/releasenotes/notes/ocata-cellsv2-support-4b3b5e70e76bc756.yaml | |
| 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: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 | |
| 00:43:32 | mriedem | dansmith: ^ | |
| 00:54:11 | mriedem | jesus is this normal http://logs.openstack.org/06/425806/8/gate/gate-tempest-dsvm-cells-ubuntu-xenial/344170a/logs/screen-n-api.txt.gz?level=TRACE ? | |
| 01:21:25 | artom | mriedem, ah, much thanks! | |
| 01:21:36 | artom | I got your review but was busy with offsprings | |
| 01:21:37 | mriedem | artom: i'll have the tempest patch up shortly | |
| 01:21:51 | artom | All sleeping now, but you got ahead of me ^_^ | |
| 01:21:54 | mriedem | i'll have to deal with offspring in a bit | |
| 01:22:12 | mriedem | dreaded reading homework, the bane of my existence | |
| 01:23:34 | artom | For what it's worth, I kept the single device_tags in parameters.yaml because, if we can't min/max/min on them, I saw no point in splitting them up | |
| 01:24:48 | mriedem | the point in splitting them up was to keep a bit more specific description for each parameter | |
| 01:25:01 | mriedem | so we didn't have to cram either/or type wording into a single description | |
| 01:25:24 | mriedem | https://review.openstack.org/#/c/426991/ is the tempest change | |
| 01:26:23 | artom | mriedem, hrmm, looks like I didn't completely grok it then | |
| 01:29:41 | artom | Oh, duh, I get the point of splitting them | |
| 01:33:01 | artom | mriedem, is there the same rc1 deadline on the novaclient bug? | |
| 01:33:22 | mriedem | artom: novaclient is frozen for ocata | |
| 01:33:49 | mriedem | we can backport https://review.openstack.org/#/c/424745/ though | |
| 01:33:51 | mriedem | later | |
| 01:34:30 | artom | Aha, right | |
| 01:35:27 | gmann | artom: nova patch looks good | |
| 01:35:55 | artom | gmann, wootness, thanks for following up :) | |
| 01:37:10 | gmann | mriedem: thanks for tempest tests, ll check today | |
| 01:40:38 | gmann | mriedem: on all_tenant policy thing - https://review.openstack.org/#/q/topic:bp/add-whitelist-for-server-list-filter-sort-parameters | |
| 01:41:24 | gmann | mriedem: if we are deferring that to pike i can update doc patch of sort/ filter (removing the policy bits) | |
| 01:41:42 | gmann | mriedem: we need to get release notes/docs for sort/filter in | |
| 01:41:45 | mriedem | gmann: yes we're deferring the policy changes to pike | |
| 01:41:48 | mriedem | so we can discuss at the PTG | |
| 01:42:04 | mriedem | and yeah i have the sort/filter reno patch in our ocata rc1 todo etherpad | |
| 01:42:11 | gmann | mriedem: cool. ll update doc one quickly | |
| 01:43:18 | openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add release note and docs for filter/sort whitelist https://review.openstack.org/421760 | |
| 01:43:26 | gmann | mriedem: ^^ | |
| 01:47:03 | mriedem | gmann: thanks. i probably won't get to that until tomorrow at this rate. | |
| 01:47:26 | gmann | mriedem: sure. | |
| 01:47:34 | mriedem | my wife has decided that simply because she has to get up at 4am, she wants to get some sleep and therefore i need to finish the parenting duties for the evening. | |
| 01:48:14 | gmann | mriedem: cool. spend some good time with your kids | |
| 01:48:59 | gmann | i have oomichi in japan today. ll catch him too :) | |
| 01:49:53 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts https://review.openstack.org/421420 | |
| 01:49:54 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell https://review.openstack.org/421459 | |
| 01:49:54 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | |
| 01:49:55 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells https://review.openstack.org/421458 | |
| 01:49:55 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts https://review.openstack.org/421442 | |
| 01:49:56 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell https://review.openstack.org/421444 | |
| 01:53:04 | diana_clarke | Now I need to switch to a GitHub workflow. I've grown to really like Gerrit. | |
| 01:55:33 | artom | diana_clarke, gerrithub? | |
| 01:56:09 | diana_clarke | artom: is that a thing... off to google it | |
| 01:59:17 | melwitt | mriedem: I just looked at the cells v1 job "Failed to get metadata for IP 10.1.3.255: no metadata" in n-api logs and I think it's normal in that, the error means it couldn't lookup a FixedIP by address. and in cells v1 fixed_ips are populated only in the child cell database, so any reads from the api cell will yield not found | |
| 02:06:06 | mriedem | melwitt: should that be an ERROR though? | |
| 02:06:15 | mriedem | plus, has that changed recently? | |