| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-06 | |||
| 14:40:39 | dansmith | like before an upgrade you mean? | |
| 14:41:41 | mriedem | "It can be seen when cell feature and services are disabled and default cell hasn't been created yet. In this case all Nova VMs hang in scheduled state and cannot be viewed." | |
| 14:41:45 | mriedem | i was just asking wtf that meant | |
| 14:41:57 | mriedem | but it looks like simple_cell_setup is run first http://logs.openstack.org/37/426737/12/check/gate-manila-tempest-dsvm-generic-no-share-servers-ubuntu-xenial-nv/064f0da/logs/devstacklog.txt.gz#_2017-02-02_13_41_26_166 | |
| 14:42:07 | mriedem | and then they create a server later http://logs.openstack.org/37/426737/12/check/gate-manila-tempest-dsvm-generic-no-share-servers-ubuntu-xenial-nv/064f0da/logs/devstacklog.txt.gz#_2017-02-02_13_45_01_035 | |
| 14:42:30 | dansmith | well, the first part of that sentence isn't valid either | |
| 14:42:56 | mriedem | i think they mean cells v1 | |
| 14:43:00 | dansmith | this is one of the reasons I wish we had used a different name than cells | |
| 14:43:03 | mriedem | but yeah that totally threw me off when first reading this | |
| 14:43:08 | dansmith | there's a lot of confusion that has stemmed from that decision | |
| 14:43:50 | mriedem | having said all that, the whole 'what don't you get about this bug' attitude in the report makes me kind of not care either | |
| 14:44:12 | mriedem | unless it's not unique to them | |
| 14:44:22 | dansmith | yeah | |
| 14:44:52 | dansmith | the change they're referencing has been in for like three weeks now right? | |
| 14:44:54 | mriedem | maybe i'll just revert their workaround and see if it fails on the same thing again and then debug from there | |
| 14:45:08 | mriedem | yeah, landed on 1/24 and they worked around it on 1/28 it looks like | |
| 14:45:17 | dansmith | doubt anyone else has had their gate completely shutdown for three weeks | |
| 14:45:27 | sfinucan | There's probably a much better solution, but hacky hack for opening a Python module path in vim https://that.guru/blog/open-python-paths-with-vim/ | |
| 14:45:30 | dansmith | and just before rc2 they bring it up.. nice | |
| 14:47:16 | mriedem | the bug was actually reported awhile ago | |
| 14:47:27 | mriedem | 1/25 | |
| 14:47:32 | mriedem | the day after your change landed https://bugs.launchpad.net/nova/+bug/1659391 | |
| 14:47:33 | openstack | Launchpad bug 1659391 in OpenStack Compute (nova) "Server list API does not show scheduled servers that are not assigned to any cell" [Undecided,New] | |
| 14:47:35 | mriedem | we just don't triage our bugs | |
| 14:49:57 | mriedem | eharney: not sure if you saw https://review.openstack.org/#/c/429476/ but if you have that recreate env for your nfs volume snapshot issue, it'd be cool if you could test that against a paused vm, because i'm not sure if that will work or not | |
| 14:52:57 | eharney | mriedem: i'll give it a shot | |
| 14:53:04 | mriedem | eharney: thanks | |
| 14:55:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Update the upgrades part of devref https://review.openstack.org/428824 | |
| 14:56:51 | dansmith | mriedem: ah, okay | |
| 14:58:22 | macsz | bauzas: i might be late, 5min tops | |
| 14:59:11 | bauzas | macsz: no worries, I'll write up an etherpad | |
| 15:04:02 | macsz | bauzas: back | |
| 15:08:01 | macsz | bauzas: sorry, hit the heavy traffic and was driving while the meeting was on - which etherpad? | |
| 15:08:34 | bauzas | macsz: just started https://etherpad.openstack.org/p/nova-scheduler-pike | |
| 15:12:14 | mriedem | johnthetubaguy: you were +2 on PS1, are you still happy with this? https://review.openstack.org/#/c/428824/ | |
| 15:13:33 | openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: WIP: Runtime image type detection https://review.openstack.org/429745 | |
| 15:20:41 | bauzas | mriedem: FWIW, I'm totally fine with https://review.openstack.org/#/c/428824/ | |
| 15:21:05 | mriedem | bauzas: i thought you were supposed to be teaching kids about devops while skiing down a mountain or something? | |
| 15:21:18 | bauzas | mriedem: starting Wed :) | |
| 15:21:21 | mriedem | wtf | |
| 15:21:32 | mriedem | you told everyone like 5 times you'd be gone in 25 minutes | |
| 15:21:40 | bauzas | heh | |
| 15:21:53 | bauzas | nah, just went to school for $daughter but back | |
| 15:21:58 | mriedem | oh i see | |
| 15:22:11 | bauzas | now, I need some homework support | |
| 15:22:14 | bauzas | but | |
| 15:22:19 | bauzas | I'm still around | |
| 15:22:33 | bauzas | and starting Wed, will be off because of a conference | |
| 15:22:39 | mriedem | ok, well i'm leaving for a couple of hours | |
| 15:23:08 | mriedem | heh | |
| 15:23:17 | mriedem | i just can't keep up with your active schedule | |
| 15:25:34 | openstackgerrit | Emilien Macchi proposed openstack/nova master: DNM - test tripleo without nova api in WSGI/apache https://review.openstack.org/429008 | |
| 15:39:35 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Spec idea on the future of quotas for Nova https://review.openstack.org/429678 | |
| 15:40:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Reserve migration placeholders for Ocata backports https://review.openstack.org/427900 | |
| 15:41:07 | dansmith | we should probably merge this now ^ | |
| 15:45:01 | diga | bauzas: Hi | |
| 15:45:15 | diga | bauzas: sorry got delay due to dinner | |
| 15:45:47 | macsz | diga: https://etherpad.openstack.org/p/nova-scheduler-pike | |
| 15:46:05 | diga | macsz: okay, will go through it | |
| 15:48:36 | diga | macsz: I dont know if someone is working on Ironic side | |
| 15:48:47 | diga | macsz: but its essential thing | |
| 15:54:06 | jroll | diga: macsz: we plan o figure out that upgrade path problem at PTG sometime, if that's what you're asking about | |
| 15:54:35 | macsz | jroll: that answers the question :) | |
| 15:54:40 | jroll | :D | |
| 16:04:05 | mdbooth | lyarwood: Incidentally, it turns out your swap_volume patch is required for my nfs patch. I was seeing a log warning from my patch about unmounting a volume which supposedly wasn't mounted. Turns out the root cause was swap_volume disconnecting the wrong volume. | |
| 16:04:23 | mdbooth | Rebased my change on yours and the problem goes away. | |
| 16:04:38 | lyarwood | mdbooth: ah nice | |
| 16:04:38 | diana_clarke | Does anyone know off the top of their head how to run the functional mysql nova tests locally? Environment variables, etc? | |
| 16:04:56 | mdbooth | lyarwood: Well, does in my testing, anyway. I should push the rebase and re-run ci. | |
| 16:06:08 | diga | jroll: Yep | |
| 16:10:35 | mdbooth | lyarwood: Mind if I rebase swap_volume, btw? | |
| 16:11:07 | mdbooth | I have a minor clash with your findmnt change, which merged after the base of your swap_volume change | |
| 16:11:18 | mdbooth | So it would be convenient to me to have both. | |
| 16:11:20 | lyarwood | mdbooth: nope, go ahead | |
| 16:11:32 | mdbooth | Thanks | |
| 16:13:09 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | |
| 16:13:59 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | |
| 16:13:59 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Remove redundant bdm serial mangling and saving during swap_volume https://review.openstack.org/427364 | |
| 16:14:38 | lyarwood | mdbooth: ^ that's another small change that takes a while to get your head around if you have any time tomorrow to review | |
| 16:26:58 | dansmith | melwitt: wanna look this over? https://review.openstack.org/#/c/429056/ | |
| 16:28:55 | openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: [WIP] Placement api: Add json_error_formatter to defaults https://review.openstack.org/395194 | |
| 16:31:39 | openstackgerrit | sahid proposed openstack/nova master: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260 | |
| 16:31:40 | openstackgerrit | sahid proposed openstack/nova master: libvirt: ensure live-migration success https://review.openstack.org/409261 | |
| 16:32:23 | melwitt | dansmith: done | |
| 16:32:45 | dansmith | melwitt: thanks | |
| 16:33:00 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 16:35:25 | dansmith | melwitt: cdent: FWIW, the reason I used a naked "placement" is because I'm referring to placement api, service, database, catalog entry, etc as a group | |
| 16:35:41 | dansmith | "placement service" would be fine too, but... that was my thought process | |
| 16:36:53 | cdent | dansmith: yeah, thus the "fine as is". I think "service" is a nice to have, because it's hard to be sure that "placement" on its own is a proper noun and not just some random thing, like layout of the knives and forks on the table. | |
| 16:37:12 | melwitt | dansmith: I see, thanks for clarifying. that makes it make a lot more sense | |
| 16:39:05 | pumaranikar | cdent, johnthetubaguy https://review.openstack.org/#/c/395194/ updated patchset for json error formatter. can you please quickly review and let me know that this is what we are expecting. ? | |
| 16:39:19 | edleafe | cdent: Capitalization maybe? E.g.: 'placement' (random thing) vs. 'Placement' (the resource thingy) | |
| 16:39:50 | cdent | edleafe: because it is new, I'm still not certain that's enough. All of us here think of it as this well known things, but it is not. | |
| 16:41:56 | edleafe | true | |
| 16:48:24 | cdent | pumaranikar: yeah, that looks like a good way to go assuming it works well. How do you feel about it? | |
| 16:49:37 | pumaranikar | cdent, thanks for quick review. Yes it will work. Only thing is we need to modify decorator everywhere | |
| 16:50:04 | pumaranikar | cdent, thats why I thought of middleware initially. | |
| 16:50:37 | cdent | that's okay, I think, as it is a pretty simple change and in any case, no matter how we add, we have to change a bunch of files to remove the keyword arg from all the raised exceptions | |
| 16:51:06 | cdent | I think to get a middleware version that actually worked properly would be considerably more complex (because of the need to parse headers and some response bodies) | |
| 16:51:51 | pumaranikar | cdent, yes. that makes sense. | |