Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-06
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.
16:52:10 pumaranikar cdent, i will try to update this today in all places.
16:52:41 cdent pumaranikar: you could consider doing a stack of changes where you change each file in handlers/*.py in a different patch
16:53:38 pumaranikar cdent, yes good idea. It would be easy for review.
16:55:39 pumaranikar cdent, are we ok with class name in wrapper file ? :)
16:56:18 cdent I think it is good enough. If people don't like it we can change it easily enough during the review process
16:57:37 pumaranikar cdent, ok. Will let you know once i make changes atleast in all allocations related actions
16:58:34 cdent cool, looking forward to it
16:58:42 edleafe cdent: First pass at functional code paths for placement testing: https://etherpad.openstack.org/p/nova-placement-functional
16:58:49 edleafe cdent: Now to disappear into meetings
16:58:59 cdent edleafe: cool, I'll look at that either this evening or tomorrow morning
16:59:34 edleafe cdent: thanks. Just wanted to get something down before I got sucked into other things
16:59:53 cdent ✔
17:03:45 mriedem dansmith: the revert on that manila change passed https://review.openstack.org/#/c/429729/ :)

Earlier   Later