| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 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? | |
| 02:06:18 | mriedem | i don't remember seeing that before | |
| 02:06:24 | melwitt | for non cells v1, yes | |
| 02:06:44 | melwitt | and agreed I don't remember seeing it repeatedly. I thought it used to only show it once. so I'm not sure what changed | |
| 02:10:02 | openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: rbd: Remove unnecessary 'encode' calls https://review.openstack.org/412356 | |
| 03:06:55 | mriedem | so, bauzas' scheduler change has failed in the gate twice in the cells v1 job on the volume swap test... | |
| 03:07:25 | mriedem | and https://review.openstack.org/#/c/399735/ merged today | |
| 03:08:17 | mriedem | and we don't run the cells v1 job on the check queue for devstack, which re-enabled the test | |
| 03:08:36 | mriedem | https://review.openstack.org/#/c/374373/ | |
| 03:08:44 | mriedem | but i'm pretty sure that just introduced some kind of weird race failure | |
| 03:08:46 | mriedem | lyarwood: ^ | |
| 03:11:24 | mriedem | mdbooth: ^ | |
| 03:12:50 | mriedem | looks like it's also killing the ceph job | |
| 03:16:15 | mriedem | i'm assuming this is probably just something to do with cells v1 not supporting the swap volume API | |
| 03:17:34 | mriedem | https://bugs.launchpad.net/nova/+bug/1660511/ | |
| 03:17:35 | openstack | Launchpad bug 1660511 in OpenStack Compute (nova) "TestVolumeSwap.test_volume_swap fails in cells v1 job waiting for first volume to detach" [High,Confirmed] | |
| 03:18:39 | mriedem | melwitt: if cells v1 doesn't support the swap volume API, shouldn't it fail in some kind of spectacular way? | |
| 03:19:28 | melwitt | mriedem: cells v1 non support usually fails in subtle crappy ways, in my experience | |
| 03:20:27 | mriedem | yeah that seems to be the case here | |
| 03:20:45 | mriedem | i was looking at the requests in a failed job earlier and at some point it looked like something just didn't happen, | |
| 03:20:49 | mriedem | like one of the requests just stopped | |
| 03:20:52 | melwitt | is this a new test, i.e. something that needs to be added to the blacklist? | |
| 03:21:01 | mriedem | it's a newly enabled test as of today | |
| 03:21:03 | mriedem | based on tempest config | |
| 03:21:04 | melwitt | ah | |
| 03:21:08 | mriedem | so i'm going to disable it for cells v1 in devstack | |
| 03:21:18 | mriedem | we could also blacklist it in nova, | |
| 03:21:30 | mriedem | which would probably actually make this go faster since we can fast approve it and not wait for devstack cores | |
| 03:21:33 | mriedem | mtreinish: are you around? | |
| 03:21:40 | melwitt | oh, disable the tempest conf option if cells v1 in devstack | |