| *** wanghao has joined #openstack-zaqar | 00:15 | |
| openstackgerrit | wanghao proposed openstack/zaqar-specs master: Service Queue in Zaqar https://review.openstack.org/449433 | 00:25 |
|---|---|---|
| *** zhurong has joined #openstack-zaqar | 00:39 | |
| openstackgerrit | wanghao proposed openstack/zaqar-specs master: Service Queue in Zaqar https://review.openstack.org/449433 | 00:48 |
| openstackgerrit | wanghao proposed openstack/zaqar-specs master: Service Queue in Zaqar https://review.openstack.org/449433 | 00:55 |
| *** XueFeng has joined #openstack-zaqar | 01:06 | |
| *** wanghao has quit IRC | 01:10 | |
| *** wanghao has joined #openstack-zaqar | 01:11 | |
| *** yanyanhu has joined #openstack-zaqar | 01:35 | |
| *** wanghao has quit IRC | 01:43 | |
| *** wanghao has joined #openstack-zaqar | 01:44 | |
| *** wanghao has quit IRC | 01:58 | |
| *** wanghao has joined #openstack-zaqar | 02:15 | |
| *** mpanetta has quit IRC | 02:23 | |
| *** mpanetta has joined #openstack-zaqar | 02:24 | |
| *** wanghao has quit IRC | 02:25 | |
| *** Kevin_Zheng has joined #openstack-zaqar | 02:54 | |
| *** rcernin has joined #openstack-zaqar | 03:59 | |
| *** rcernin is now known as rcernin|wfh|mtg | 03:59 | |
| openstackgerrit | Merged openstack/zaqar master: Add .idea pattern to .gitignore https://review.openstack.org/455223 | 03:59 |
| *** zhurong has quit IRC | 04:06 | |
| *** zhurong has joined #openstack-zaqar | 04:34 | |
| *** lhx__ has joined #openstack-zaqar | 04:43 | |
| *** lhx__ has quit IRC | 05:02 | |
| *** lhx__ has joined #openstack-zaqar | 05:02 | |
| *** jtomasek_ has joined #openstack-zaqar | 05:04 | |
| *** lhx__ has quit IRC | 05:05 | |
| *** rcernin|wfh|mtg has quit IRC | 05:11 | |
| *** lhx__ has joined #openstack-zaqar | 05:19 | |
| *** rcernin has joined #openstack-zaqar | 05:20 | |
| *** lhx__ has quit IRC | 05:42 | |
| *** jtomasek_ has quit IRC | 05:44 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 05:45 | |
| *** jtomasek has quit IRC | 05:49 | |
| *** pcaruana has joined #openstack-zaqar | 06:30 | |
| *** lhx__ has joined #openstack-zaqar | 06:44 | |
| *** tesseract has joined #openstack-zaqar | 06:49 | |
| *** rcernin has quit IRC | 06:59 | |
| *** lhx__ has quit IRC | 07:15 | |
| *** rcernin has joined #openstack-zaqar | 07:17 | |
| *** lhx__ has joined #openstack-zaqar | 07:18 | |
| *** nadya has joined #openstack-zaqar | 07:22 | |
| *** nadya has quit IRC | 07:32 | |
| *** nadya has joined #openstack-zaqar | 07:33 | |
| *** rcernin has quit IRC | 07:37 | |
| *** rcernin_ has joined #openstack-zaqar | 07:37 | |
| *** rcernin_ has quit IRC | 07:38 | |
| *** rcernin_ has joined #openstack-zaqar | 07:38 | |
| *** jaosorior has joined #openstack-zaqar | 07:39 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/zaqar master: swift driver: rely on keystone session for swift connection https://review.openstack.org/456098 | 07:42 |
| jaosorior | therve: could you check that out? ^^ I had some issues in my tripleO deployment where zaqar's token would expire. Using that actually avoids that issue | 07:43 |
| therve | Hum | 07:44 |
| therve | jaosorior, How did the issue show up? | 07:44 |
| jaosorior | by failing when I did the deploy. It would just give me a 401 IIRC | 07:44 |
| jaosorior | after trying to deploy the next day | 07:45 |
| *** nadya has quit IRC | 07:45 | |
| *** lhx__ has quit IRC | 07:45 | |
| therve | Weird | 07:46 |
| jaosorior | therve: either way, zaqar shouldn't be trying to handle reauthentication and neither should swiftclient | 07:46 |
| jaosorior | lets just let the session object do it's thing :) | 07:46 |
| *** lhx__ has joined #openstack-zaqar | 07:47 | |
| therve | Well, if that works sure. I haven't had good experience with keystone client libs in the past :) | 07:47 |
| *** lhx__ has quit IRC | 07:47 | |
| jaosorior | nobody has | 07:47 |
| therve | jaosorior, While your approach is certainly better, I'd like to understand why mine doesn't work | 07:50 |
| jaosorior | therve: I actually have no idea dude, all I know is I was getting 401's cause it didn't reauthenticate | 07:53 |
| jaosorior | therve: might wanna ask jamielennox from the keystone team | 07:54 |
| *** nadya has joined #openstack-zaqar | 07:56 | |
| *** nadya has quit IRC | 08:00 | |
| *** nadya has joined #openstack-zaqar | 08:01 | |
| jaosorior | therve: when would the getattr be called? | 08:06 |
| *** nadya has quit IRC | 08:06 | |
| therve | jaosorior, In theory every time a call to swift is made | 08:18 |
| jaosorior | funky | 08:21 |
| jaosorior | therve: oh https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L56 | 08:23 |
| jaosorior | therve: might be that you needed to fetch a new AccessInfo after calling that get_auth_ref | 08:24 |
| therve | jaosorior, Oh, of course. I'm not actually validating the token | 08:35 |
| therve | jaosorior, Thanks, that looks awesome. Reduces keystone requests quite a bit too | 08:39 |
| therve | It might help my tripleo timing issue | 08:40 |
| therve | I'm afraid I may have broken something else though | 08:47 |
| therve | Sigh | 08:47 |
| jaosorior | therve: where? | 08:48 |
| therve | jaosorior, See the ooo build on your patch | 08:48 |
| jaosorior | shit http://logs.openstack.org/98/456098/1/check/gate-tripleo-ci-centos-7-scenario002-multinode-oooq-puppet-nv/f6f334e/logs/undercloud/home/jenkins/overcloud_deploy.log.txt.gz#_2017-04-12_08_19_33 | 08:51 |
| therve | Yeah | 08:51 |
| therve | jaosorior, Probably because of https://review.openstack.org/455382 ? | 08:51 |
| therve | I can never remember what's used where | 08:51 |
| jaosorior | therve: maybe, is that set in the instack-undercloud? | 08:52 |
| therve | I don't know :/ | 08:52 |
| jaosorior | haha shit | 08:52 |
| jaosorior | well | 08:53 |
| jaosorior | for a strange reason | 08:53 |
| jaosorior | it's getting a lot of 404s from swift http://logs.openstack.org/98/456098/1/check/gate-tripleo-ci-centos-7-scenario002-multinode-oooq-puppet-nv/f6f334e/logs/undercloud/var/log/zaqar/zaqar.log.txt.gz | 08:53 |
| therve | I think those are ok | 08:53 |
| jaosorior | oh, alright | 08:53 |
| jaosorior | therve: so yeah, gotta check if it's set in instack-undercloud | 08:55 |
| therve | jaosorior, http://logs.openstack.org/98/456098/1/check/gate-tripleo-ci-centos-7-scenario002-multinode-oooq-puppet-nv/f6f334e/logs/undercloud/var/log/messages.txt.gz#_Apr_12_08_09_42 | 08:55 |
| jaosorior | therve: would it be that zaqar started before keystone? | 08:55 |
| therve | Maybe, but it fails with a 401 later on | 08:56 |
| jaosorior | oh, I didn't see the 401 | 08:57 |
| jaosorior | was just seeing connection refused | 08:57 |
| therve | bbiab | 08:57 |
| therve | O | 08:59 |
| therve | Oh | 08:59 |
| therve | jaosorior, Don't create the session in init | 09:00 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/zaqar master: swift driver: rely on keystone session for swift connection https://review.openstack.org/456098 | 09:01 |
| jaosorior | therve: something like that? ^^ | 09:01 |
| jaosorior | therve: oh, I guess it worked in my deployment cause I had already everything started. | 09:03 |
| therve | jaosorior, I think we want to keep the session object, otherwise we'll fetch tokens every time | 09:31 |
| jaosorior | uhm... | 09:31 |
| therve | jaosorior, Basically keep refresh_auth, just set the session instead of the token | 09:35 |
| jaosorior | therve: I'm not sure if I get it, can you modify the commit? | 09:39 |
| therve | I won't be able to review it :D | 09:40 |
| therve | jaosorior, Something like that: http://paste.openstack.org/show/606247/ | 09:42 |
| therve | Maybe rename it to _set_session or something | 09:43 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/zaqar master: swift driver: rely on keystone session for swift connection https://review.openstack.org/456098 | 09:51 |
| jaosorior | therve: lets give it a try ^ | 09:51 |
| therve | jaosorior, a bunch of issues: self.auth is gone, and you can remove self.url | 09:52 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/zaqar master: swift driver: rely on keystone session for swift connection https://review.openstack.org/456098 | 09:53 |
| jaosorior | weeell, that's not a bunch :P | 09:53 |
| jaosorior | anyway, fixed | 09:53 |
| therve | jaosorior, Still got self.url in init_auth :p | 09:56 |
| therve | Also init_auth doesn't take an arg | 09:56 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/zaqar master: swift driver: rely on keystone session for swift connection https://review.openstack.org/456098 | 10:04 |
| *** yanyanhu has quit IRC | 10:50 | |
| *** zhurong has quit IRC | 11:26 | |
| openstackgerrit | Feilong Wang proposed openstack/zaqar master: Fix queues actions for pooling https://review.openstack.org/453440 | 11:38 |
| openstackgerrit | Feilong Wang proposed openstack/zaqar master: Fix queues actions for pooling https://review.openstack.org/453440 | 11:47 |
| jaosorior | therve: what do you think of Jamie's comment https://review.openstack.org/#/c/456098/ ? | 11:51 |
| *** shu-mutou is now known as shu-mutou-AWAY | 11:55 | |
| therve | jaosorior, The whole point of the class is to *not* save the client :) | 12:14 |
| jaosorior | therve: ok, then I guess it's fine as it is | 12:14 |
| *** mpanetta has quit IRC | 12:24 | |
| *** mpanetta has joined #openstack-zaqar | 12:25 | |
| openstackgerrit | Thomas Herve proposed openstack/zaqar master: Add missing modules to config generator https://review.openstack.org/456213 | 13:14 |
| *** zhurong has joined #openstack-zaqar | 13:46 | |
| *** zhurong_ has joined #openstack-zaqar | 13:59 | |
| *** zhurong has quit IRC | 14:01 | |
| *** rbrady is now known as rbrady-afk | 14:35 | |
| *** wxy| has joined #openstack-zaqar | 14:47 | |
| *** zhurong_ has quit IRC | 14:57 | |
| *** jaosorior is now known as jaosorior_away | 15:02 | |
| *** rcernin_ has quit IRC | 15:04 | |
| *** tesseract has quit IRC | 15:58 | |
| *** Kevin_Zheng has quit IRC | 16:13 | |
| *** wxy| has quit IRC | 16:48 | |
| *** rbrady-afk is now known as rbrady | 17:14 | |
| *** jtomasek has joined #openstack-zaqar | 17:45 | |
| *** jtomasek has quit IRC | 17:46 | |
| *** jtomasek has joined #openstack-zaqar | 17:46 | |
| *** jtomasek has quit IRC | 17:47 | |
| *** jtomasek has joined #openstack-zaqar | 17:49 | |
| *** jtomasek_ has joined #openstack-zaqar | 17:50 | |
| *** jtomasek has quit IRC | 17:53 | |
| *** jtomasek_ has quit IRC | 17:56 | |
| *** nadya has joined #openstack-zaqar | 18:53 | |
| *** nadya has quit IRC | 18:57 | |
| *** nadya has joined #openstack-zaqar | 18:57 | |
| *** nadya has quit IRC | 19:01 | |
| openstackgerrit | Feilong Wang proposed openstack/zaqar master: Fix queues actions for pooling https://review.openstack.org/453440 | 20:19 |
| *** mpanetta has quit IRC | 20:50 | |
| *** mpanetta has joined #openstack-zaqar | 20:51 | |
| -openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup. | 21:27 | |
| *** Qiming has quit IRC | 21:41 | |
| *** Qiming has joined #openstack-zaqar | 21:46 | |
| *** pcaruana has quit IRC | 21:51 | |
| openstackgerrit | Feilong Wang proposed openstack/zaqar master: Fix queues actions for pooling https://review.openstack.org/453440 | 23:18 |
| openstackgerrit | Feilong Wang proposed openstack/zaqar master: Fix queues actions for pooling https://review.openstack.org/453440 | 23:35 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!