| *** zigo has quit IRC | 00:05 | |
| *** mvkr has quit IRC | 00:27 | |
| *** mvkr has joined #openstack-swift | 00:38 | |
| *** sorrison has joined #openstack-swift | 00:52 | |
| *** two_tired2 has joined #openstack-swift | 02:24 | |
| openstackgerrit | Fatema Khalid Sherif proposed openstack/swift master: Enabling direct_client users to overwrite X-Timestamp https://review.openstack.org/603261 | 02:32 |
|---|---|---|
| *** psachin has joined #openstack-swift | 02:43 | |
| zaitcev | Anyone has an idea why a review would be stuck after +A, and if "reverify" is the right approach to restart it? | 03:09 |
| zaitcev | https://review.openstack.org/#/c/599042/ | 03:09 |
| patchbot | patch 599042 - swift - py3: Port listing_formats middleware - 4 patch sets | 03:09 |
| zaitcev | OHHH | 03:09 |
| zaitcev | It depends on a bunch, I see | 03:09 |
| openstackgerrit | Pete Zaitcev proposed openstack/swift master: py3: Adapt db.py https://review.openstack.org/581905 | 03:23 |
| *** fatema_ has joined #openstack-swift | 03:56 | |
| fatema_ | Good Morning | 03:57 |
| fatema_ | Is there something up with Zuul? all tests for my patch are queued for more than an hour now. | 03:58 |
| kota_ | good afternoon | 04:02 |
| notmyname | good evening | 04:07 |
| notmyname | fatema_: yeah, from http://zuul.openstack.org it seems like the jobs are taking a long time | 04:08 |
| fatema_ | ok cool it's nothing with my patch but I hoped I could have it tested before going to college. | 04:12 |
| fatema_ | notmyname, Thank you | 04:12 |
| notmyname | fatema_: can you link your patch again? | 04:13 |
| fatema_ | patch 603261 | 04:14 |
| patchbot | https://review.openstack.org/#/c/603261/ - swift - Enabling direct_client users to overwrite X-Timestamp - 5 patch sets | 04:14 |
| fatema_ | notmyname, ^^ | 04:14 |
| *** rcernin has quit IRC | 04:24 | |
| *** e0ne has joined #openstack-swift | 04:28 | |
| *** rcernin has joined #openstack-swift | 04:38 | |
| fatema_ | well my patch almost done with testing. Out of curiosity, you pushed Zuul to do my testing job first, notmyname ? | 04:47 |
| notmyname | nope. I don't have that power | 04:47 |
| notmyname | in the check queue, each patch is tested by itself. in the gate queue, it's tested pretty much in the order it was approved, along with approved patches from a bunch of other projects | 04:48 |
| notmyname | in all cases, the jobs are run when VMs are available to run them | 04:48 |
| *** e0ne has quit IRC | 04:51 | |
| *** e0ne has joined #openstack-swift | 05:09 | |
| *** e0ne has quit IRC | 05:10 | |
| fatema_ | notmyname, ok makes sense | 05:13 |
| fatema_ | and the patch 603261 is waiting for the reviews :D | 05:15 |
| patchbot | https://review.openstack.org/#/c/603261/ - swift - Enabling direct_client users to overwrite X-Timestamp - 5 patch sets | 05:15 |
| *** fatema_ has quit IRC | 05:16 | |
| *** rcernin_ has joined #openstack-swift | 05:17 | |
| *** rcernin has quit IRC | 05:19 | |
| zaitcev | Noobs are kinda impatient, le sigh | 05:26 |
| *** two_tired2 has quit IRC | 05:40 | |
| *** pcaruana has joined #openstack-swift | 05:41 | |
| *** rcernin_ has quit IRC | 07:05 | |
| *** hseipp has joined #openstack-swift | 07:12 | |
| *** psachin has quit IRC | 07:41 | |
| *** psachin has joined #openstack-swift | 07:49 | |
| *** e0ne has joined #openstack-swift | 07:52 | |
| *** e0ne has quit IRC | 07:53 | |
| *** silor has joined #openstack-swift | 08:05 | |
| *** mikecmpbll has joined #openstack-swift | 08:23 | |
| *** mikecmpbll has quit IRC | 08:58 | |
| *** e0ne has joined #openstack-swift | 08:59 | |
| *** mikecmpbll has joined #openstack-swift | 08:59 | |
| *** e0ne has quit IRC | 10:12 | |
| *** rcernin_ has joined #openstack-swift | 10:43 | |
| *** rcernin_ has quit IRC | 10:52 | |
| *** e0ne has joined #openstack-swift | 10:59 | |
| *** mark-mcardle has quit IRC | 11:00 | |
| *** joeljwright has quit IRC | 11:01 | |
| *** joeljwright has joined #openstack-swift | 11:01 | |
| *** ChanServ sets mode: +v joeljwright | 11:01 | |
| *** pcaruana has quit IRC | 11:15 | |
| *** e0ne_ has joined #openstack-swift | 11:22 | |
| *** e0ne has quit IRC | 11:25 | |
| *** psachin has quit IRC | 12:07 | |
| *** ujjain- has joined #openstack-swift | 13:06 | |
| *** ujjain- is now known as ujjain | 13:09 | |
| *** ujjain has left #openstack-swift | 13:10 | |
| *** mikecmpbll has quit IRC | 13:11 | |
| *** mikecmpbll has joined #openstack-swift | 13:13 | |
| *** e0ne_ has quit IRC | 14:10 | |
| *** e0ne has joined #openstack-swift | 14:10 | |
| *** two_tired2 has joined #openstack-swift | 14:15 | |
| *** two_tired2 has quit IRC | 14:30 | |
| *** silor has quit IRC | 14:54 | |
| *** silor has joined #openstack-swift | 15:12 | |
| *** e0ne has quit IRC | 15:37 | |
| *** silor has quit IRC | 15:40 | |
| *** silor has joined #openstack-swift | 15:44 | |
| *** mikecmpbll has quit IRC | 15:54 | |
| *** mikecmpbll has joined #openstack-swift | 15:56 | |
| *** _d34dh0r53_ is now known as d34dh0r53 | 16:05 | |
| *** pcaruana has joined #openstack-swift | 16:07 | |
| *** silor has quit IRC | 16:10 | |
| notmyname | good morning | 16:10 |
| *** hseipp has quit IRC | 16:26 | |
| *** mikecmpbll has quit IRC | 16:44 | |
| timburke | zaitcev: yeah, i was gonna mention :-) | 16:45 |
| timburke | really, it just depends on https://review.openstack.org/#/c/578958/ -- the other two in Related Changes require the listing_formats patch | 16:46 |
| patchbot | patch 578958 - swift - py3: get proxy-server willing and able to respond ... - 6 patch sets | 16:46 |
| timburke | also, even relatively old hats like me are impatient :P | 16:47 |
| *** tdasilva has joined #openstack-swift | 16:52 | |
| *** ChanServ sets mode: +v tdasilva | 16:52 | |
| zaitcev | Oh really. I thought 578958 on top of the chain... | 16:57 |
| zaitcev | Didn't even look at it. I'll take a look now. | 16:58 |
| timburke | yeah, chains read bottom-up | 17:04 |
| *** zaitcev has quit IRC | 17:14 | |
| *** zaitcev has joined #openstack-swift | 17:29 | |
| *** ChanServ sets mode: +v zaitcev | 17:29 | |
| notmyname | zaitcev: feel free to +A if you think these are ready to land: https://review.openstack.org/#/c/602822/ and https://review.openstack.org/#/c/602821/ | 17:53 |
| patchbot | patch 602822 - swift - py3: port direct_client - 2 patch sets | 17:53 |
| patchbot | patch 602821 - swift - py3: port request_helpers - 1 patch set | 17:53 |
| notmyname | mattoliverau: you left a +2 and a comment saying you'd let it sit a bit then you'd come back and +A on https://review.openstack.org/#/c/569884/ | 17:54 |
| patchbot | patch 569884 - swift - Clean up direct_client's gen_headers - 2 patch sets | 17:54 |
| notmyname | https://review.openstack.org/#/c/447129/ has a +2 from both clayg and kota_. are we waiting for anything else, or do we need more than those two to review before landing it? | 17:54 |
| patchbot | patch 447129 - swift - Configure diskfile per storage policy - 20 patch sets | 17:54 |
| clayg | yes, tdasilva was going to something something gluster? | 17:55 |
| clayg | i was all ready to merge! | 17:55 |
| notmyname | clayg: ah! that's right. ok, tdasilva, it's waiting on you then :-) | 17:55 |
| notmyname | clayg: you've got a +2 on https://review.openstack.org/#/c/569837/ but no +A | 17:56 |
| patchbot | patch 569837 - swift - Fix-up signal handling - 1 patch set | 17:56 |
| timburke | notmyname: oh! no *wonder* i already had so many opinions about p 603261! | 17:56 |
| patchbot | https://review.openstack.org/#/c/603261/ - swift - Enabling direct_client users to overwrite X-Timestamp - 5 patch sets | 17:56 |
| timburke | i shoulda put a Closes-Bug on p 569884 | 17:57 |
| patchbot | https://review.openstack.org/#/c/569884/ - swift - Clean up direct_client's gen_headers - 2 patch sets | 17:57 |
| notmyname | timburke: I rechecked https://review.openstack.org/#/c/535981/ since last time it was run was january. if it still passes, looks like an easy one to click +A on | 17:59 |
| patchbot | patch 535981 - swift - Clean up comment about 404s leaking out during COPYs - 1 patch set | 17:59 |
| timburke | look at you, cleaning up old patches! | 17:59 |
| notmyname | zaitcev: I found one more with your +2 and no +A and no comment implying some follow-up.. https://review.openstack.org/#/c/490730/ | 17:59 |
| patchbot | patch 490730 - swift - Replacing six.iter/dict.iter usages - 4 patch sets | 17:59 |
| zaitcev | notmyname: give me a few | 18:00 |
| notmyname | no worries. despite this being IRC, I'm throwing this stuff into the channel as more of a "oh by the way...". handle it as async as needed. it's not like the patch from 6 months ago is going to get too much worse with another few hours of sitting there ;-) | 18:01 |
| notmyname | ok, that's most of the patches that could land according to existing votes and also look perhaps simple enough to actually go ahead and land | 18:02 |
| notmyname | there's a few more that are either extra old or perhaps needing a bit more examination before making them rise from the dead | 18:03 |
| tdasilva | notmyname, clayg: re p 447129 i'm still waiting to hear back from gluster guys | 18:04 |
| patchbot | https://review.openstack.org/#/c/447129/ - swift - Configure diskfile per storage policy - 20 patch sets | 18:04 |
| notmyname | FWIW the ones that are more complicated are https://review.openstack.org/#/c/463849/ and https://review.openstack.org/#/c/437196/ | 18:05 |
| patchbot | patch 463849 - swift - Delete a non-SLO object with ?multipart-manifest. - 2 patch sets | 18:05 |
| patchbot | patch 437196 - swift - Add X-Backend-Versioning-Mode-Override - 2 patch sets | 18:05 |
| clayg | just start merging stuff until someone tells you to stop | 18:05 |
| timburke | zaitcev: don't worry about p 490730 -- lgtm | 18:06 |
| patchbot | https://review.openstack.org/#/c/490730/ - swift - Replacing six.iter/dict.iter usages - 4 patch sets | 18:06 |
| timburke | yeah, the SLO one... people seemed on the fence about it for sure | 18:07 |
| timburke | on the versioning override, i've got an interest in that getting in so we can revisit some swift3 patches (and apply them to s3api): p 436568, p 436569, p 436570, p 436571 | 18:10 |
| patchbot | https://review.openstack.org/#/c/436568/ - swift3 - Implement versioning status API - 4 patch sets | 18:10 |
| patchbot | https://review.openstack.org/#/c/436569/ - swift3 - Listing of versioned objects - 2 patch sets | 18:10 |
| patchbot | https://review.openstack.org/#/c/436570/ - swift3 - Modify obj GETorHEAD to support versions (ABANDONED) - 3 patch sets | 18:10 |
| patchbot | https://review.openstack.org/#/c/436571/ - swift3 - Delete versioned objects - 2 patch sets | 18:10 |
| timburke | oh! and p 437523! gotta look at that again... | 18:11 |
| patchbot | https://review.openstack.org/#/c/437523/ - swift - Store version id when copying object to archive - 8 patch sets | 18:11 |
| zaitcev | Oh yeah, good job, Tim. Less for me to approve :-) | 18:16 |
| timburke | anything i can do to keep your attention on the py3 chains ;-) | 18:17 |
| *** f0o has quit IRC | 18:20 | |
| *** f0o has joined #openstack-swift | 18:29 | |
| *** mikecmpbll has joined #openstack-swift | 18:43 | |
| *** guimaluf has joined #openstack-swift | 18:52 | |
| zaitcev | timburke: do you have a py35 somewhere ready | 18:53 |
| clayg | is p 604937 going to recheck? | 19:10 |
| patchbot | https://review.openstack.org/#/c/604937/ - swift - Allow kmip_keymaster to be configured in proxy-ser... - 1 patch set | 19:10 |
| timburke | zaitcev: i do | 19:12 |
| zaitcev | timburke, sorry, but mine went down and it's a long story | 19:12 |
| zaitcev | does this work json.loads(b'[]') | 19:13 |
| zaitcev | works on py36 but... | 19:13 |
| zaitcev | https://review.openstack.org/#/c/578958/6/swift/common/middleware/symlink.py | 19:13 |
| patchbot | patch 578958 - swift - py3: get proxy-server willing and able to respond ... - 6 patch sets | 19:13 |
| timburke | zaitcev: nope :( TypeError: the JSON object must be str, not 'bytes' | 19:14 |
| zaitcev | I see, thanks a lot | 19:14 |
| zaitcev | Sadly, some wise guy decided that we're stuck with py35 for the next 10 years | 19:14 |
| zaitcev | Because RHEL 8 | 19:14 |
| timburke | (at least, on 3.5.5) | 19:14 |
| zaitcev | As if having py26 was not bad enough | 19:14 |
| timburke | fwiw, they apparently viewed https://bugs.python.org/issue17909 as an enhancement; i woulda called it a bug... | 19:17 |
| timburke | oh yeah, and i wonder if i oughta bug someone about https://github.com/python/cpython/pull/7932 ... | 19:22 |
| timburke | our func tests totally fall under the "previously working (if misbehaving) clients" that will start getting HTTP/0.9 errors | 19:24 |
| zaitcev | BTW | 19:25 |
| zaitcev | n/m | 19:26 |
| *** e0ne has joined #openstack-swift | 19:42 | |
| timburke | clayg: looks like it's ~80th in line? gate's backed-up | 19:43 |
| clayg | neat | 19:43 |
| openstackgerrit | Merged openstack/swift master: py3: port request_helpers https://review.openstack.org/602821 | 20:01 |
| *** e0ne has quit IRC | 20:05 | |
| openstackgerrit | Merged openstack/swift master: py3: get proxy-server willing and able to respond to some API requests https://review.openstack.org/578958 | 20:13 |
| openstackgerrit | Merged openstack/swift master: py3: Port listing_formats middleware https://review.openstack.org/599042 | 20:13 |
| *** e0ne has joined #openstack-swift | 20:19 | |
| *** pcaruana has quit IRC | 20:43 | |
| *** e0ne has quit IRC | 20:43 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: py3: port direct_client https://review.openstack.org/602822 | 20:47 |
| openstackgerrit | Tim Burke proposed openstack/swift master: py3: port internal_client https://review.openstack.org/602823 | 20:47 |
| *** mcape has joined #openstack-swift | 21:14 | |
| timburke | hi mcape! | 21:14 |
| mcape | hi! i see another error message in logs | 21:15 |
| mcape | http://paste.openstack.org/show/730907/ | 21:15 |
| timburke | let's look at your proxy-server logs. in particular, i'm looking for something like 'HTTP connection exception' or 'Error on keystone reply'... ah, good, there it is :-) | 21:16 |
| timburke | that's quite the port number | 21:16 |
| mcape | wow | 21:16 |
| timburke | i think you maybe meant 35357 instead of 353573? | 21:17 |
| timburke | ;-) | 21:17 |
| mcape | it happened when i removed '/v3' in vi | 21:18 |
| mcape | it works now | 21:18 |
| mcape | HURRAY! | 21:19 |
| timburke | yay! | 21:19 |
| * mcape jumps around | 21:19 | |
| timburke | glad it was relatively easy | 21:19 |
| mcape | thank you very much for your help | 21:19 |
| timburke | fwiw i wrote up https://bugs.launchpad.net/swift/+bug/1794374 -- that client error was pretty terrible | 21:19 |
| openstack | Launchpad bug 1794374 in OpenStack Object Storage (swift) "s3token's _deny_request is busted" [Undecided,New] | 21:19 |
| timburke | i suppose we could also try for some validation on port numbers? i'm on the fence about the value of that... | 21:20 |
| mcape | perhaps rocky's proxy-server.conf should contain auth_version variable with some comments | 21:21 |
| mcape | or release notes... i read them, but this information was not there (the need to have auth_version set in proxy-server.conf) | 21:24 |
| mcape | anyway thank you very much! probably i'd just downgraded tomorrow early morning without your help | 21:24 |
| * mcape is very happy and grateful | 21:38 | |
| *** mcape has quit IRC | 21:38 | |
| openstackgerrit | Sam Morrison proposed openstack/swift master: s3 secret caching https://review.openstack.org/603529 | 22:15 |
| timburke | thanks sorrison! | 22:17 |
| sorrison | timburke: kinda feel slightly dirty doing a catch all but I think that is best? | 22:17 |
| *** mvkr has quit IRC | 22:22 | |
| *** mvkr has joined #openstack-swift | 22:35 | |
| *** rcernin has joined #openstack-swift | 23:07 | |
| mattoliverau | morning | 23:19 |
| notmyname | mattoliverau: hello | 23:20 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!