| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-06 | |||
| 14:00:48 | ttx | \o/ and fresh snow | |
| 14:02:58 | bauzas | ttx: zactly, 30cm waiting for you ;) | |
| 14:05:30 | openstackgerrit | Timofey Durakov proposed openstack/nova master: nfs fix for xenial images https://review.openstack.org/329466 | |
| 14:06:09 | kashyap | sdague: Heya, random ping -- someone (a Kernel developer, actually) at FOSDEM asked me yesterday: Why DevStack, why not...just use 'Gentoo'. They were serious :-) | |
| 14:06:33 | sdague | kashyap: :) | |
| 14:06:45 | sdague | sounds like they never actually looked into the issue | |
| 14:07:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Move proxies to 'console/proxy' https://review.openstack.org/408192 | |
| 14:07:12 | kashyap | sdague: Yeah, right. But they were not really mean or anything, just thinking out loud | |
| 14:07:36 | kashyap | sdague: I explained them, also the reason why it's shell script (that a human / admin / operator should be able to read it, etc). | |
| 14:10:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate two more nova-net options https://review.openstack.org/412390 | |
| 14:23:34 | mriedem | superdan: first, where is dansmith | |
| 14:23:43 | mriedem | superdan: second, alex_xu and i have comments in https://review.openstack.org/#/c/428824/ | |
| 14:32:02 | dansmith | mriedem: okay coffee then that | |
| 14:34:22 | mriedem | manila also reported a bug that the change for multi-cell instance listing broke them https://bugs.launchpad.net/nova/+bug/1659391 but so far i'm not able to find a job that failed due to that to debug it, | |
| 14:34:22 | 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:34:59 | mriedem | plus the description is overly confusing with references to cells being disabled and default cells, which really throws me off | |
| 14:38:17 | dansmith | hmm, okay | |
| 14:39:01 | dansmith | mriedem: well the description says that after that change, nova only returns instances assigned to cells | |
| 14:39:03 | dansmith | which is 100% true :) | |
| 14:40:02 | mriedem | yeah i left some more questions, it's not clear to me if the server they are creating is done before simple_cell_setup or not | |
| 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 | 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:38 | lyarwood | mdbooth: ah nice | |
| 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: Remove redundant bdm serial mangling and saving during swap_volume https://review.openstack.org/427364 | |
| 16:13:59 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | |
| 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/ | |