| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-23 | |||
| 14:29:25 | dansmith | I really want that context switch patch to be really tight | |
| 14:29:54 | mriedem | or just leave it and point it out in the commit message in this one | |
| 14:29:54 | dansmith | okay, I totally don't understand why that's different than having it in the first patch, but I'll do it anyway :) | |
| 14:30:03 | mriedem | the fact 3 people had to ask about it, | |
| 14:30:05 | mriedem | means it's not clear | |
| 14:30:13 | mriedem | so just point it out in the commit message | |
| 14:30:18 | dansmith | if that's good enough then I'd like that option | |
| 14:30:25 | mriedem | fine by me | |
| 14:30:43 | mriedem | melwitt: it's 7:30am, i assume you're 2 hours into addressing https://review.openstack.org/#/c/446239/ | |
| 14:30:57 | mriedem | because i want to +2 that one | |
| 14:41:21 | mriedem | alex_xu: replied in https://review.openstack.org/#/c/459569/ | |
| 14:42:48 | mriedem | alex_xu: Kevin_Zheng: jaypipes: why is this a 403 and not a 409? https://review.openstack.org/#/c/464548/5/nova/api/openstack/compute/servers.py | |
| 14:42:58 | mriedem | i thought we normally handled over quota as 409 in the api | |
| 14:43:46 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Add `hide_hypervisor_id` image property https://review.openstack.org/459753 | |
| 14:44:05 | mriedem | nevermind, i guess at some point those were switched to 403 | |
| 14:44:30 | melwitt | mriedem: yeah, I got most of it done yesterday and I'm stuck on a unit test of all things. hope I'll get past that soon | |
| 14:44:55 | Kevin_Zheng | mriedem: I did a search and I saw it was handled in that way somewhere else | |
| 14:45:05 | mriedem | Kevin_Zheng: yeah me too, i didn't realize that was changed | |
| 14:45:56 | Kevin_Zheng | Btw, I tired to cherry pick it to N but seems conflicting | |
| 14:46:17 | mriedem | then you have to do it manually using git cherry-pick -x | |
| 14:46:35 | mriedem | resolve the merge conflict locally, and leave a note in the commit message about what the conflict was | |
| 14:47:36 | mriedem | lyarwood: when you have a minute i'd like to get this released in ocata since it's an upgrade issue https://review.openstack.org/#/c/466774/ | |
| 14:48:01 | lyarwood | mriedem: looking | |
| 14:48:25 | lyarwood | mriedem: while we are talking about stable/ocata, should we drop Citrix XenServer CI to non-voting until they get it passing? | |
| 14:48:54 | Kevin_Zheng | mriedem: ok I will try tomorrow | |
| 14:49:10 | mriedem | lyarwood: did that last night, emailed the team and bobball is already aware and disabled the failing tests until fixed | |
| 14:49:27 | lyarwood | mriedem: cool, thanks :) | |
| 14:52:56 | fishbone_ | Hello all, I am having a strange issue...After an upgrade from newton to ocata all of my old virtual servers are innaccessible | |
| 14:53:30 | mriedem | lyarwood: the depends-on here seems wrong https://review.openstack.org/#/c/466480/1 | |
| 14:53:40 | fishbone_ | I am able to see them in the database and when runnin openstack server list, however if i run openstack server show ID it will say the server does not exist | |
| 14:54:21 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Add `hide_hypervisor_id` image property https://review.openstack.org/459753 | |
| 14:54:47 | lyarwood | mriedem: yeah that's wrong, gerritui just copied it across for the original commit, I'll remove it now | |
| 14:55:00 | mriedem | fishbone_: did you run nova-manage cell_v2 map_instances as part of the upgrade? | |
| 14:56:06 | mriedem | fishbone_: check to see if you have instance_mappings in the nova_api db | |
| 14:56:09 | fishbone_ | yes I did, I wonder if the fact that I had servers before doing that is the cause of the issue | |
| 14:56:21 | fishbone_ | Any new servers I create register correctly | |
| 14:56:30 | fishbone_ | oh wait | |
| 14:56:42 | fishbone_ | mriedem: I dont think I ran the one for instances | |
| 14:56:46 | mriedem | map_instances is specifically for mapping existing instances to a cell | |
| 14:56:59 | mriedem | https://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v2 | |
| 14:57:39 | mriedem | fishbone_: note that nova-manage cell_v2 simple_cell_setup will also do map_instances automatically | |
| 14:58:29 | fishbone_ | mriedem: I did not use that command when I performed the upgrade in this case | |
| 14:58:40 | fishbone_ | mriedem: I will try to run that command now | |
| 14:58:50 | mriedem | ok, if you don't use simple_cell_setup, then you need to run map_instances | |
| 14:59:23 | mriedem | from the release note too http://git.openstack.org/cgit/openstack/nova/tree/releasenotes/notes/cells-discover-hosts-06a3079ba687e092.yaml | |
| 15:00:05 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add strict option to discover_hosts https://review.openstack.org/467218 | |
| 15:10:52 | mriedem | melwitt: so can we abandon this? https://review.openstack.org/#/c/462744/ | |
| 15:10:58 | fishbone_ | mriedem: Thanks a lot, that worked, I didn't use the simple cell setup because when I did it wasn't naming the cell appropriately | |
| 15:11:32 | mriedem | fishbone_: yup, which i'm resurrecting in newton https://review.openstack.org/#/c/420461/ | |
| 15:11:37 | mriedem | but that doesn't help you here | |
| 15:11:53 | mriedem | fishbone_: i'm going to start a faq in the nova devref for cells v2 things, i'll put your case in there | |
| 15:12:32 | jaypipes | sdague, mriedem: can you point to docs on how to update the global requirements properly for a new dependency (in this case, os-vif and os-traits) | |
| 15:12:46 | melwitt | mriedem: yeah. I'm disturbed by the fact that the message seems to vanish when not converted to unicode (since that will happen to other people in the wild). I was trying to decide if I wanted to recheck it again to see if I could locate the unconverted messages in the logs | |
| 15:12:47 | jaypipes | sdague, mriedem: reference this patch https://review.openstack.org/#/c/462769/ | |
| 15:12:48 | fishbone_ | mriedem: Thanks again | |
| 15:13:22 | mriedem | jaypipes: so you need a new minimum os-traits in g-r right? | |
| 15:13:42 | mriedem | jaypipes: just bump the version here https://github.com/openstack/requirements/blob/master/global-requirements.txt#L159 | |
| 15:13:51 | jaypipes | mriedem: yeah. and AJaeger sent an email to the ML mentioning some problems with os-vif as well. | |
| 15:14:07 | mriedem | well, | |
| 15:14:12 | jaypipes | mriedem: got it. that's easy enough. ty. | |
| 15:14:15 | mriedem | do we have merged code that needs newer minimums? | |
| 15:14:41 | mriedem | jaypipes: anyway, for a pending patch that needs a new minimum of a library/client, bump g-r in the openstack/requirements repo and then add a depends-on to your nova change to the reqs change and you should be good | |
| 15:14:54 | mriedem | you'd want to also bump the requirements.txt in your nova patch | |
| 15:14:59 | mriedem | but the depends-on will keep the ci happy | |
| 15:15:07 | jaypipes | ok, thanks | |
| 15:15:08 | openstackgerrit | Dan Smith proposed openstack/nova master: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 15:15:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 15:15:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 15:15:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell on local delete https://review.openstack.org/464059 | |
| 15:15:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment https://review.openstack.org/458537 | |
| 15:17:40 | mriedem | melwitt: btw, remember this? https://review.openstack.org/#/c/420461/ | |
| 15:18:04 | mriedem | melwitt: i need that for backporting the patch to handle special characters in the db connection url | |
| 15:18:10 | mriedem | which makes me sad | |
| 15:18:14 | melwitt | ah, right | |
| 15:18:31 | melwitt | bummer | |
| 15:18:35 | mriedem | despite grenade failing, i'm kind of hesitant to put that out in a newton fix, but it would be good for anyone that hasn't rolled up yet | |
| 15:18:40 | mriedem | because i think fishbone_ ran into the same thing | |
| 15:18:49 | mriedem | the cell0 db name was wrong on first attempt | |
| 15:18:58 | mriedem | and i think i've seen that in the forum | |
| 15:19:04 | mriedem | aka ask.o.o | |
| 15:19:35 | melwitt | ugh | |
| 15:24:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add links to summit videos on cells v2 https://review.openstack.org/467284 | |
| 15:24:30 | mriedem | alaski: dansmith: ^ etched in stone for all time | |
| 15:24:42 | dansmith | oh my | |
| 15:25:20 | alaski | not sure how I feel about that | |
| 15:25:28 | mriedem | you feel great about it | |
| 15:25:34 | mriedem | you put in all that hard work preparing | |
| 15:25:44 | mriedem | picked out your best and most professional shorts | |
| 15:25:52 | mriedem | i kid | |
| 15:26:13 | dansmith | heh | |
| 15:26:27 | alaski | :) | |
| 15:26:37 | alaski | I should get some suit pants made into shorts someday for formal occassions | |
| 15:26:38 | mriedem | don't worry i'm going to do the same for jaypipes's videos on placement | |
| 15:26:58 | mriedem | alaski: "get made into"? you have a scissors in the house right? | |
| 15:27:39 | alaski | yep, it's the sewing up after that would be new territory for me | |
| 15:27:48 | mriedem | you're putting too much work into this | |
| 15:28:02 | mriedem | the cut off look is part of the style | |
| 15:28:32 | mriedem | it's like a mullet | |
| 15:28:35 | melwitt | you need to choose the hem for the suit shorts | |
| 15:28:38 | mriedem | business in the front, party in the back | |