| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-25 | |||
| 17:12:39 | mriedem | https://github.com/openstack/nova-lxd#support-and-discussions | |
| 17:12:54 | BrazierCustoms | thank you mriedem | |
| 17:12:59 | mriedem | yw | |
| 17:13:25 | mriedem | arxcruz: looking at the nova code that returns that exception, unfortunately we return it for just about any kind of response from cinder | |
| 17:13:59 | arxcruz | mriedem: yeah, i tried to adventure in the wild nova code and notice that | |
| 17:14:13 | mriedem | arxcruz: are the computes running queens code? | |
| 17:14:20 | arxcruz | mriedem: yes | |
| 17:14:59 | mriedem | arxcruz: is this repeatable? | |
| 17:15:03 | mriedem | we could put up a debug patch | |
| 17:15:30 | mriedem | we're either failing on a 400 when reserving the volume, or a 404, but it doesn't seem like it should be a 404 | |
| 17:16:19 | mriedem | i don't see any POST action to reserve the volume in the cinder-api logs though | |
| 17:17:18 | arxcruz | mriedem: yes, it's repeatable, and I have an environment up | |
| 17:17:21 | arxcruz | if you want to take a look | |
| 17:17:29 | arxcruz | on rdo cloud | |
| 17:18:54 | mriedem | arxcruz: can't right now, but i'd just dump a LOG.exception() here before raising https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1354 | |
| 17:19:03 | mriedem | and run the recreate with that | |
| 17:19:54 | arxcruz | mriedem: sure, I'll do that, but not right now as well, but I'll update you guys | |
| 17:35:41 | openstackgerrit | Stephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture https://review.openstack.org/515146 | |
| 17:35:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff https://review.openstack.org/515147 | |
| 17:35:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy' https://review.openstack.org/515148 | |
| 17:35:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149 | |
| 17:41:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | |
| 17:52:53 | openstackgerrit | Eric Fried proposed openstack/nova master: Process granular resources/traits in extra_specs https://review.openstack.org/515151 | |
| 17:53:15 | efried | jaypipes cdent ^ | |
| 17:53:30 | efried | Doesn't actually do anything with it yet; just parses it. | |
| 17:54:10 | cdent | efried: cool, will look, probably tomorrow morning | |
| 17:57:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | |
| 17:57:57 | efried | That's a cool review number :) | |
| 17:58:41 | cdent | yes, yes it is | |
| 18:10:13 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] manage cache headers for allocations https://review.openstack.org/515156 | |
| 18:56:26 | mriedem | sdague: can you think of anything obvious where https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/fakes.py#L66 would ignore the version in the request object? | |
| 18:57:00 | mriedem | i'm having a hell of a time with making this work https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/compute/test_flavor_manage.py#L126 | |
| 18:57:28 | mriedem | even though i've passed version='2.54' to https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/compute/test_flavor_manage.py#L343 | |
| 18:58:29 | mriedem | i get a 400 back and when i dump the headers in the response, the version is always 2.1 | |
| 18:59:50 | openstackgerrit | Merged openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 18:59:56 | openstackgerrit | Merged openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 19:07:55 | mriedem | oh ffs | |
| 19:07:59 | mriedem | !!! | |
| 19:08:00 | openstack | mriedem: Error: "!!" is not a valid command. | |
| 19:08:37 | melwitt | what is it | |
| 19:08:50 | mriedem | i'd rather not say | |
| 19:08:58 | mriedem | i knew it would be so stupid | |
| 19:09:24 | melwitt | heh k | |
| 19:09:46 | mriedem | let's just say self.request_body['description'] = 'test description' | |
| 19:09:53 | mriedem | was putting the description key in the wrong part of the request body | |
| 19:11:21 | melwitt | that's not related to the microversion in the response headers though, right? that is, it always returns 2.1, is it supposed to? | |
| 19:11:39 | mriedem | that i don't know about | |
| 19:11:50 | mriedem | i gave up with the wsgi app routing and just called the controller method directly | |
| 19:11:54 | mriedem | like most of our unit tests | |
| 19:12:05 | mriedem | i assume the fake wsgi app router stuff assumes 2.1 | |
| 19:12:09 | melwitt | oh, okay. I was looking at how it returns headers and didn't find it yet | |
| 19:12:10 | mriedem | since no tests using >2.1 rely on it | |
| 19:12:14 | melwitt | I see | |
| 19:13:24 | mriedem | in other good news, the guy that "fixed" our dryer about 1 year ago is coming back on friday to "fix" it again | |
| 19:13:42 | mriedem | if i get it fixed one more time, the repair costs will have exceeded the original purchase price | |
| 19:14:22 | melwitt | yikes | |
| 19:15:19 | melwitt | I've been waiting for our old washer/dryer to fail to have an excuse to get new ones. and they just keep on working | |
| 19:15:28 | mriedem | brand please | |
| 19:18:27 | melwitt | kenmore, and they look like they're from the 90s | |
| 19:18:44 | mriedem | that's what our old ones were, lasted forever | |
| 19:19:06 | melwitt | lol | |
| 19:21:11 | melwitt | what brand are your new ones? so I can avoid if these ever fail | |
| 19:21:25 | mriedem | samsung | |
| 19:21:53 | mriedem | and not even the model that i can sync to my samsung phone so i can talk to it while it's drying! | |
| 19:22:29 | melwitt | well that's a shame | |
| 19:23:49 | artom | Is Quobyte CI still a thing? | |
| 19:31:03 | mriedem | kaisers1: ^ | |
| 19:31:13 | mriedem | melwitt: ok so with the fake wsgi app thing, i figured it out | |
| 19:31:27 | mriedem | you have to specify the microversion in the header directly req.headers['X-OpenStack-Nova-API-Version'] = self.microversion | |
| 19:31:32 | mriedem | which makes sense i guess | |
| 19:32:20 | melwitt | cool, good to know | |
| 19:36:55 | sdague | mriedem: sorry, got got back to scrollback | |
| 19:37:11 | sdague | yeh, having to specify the headers raw rings a bell | |
| 19:40:40 | artom | kaisers1, mriedem, it appears to be still reporting on patches | |
| 19:40:56 | artom | kaisers1, are you guys still running with libvirt dynamic_ownership=0? | |
| 19:41:07 | artom | I can't seem to find the configs you use anywhere | |
| 19:45:30 | dansmith | mriedem: so i think I've realized why they call it "slow render" | |
| 19:46:47 | mriedem | because it renders slowly? | |
| 19:47:48 | artom | It's actually some dyslexic nerder owls | |
| 19:51:38 | dansmith | the +10 and -10 things are _painfully_ slow for me now | |
| 19:51:42 | dansmith | it's really terrible | |
| 19:51:58 | mriedem | not sure if i've noticed yet | |
| 19:52:28 | melwitt | that might just be gerrit getting slower and slower as it runs out of memory or whatever (the usual) | |
| 19:52:33 | melwitt | I noticed it too today | |
| 19:54:59 | dansmith | this is ui lag I think | |
| 19:55:36 | melwitt | okay. I was getting lag just trying to open changes so I thought it was related | |
| 19:55:39 | dansmith | yep, no new network calls made when I do +10 | |
| 19:55:46 | melwitt | oh | |
| 19:55:49 | dansmith | this is waaaay slower than opening new changes or files | |
| 19:56:56 | mriedem | dansmith: and fast render fixes it? | |
| 19:57:04 | dansmith | I haven't switched back to test | |
| 19:57:12 | melwitt | do it | |
| 19:57:24 | dansmith | because it seems to depend on the size of the file | |
| 19:57:48 | mriedem | ... | |
| 19:58:26 | dansmith | yep, fast render makes it normal | |
| 19:58:42 | mriedem | huh, well, | |
| 19:58:52 | mriedem | i guess just toggle slow render if needed | |
| 19:58:54 | mriedem | like spec reviews | |
| 19:59:01 | dansmith | or stop reviewing specs | |
| 19:59:06 | melwitt | yeah, have to pick n choose now | |