| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-21 | |||
| 18:18:16 | mriedem | ok | |
| 18:18:37 | melwitt | so I guess we could do the retry for now and then replace it with intercept whenever one of us gets it working | |
| 18:18:53 | melwitt | assuming that getting intercept to work might be not easy | |
| 18:19:42 | cdent | melwitt: it may require unwinding some of the fixture’s pieces, because the deal wsgi intercept is it takes away the need for fakes | |
| 18:19:57 | mriedem | i think i've got an easier workaround for now | |
| 18:19:57 | cdent | fake requests I mean, you use a real http client to make real http requests to a fake socket | |
| 18:20:13 | melwitt | cdent: ah, okay | |
| 18:20:45 | cdent | melwitt: I think it is probably worth doing regardless of the outcome here, but I’m probably a bit too biased to be the decider on such thing | |
| 18:21:08 | cdent | mriedem: i’m starting to die from lack of air, halp | |
| 18:21:54 | melwitt | cdent: I agree we should do it. just wanted to be clear that I wasn't suggesting we hold off on a workaround because of it | |
| 18:22:27 | openstackgerrit | Merged openstack/nova master: Don't cast cinderclient microversions to float https://review.openstack.org/486096 | |
| 18:22:46 | mriedem | sec | |
| 18:22:47 | mriedem | pushing it pu | |
| 18:22:49 | mriedem | *up | |
| 18:22:51 | melwitt | mriedem is trying to kill cdent | |
| 18:22:59 | cdent | I knew it | |
| 18:26:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass X-Auth-Token in TestOpenStackClient._authenticate https://review.openstack.org/486193 | |
| 18:28:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Pass X-Auth-Token in TestOpenStackClient._authenticate https://review.openstack.org/486193 | |
| 18:28:33 | mriedem | logstash is back up | |
| 18:28:52 | melwitt | f yes | |
| 18:29:30 | mriedem | language | |
| 18:32:34 | mriedem | doesn't seem to be finding anything thoguh | |
| 18:32:36 | mriedem | *though | |
| 18:32:39 | cdent | mriedem: interesting. I’m not sure your solution will work. It’s trying to prevent the 401 response from placement happening (which it will) but placement’s no auth will not response with the response.headers that the _authenticate method is supposed to provide to its callers | |
| 18:33:14 | cdent | placement no auth middleware doesn’t not set response headers | |
| 18:33:45 | cdent | so the retry thing that I did is more likely to get the desired outcome, isn’t it? (I’m not entirely sure, the gears withing gears isn’t clear) | |
| 18:34:01 | cdent | (double negative above not intentional) | |
| 18:34:06 | mriedem | yo'ure probably right, because the compute api noauth returns a request context, | |
| 18:34:10 | mriedem | that the compute api code requires | |
| 18:34:27 | mriedem | so even though we avoid the 401, the request won't have a context in it | |
| 18:34:50 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 18:34:51 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Translate the return value of attachment_create and _update https://review.openstack.org/486194 | |
| 18:35:20 | ildikov | mriedem: refactor + fixed old flow tests ^^ | |
| 18:36:42 | cdent | whatever the outcome, I can investigate the wsgi intercept stuff on monday, at least to spike it to see if it has any potential | |
| 18:36:45 | mriedem | cdent: let's un-DNM yours | |
| 18:36:49 | mriedem | and just push it in | |
| 18:36:55 | cdent | k, on it | |
| 18:39:43 | openstackgerrit | Chris Dent proposed openstack/nova master: retry on authentication failure in api_client https://review.openstack.org/486190 | |
| 18:41:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: retry on authentication failure in api_client https://review.openstack.org/486190 | |
| 18:41:12 | cdent | pushed that before I saw your coments melwitt and mriedem, got the NOTE anyway by chance, but efect typo still there | |
| 18:41:23 | mriedem | it is?!!?!??!?! | |
| 18:41:42 | melwitt | lol | |
| 18:41:58 | cdent | dood, I’m just going to let you do everything henceforth | |
| 18:42:06 | cdent | you are fleet of foot | |
| 18:42:20 | mriedem | i also have a light step | |
| 18:42:24 | mriedem | you'll never see it coming | |
| 18:42:56 | melwitt | well, that was fun | |
| 18:43:05 | mriedem | back to business | |
| 18:43:51 | cdent | this is me when you come for me mriedem https://www.youtube.com/watch?v=vZvbhy5lDgY | |
| 18:44:01 | melwitt | not getting any results in logstash, I see they're talking about some missing entries in #openstack-infra | |
| 18:44:15 | mriedem | i don't want to be the guy from blue velvet | |
| 18:44:22 | melwitt | haha | |
| 18:44:23 | mriedem | the 02 guy | |
| 18:44:54 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/485634 | |
| 18:45:42 | cdent | I’ve always felt a kinship for that guy in the oil, waiting to blow up. I think there ought to be a DSM-5 entry for that specific mental illness. | |
| 18:46:52 | melwitt | hah | |
| 18:48:15 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Translate the return value of attachment_create and _update https://review.openstack.org/486194 | |
| 18:48:16 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 18:52:51 | ildikov | mriedem: I cleaned up the above as much as I could, so you can review the refactor and the current content of the attach patch will not change either | |
| 18:53:12 | ildikov | mriedem: I will add separate new tests in the next patch set | |
| 18:53:52 | mriedem | ok | |
| 19:05:51 | openstackgerrit | Peter Hamilton proposed openstack/nova master: WIP Add trusted_certificates to REST API https://review.openstack.org/486204 | |
| 19:17:11 | figleaf | cdent: sorry, been battling connectivity issues most of the day | |
| 19:18:12 | figleaf | cdent: if I understood you, you'd like to simplify GET /resource_providers to just return those RPs that can satisfy *all* the requested resources themselves, ignoring shared resources | |
| 19:18:47 | figleaf | If so, then I would agree with that, if 'allocation_candidates' is how we're going to handle the fancy nova-specific stuff | |
| 19:28:52 | melwitt | I feel like I'm seeing tempest jobs time out fairly often lately too | |
| 19:29:36 | openstackgerrit | Chris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped https://review.openstack.org/486208 | |
| 19:30:50 | cfriesen | mriedem: dansmith: is this ^ the right way to handle the issue? I need to deal with the testcases still. | |
| 19:34:55 | sdague | melwitt: yeh, I noticed that as well. I haven't gone and looked yet | |
| 19:35:03 | sdague | melwitt: you have a link to some? | |
| 19:35:37 | melwitt | sdague: I think this is one, the most recent I've seen http://logs.openstack.org/75/419975/20/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/17f4349/ | |
| 19:36:54 | sdague | melwitt: also, if you are inclined, bringing back request logging under uwsgi - https://review.openstack.org/#/c/485602 | |
| 19:38:11 | melwitt | sdague: this one says "Request timed out" http://logs.openstack.org/88/485088/6/gate/gate-tempest-dsvm-cells-ubuntu-xenial/17bb8bf/logs/testr_results.html.gz | |
| 19:38:30 | melwitt | k, looking | |
| 19:53:34 | cdent | figleaf: that is what it is currently doing | |
| 19:54:01 | figleaf | cdent: ah, I thought it was taking shared resources into account, too | |
| 19:54:16 | cdent | it was supposed to be but that was never actually implemented | |
| 19:54:43 | figleaf | cdent: then if it isn't needed, I would prefer to leave it as is | |
| 19:54:52 | figleaf | since we're using allocation_candidates | |
| 19:54:59 | sdague | melwitt: yeh, trying to figure out with infra if this is persistent issue or not | |
| 19:55:11 | cdent | figleaf: yeah, that’s kinda where I landed too | |
| 19:55:49 | figleaf | cdent: maybe later if it is needed, we could add an additional query param to indicate "include shared" | |
| 19:58:10 | leakypipes | mriedem, cdent, melwitt, superdan, figleaf: I swear I need to hire someone to just keep rechecking my patches through the gate :( | |
| 19:58:16 | leakypipes | http://logs.openstack.org/66/483566/8/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/eda58dd/console.html | |
| 19:58:21 | leakypipes | latest random failure... | |
| 19:58:28 | melwitt | you and me both | |
| 19:58:33 | melwitt | recheck bots | |
| 19:58:42 | figleaf | leakypipes: I thought you had a team of lackeys for such mundane tasks | |
| 19:59:01 | leakypipes | honestly, our gate should see 1 test failure like that out of 1319 tempest tests and say to itself "fuck it, close enough. merge." | |
| 19:59:13 | melwitt | heh | |
| 20:00:09 | openstackgerrit | Chris Dent proposed openstack/nova master: retry on authentication failure in api_client https://review.openstack.org/486190 | |
| 20:00:27 | cdent | mriedem: you want +w that ^ again, had a pep8 failure | |
| 20:02:21 | openstackgerrit | Merged openstack/nova master: Change default policy to view quota details https://review.openstack.org/386008 | |
| 20:02:42 | cdent | leakypipes: that auth retry thing _may_ help the gate | |
| 20:03:59 | leakypipes | cdent: +Wallaby'd | |
| 20:04:20 | melwitt | whoa something merged, amaze | |
| 20:09:34 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - add alternate hosts https://review.openstack.org/486215 | |
| 20:09:47 | figleaf | leakypipes: ^^ very rough for now. Just wanted to get feedback on direction | |
| 20:10:45 | figleaf | leakypipes: planning on a subsequent patch to return allocation_candidates along with hosts, and then return both the alternates with the candidates | |
| 20:11:22 | figleaf | return both to the compute build_and_run_instances() call | |