| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-25 | |||
| 16:55:01 | arxcruz | dansmith: yes | |
| 16:55:11 | mriedem | arxcruz: i think i have a tempest patch for this already... | |
| 16:55:12 | mriedem | sec | |
| 16:55:54 | arxcruz | mriedem: cool :D | |
| 16:55:58 | mriedem | arxcruz: https://review.openstack.org/#/c/510951/ ? | |
| 16:56:02 | mriedem | that merged already though | |
| 16:56:22 | mriedem | you need to provide more details on where this actually fails, like trace the request as dan said | |
| 16:56:31 | mriedem | is it on attach, detach, other? | |
| 16:57:39 | mriedem | although that tempest patch wouldn't fix test_volume_boot_pattern which is a scenario test | |
| 16:59:28 | mriedem | looks like req-733a784c-6fed-48c5-a62b-3876edb56d57 is the request id | |
| 16:59:51 | sdague | mriedem: if it's being used to choose and understand, I sort of assumed it would be in GET /flavors | |
| 16:59:56 | sdague | but honestly, I can go either way | |
| 17:00:07 | sdague | maybe just ask the folks that wanted it | |
| 17:01:19 | arxcruz | mriedem: it's on test_volume_boot_pattern test, from my debug, it create the volume successfully, then when it tries to create the instance from the volume, it fails | |
| 17:01:46 | arxcruz | mriedem: here's the track from tempest https://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset006-master/0bb0995/undercloud/home/jenkins/tempest_output.log.txt.gz#_2017-10-23_17_30_43 | |
| 17:03:15 | mriedem | arxcruz: why don't i see the POST /servers request id in the nova-api undercloud logs? https://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset006-master/0bb0995/undercloud/var/log/nova/nova-api.log.txt.gz | |
| 17:03:20 | mriedem | or would it be in the overcloud? | |
| 17:03:54 | arxcruz | in overcloud | |
| 17:04:19 | arxcruz | mriedem: https://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset006-master/0bb0995/subnode-2/var/log/nova/nova-api.log.txt.gz | |
| 17:04:23 | arxcruz | this is the overcloud nova log | |
| 17:05:00 | arxcruz | https://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset006-master/0bb0995/subnode-2/var/log/nova/nova-api.log.txt.gz#_2017-10-23_17_30_30_020 | |
| 17:05:04 | arxcruz | the failure | |
| 17:05:40 | mriedem | huh, not helpful at all | |
| 17:06:56 | arxcruz | yeah, that's all that I can see... | |
| 17:07:04 | arxcruz | there's no call to cinder, just trow this error | |
| 17:07:33 | mriedem | in the dsvm ci logs we trace the requests to cinder via python-cinderclient | |
| 17:07:44 | mriedem | that would be helpful here in debugging failures in triple0 logs | |
| 17:10:42 | BrazierCustoms | hello. I am looking for a way to import an existing LXD container into glance for use as a base image. is this possible? | |
| 17:10:55 | BrazierCustoms | for nova-lxd | |
| 17:11:50 | mriedem | arxcruz: looks like it's getting the volume fine from the cinder-api side https://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset006-master/0bb0995/subnode-2/var/log/cinder/cinder-api.log.txt.gz#_2017-10-23_17_30_30_474 | |
| 17:11:54 | mriedem | around the time of the nova-api failure | |
| 17:11:57 | BrazierCustoms | I have tried searching and I have tried a couple things I thought would have worked but unsuccessful. | |
| 17:12:04 | mriedem | BrazierCustoms: this wouldn't be the channel for nova-lxd | |
| 17:12:10 | mriedem | as it's not an in-tree driver | |
| 17:12:19 | BrazierCustoms | i couldnt fint that either :( | |
| 17:12:30 | mriedem | it's in the nova-lxd repo readme i think | |
| 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! | |