| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-21 | |||
| 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 | |
| 20:16:49 | openstackgerrit | Merged openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751 | |
| 20:17:58 | melwitt | sdague: what's the idea here? if you get an exception while logging, log again? https://review.openstack.org/#/c/485602/5/nova/api/openstack/requestlog.py@92 | |
| 20:19:38 | sdague | melwitt: because of the way the paste pipelines work, if we get an unexpected exception it's going to end up going through that layer | |
| 20:19:49 | sdague | and handled at the fault layer above it | |
| 20:20:16 | sdague | if the log message was not emitted in the except block, you'd never have a record of the request | |
| 20:20:23 | sdague | you'd just have the fault error | |
| 20:20:34 | melwitt | sdague: ah, cool. thanks | |
| 20:21:06 | mriedem | sdague: in case you didn't know, the neutron linuxbridge job is the one that has been timing out all week | |
| 20:21:12 | mriedem | kevinbenton: ^ know what's going on there? | |
| 20:21:22 | sdague | mriedem: it's also set at 130minutes | |
| 20:21:26 | sdague | which is shorter than the rest | |
| 20:21:57 | sdague | I've definitely seen passing neutron jobs at 1:45 and 2:00 | |
| 20:22:06 | sdague | which means everything is just long | |
| 20:22:27 | sdague | so a 130 is going to fail just because of statistics on slow node events | |
| 20:22:32 | melwitt | huh, interesting. I would have assumed they all had the same timeout | |
| 20:22:55 | sdague | I think at some point the linux bridge job was running less stuff | |
| 20:23:06 | sdague | but who knows :) | |
| 20:23:43 | melwitt | heh | |
| 20:27:29 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Expect id and disabled_reason in GET /os-services response https://review.openstack.org/485409 | |
| 20:27:29 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.53 - services and hypervisors using UUIDs https://review.openstack.org/485435 | |
| 20:52:07 | cfriesen | has anyone considered a "run this command in parallel on all cells and aggregate the responses" helper function? there seem to be quite a few "for cell in cells:" loops in the code. | |
| 20:53:40 | cfriesen | separate issue...nova/compute/api.py has a CELLS variable. How does this get refreshed to pick up new cells created after the variable was initially set? | |
| 20:53:58 | melwitt | cfriesen: yes https://github.com/openstack/nova/blob/master/nova/context.py#L531 | |
| 20:54:13 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |