Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-03
16:51:06 cdent pumaranikar: sorry was on the phone. yes that's right, thus the difficulty
16:54:59 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859
16:56:53 openstackgerrit Maciej Szankin proposed openstack/nova master: WIP conf: remove invalid TODOs https://review.openstack.org/428811
16:58:33 pumaranikar cdent, i will try with middleware and update you
16:58:53 openstackgerrit Maciej Szankin proposed openstack/nova master: WIP conf: remove invalid TODOs https://review.openstack.org/428811
17:03:31 johnthetubaguy pumaranikar: we did that yesterday right, and it failed with middlewear?
17:04:05 johnthetubaguy cdent: the exception just didn't seem to get as fair as that fault wrapper middleware
17:04:27 johnthetubaguy it was already converted into a response with the correct error code, at least is seemed that way
17:04:31 pumaranikar johnthetubaguy, yes. because we were getting valid response
17:04:34 cdent yeah, if you use middleware, it has to be body processing instead of exception inspecting
17:05:00 johnthetubaguy but the body doesn't have the data present that was in the exception I guess?
17:05:25 cdent if the response ststus >= 400 and <500, then if it is json, then reformat it to the structure we want
17:05:37 johnthetubaguy cdent: my suggestion to pumaranikar was to replace the wsgify with our own decorator, that ensure the exception gets the formatter added, its not ideal though
17:06:19 johnthetubaguy it seems easier to remember that adding the formatter to the error message, but that might just be me
17:06:22 cdent if there's a way to do that doesn't involve double decorating that might work
17:06:48 johnthetubaguy yeah, its just the one decorator, I pushed that up yesterday into pumaranikar's patch
17:07:29 cdent that's what I mean by double decorator
17:07:32 mriedem artom: on https://blueprints.launchpad.net/nova/+spec/fix-tag-attribute-disappearing i think i'm going to just mark that as partially completed and then we'll finish the rest in pike
17:07:43 cdent I think it would be better to subclass webob.dec.wsgify (which is a class)
17:07:50 johnthetubaguy cdent: its not the nicest thing, but it seems to work: https://review.openstack.org/#/c/395194/5/nova/api/openstack/placement/handlers/allocation.py@274
17:07:59 artom mriedem, yep :)
17:08:13 johnthetubaguy cdent: yeah, thats nicer
17:08:21 johnthetubaguy pumaranikar: does that make sense?
17:08:33 cdent except, unfortunately the part that one would want to override is smack in a tall stack of code :(
17:08:42 artom mriedem, it's debatable whether the changes to tests are parts of the spec proper, in any case
17:09:10 johnthetubaguy cdent: ah, doh, anyways my hack appeared to work, we could always make that nicer later
17:09:12 cdent hmm, maybe override call_func
17:09:32 johnthetubaguy cdent: did you see this way: https://review.openstack.org/#/c/395194/5/nova/api/openstack/placement/handlers/allocation.py@274 ?
17:09:37 cdent pumaranikar: have a look at the wsgify class in webob/dec.py, may be some ideas
17:10:13 cdent johnthetubaguy: yeah, saw that
17:10:52 cdent I think that will work, but it feels icky for a reason I can't put my finger on
17:11:06 johnthetubaguy cdent: totally with you on that
17:11:08 pumaranikar cdent, looking at wsgify class.
17:11:19 cdent it might, however, be less messy than what I'm thinking. won't really know until we try it
17:11:26 johnthetubaguy cdent: +1
17:11:34 cdent pumaranikar: I think overriding the call_fun method might work
17:15:55 johnthetubaguy cdent: I was curious on your gabbi thoughts too, this seemed the simplest way to ensure the formatter was present: https://review.openstack.org/#/c/395194/5/nova/tests/functional/api/openstack/placement/gabbits/allocations.yaml@158
17:16:45 cdent johnthetubaguy: yeah, pretty much that's it. look for a key and value that wouldn't be there if the formatter was not swinging
17:18:09 EmilienM mriedem: yes it's IPv6. Do you test nova in ipv6?
17:18:40 EmilienM osapi_compute_listen=[::1] doesn't seem to work
17:19:44 mriedem EmilienM: not sure off hand
17:19:50 mriedem sdague or sc68cal might know
17:19:58 EmilienM I'll try with ::1
17:20:30 EmilienM mriedem: triple jobs have ipv6 coverage, fyi
17:21:42 openstackgerrit Dan Smith proposed openstack/nova master: Update the upgrades part of devref https://review.openstack.org/428824
17:21:43 EmilienM mriedem: btw, I saw your comment on https://review.openstack.org/#/c/428424/ - and I think we should have the job asap since things are broken for us, it would be helpful on both sides to have the job in place without having to run "check experimental".
17:23:48 superdan mriedem: see my doc update for the upgrades.rst just now
17:24:02 superdan mriedem: I was explaining that process to someone recently and wanted to RTFM them to that doc
17:24:11 superdan mriedem: but then realized it needed some serious freshening
17:31:05 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716
17:31:06 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163
17:31:07 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Wait for abort_job to complete when not resizing https://review.openstack.org/382449
17:32:11 sc68cal mriedem: EmilienM: [::1]:<port>
17:32:11 johnthetubaguy superdan: I will take a peak at that upgrade doc, looks interesting
17:32:23 superdan johnthetubaguy: thanks
17:32:52 sc68cal mriedem: EmilienM: so I guess [::1]:8774 is what you're looking for
17:33:31 EmilienM sc68cal: yeah? we had osapi_compute_listen=host until now
17:33:47 EmilienM and osapi_compute_listen_port for the port
17:35:33 sc68cal ok, then osapi_compute_listen=::1 should work
17:35:41 EmilienM sc68cal: http://git.openstack.org/cgit/openstack/nova/tree/nova/conf/service.py#n96
17:35:46 EmilienM sc68cal: ok I'm trying it, thanks
17:35:58 EmilienM sc68cal: FYI we add brackets because some other services need it
17:36:00 sc68cal hang on let me poke it with netcat
17:36:07 EmilienM and the best practise in IPv6 is to have brackets
17:36:22 EmilienM tbh, I think having [::1] rather than ::1 is much safer
17:36:31 sc68cal oh yes much safer
17:42:47 sc68cal some services want brackets, others don't
17:44:56 sc68cal are we passing the correct options to eventlet.listen? there's an optional kwarg for address family
17:45:10 sc68cal it might be defaulting to v4 and trying to resolve ::1 as a dns name
17:45:40 johnthetubaguy superdan: I thought SIG_HUP was enough after the initial restart?
17:46:07 johnthetubaguy superdan: oh, missread, I see your point now
17:47:17 superdan johnthetubaguy: not for static configs, yeah
17:47:37 superdan that subtle detail could have been pretty annoying for someone :)
17:48:05 johnthetubaguy yeah, totally
17:51:49 superdan johnthetubaguy: I was tempted to put "read the effing release notes" as a formal step in there :)
17:52:29 johnthetubaguy superdan: step 2, no really, read them.
17:53:10 openstackgerrit Ed Leafe proposed openstack/nova master: Add check for invalid inventory amounts https://review.openstack.org/428800
17:53:23 superdan johnthetubaguy: heh, right
17:53:24 figleaf cdent: ^^ Fixed
17:54:09 cdent figleaf: ossum
17:54:47 openstackgerrit Anusha Unnam proposed openstack/nova master: Reno for additional-notification-fields-for-searchlight https://review.openstack.org/428835
18:00:34 sc68cal EmilienM: so looks like we had a netsplit
18:00:43 sc68cal EmilienM: where did I lose you
18:01:17 EmilienM sc68cal: better: http://logs.openstack.org/86/428786/3/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/d244aec/logs/nova/nova-api.txt.gz#_2017-02-03_17_44_34_097
18:01:35 EmilienM sc68cal: so using ::1 works :-/
18:02:39 sc68cal EmilienM: yep..... we have same issue with mysql vs alembic/sqlchemy
18:02:45 aunnam gibi, if possible can you review this patch https://review.openstack.org/#/c/428835/
18:03:23 sc68cal EmilienM: https://github.com/openstack-dev/devstack/blob/master/lib/database#L77
18:05:29 openstackgerrit Bob Ball proposed openstack/nova master: Fix live migrate with XenServer https://review.openstack.org/424428
18:05:31 BobBall johnthetubaguy: ^^ Hopefully that'll fix the different network concern
18:09:43 cdent mriedem, johnthetubaguy: the tagging has not happened has it?
18:10:04 cdent writing up some notes and wanted to know if https://review.openstack.org/#/c/428375/ was part of the now-ish release
18:11:07 johnthetubaguy https://review.openstack.org/#/c/428531/ says we haven't tagged yet, looks like we can include the ironic one if we wish
18:26:03 mriedem cdent: not yet
18:26:25 mriedem but now that https://review.openstack.org/#/c/428375/ is merged and i'm back from lunch, i'm going to rev the rc1 tag patch
18:27:35 cdent mriedem: how very exciting
18:31:05 mriedem i know, i'm cutting glass over here
18:33:02 superdan lol
18:33:03 openstackgerrit Sujitha proposed openstack/nova master: Reno for additional-notification-fields-for-searchlight https://review.openstack.org/428835
18:34:33 johnthetubaguy mriedem: I am guessing there is no way to backport this one, if it misses RC1 right? https://review.openstack.org/#/c/424428

Earlier   Later