| *** openstack has joined #openstack-swift | 00:23 | |
| *** ChanServ sets mode: +o openstack | 00:23 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: lock_path: Prevent aquiring deleted/moved locks https://review.opendev.org/c/openstack/swift/+/791022 | 00:52 |
|---|---|---|
| openstackgerrit | Tim Burke proposed openstack/swift master: DNM: make sure we really can turn off strict locks https://review.opendev.org/c/openstack/swift/+/792307 | 00:52 |
| *** gyee has quit IRC | 02:31 | |
| *** fingo has quit IRC | 04:57 | |
| *** fingo has joined #openstack-swift | 04:57 | |
| *** fingo has quit IRC | 04:58 | |
| *** fingo_ has joined #openstack-swift | 04:58 | |
| *** fingo_ is now known as fingo | 04:58 | |
| *** fingo has quit IRC | 05:00 | |
| *** fingo has joined #openstack-swift | 05:01 | |
| openstackgerrit | Matthew Oliver proposed openstack/swift master: WIP: sharding: Allow rows_per_shard to be 'auto' https://review.opendev.org/c/openstack/swift/+/791340 | 05:42 |
| *** rpittau|afk is now known as rpittau | 07:24 | |
| *** fyx has quit IRC | 07:34 | |
| *** donnyd has quit IRC | 07:34 | |
| *** clayg has quit IRC | 07:35 | |
| *** gmann has quit IRC | 07:36 | |
| *** nicolasbock has quit IRC | 07:36 | |
| *** clayg has joined #openstack-swift | 07:37 | |
| *** ChanServ sets mode: +v clayg | 07:37 | |
| *** fyx has joined #openstack-swift | 07:37 | |
| *** nicolasbock has joined #openstack-swift | 07:37 | |
| *** donnyd has joined #openstack-swift | 07:38 | |
| *** gmann has joined #openstack-swift | 07:38 | |
| *** rcernin has quit IRC | 07:52 | |
| *** rcernin has joined #openstack-swift | 09:04 | |
| *** jv has quit IRC | 09:36 | |
| *** rcernin has quit IRC | 09:38 | |
| *** jv has joined #openstack-swift | 09:43 | |
| *** rcernin has joined #openstack-swift | 09:53 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Use ContainerSharderConf class in sharder and manage-shard-ranges https://review.opendev.org/c/openstack/swift/+/792177 | 10:14 |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Add absolute values for shard shrinking config options https://review.opendev.org/c/openstack/swift/+/792182 | 10:14 |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Use ContainerSharderConf class in sharder and manage-shard-ranges https://review.opendev.org/c/openstack/swift/+/792177 | 10:20 |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Add absolute values for shard shrinking config options https://review.opendev.org/c/openstack/swift/+/792182 | 10:20 |
| *** rcernin has quit IRC | 12:14 | |
| *** m75abrams has joined #openstack-swift | 12:31 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Extra unit test for relinker lock cleanup https://review.opendev.org/c/openstack/swift/+/792369 | 12:41 |
| *** fingo has quit IRC | 13:00 | |
| *** fingo has joined #openstack-swift | 13:00 | |
| *** m75abrams has quit IRC | 14:11 | |
| *** godog has quit IRC | 14:12 | |
| *** godog has joined #openstack-swift | 14:13 | |
| *** godog has quit IRC | 14:13 | |
| *** godog has joined #openstack-swift | 14:14 | |
| *** godog has quit IRC | 14:15 | |
| *** godog has joined #openstack-swift | 14:16 | |
| *** godog has quit IRC | 14:31 | |
| *** godog has joined #openstack-swift | 14:31 | |
| *** TiagoPrimini has quit IRC | 14:48 | |
| openstackgerrit | wdoekes proposed openstack/swift master: tempurl: Fix PUT upload to temp url https://review.opendev.org/c/openstack/swift/+/792397 | 15:11 |
| *** fozboz has joined #openstack-swift | 15:32 | |
| fozboz | I have a question about the recent update to the s3api that allows CORS preflight. It looks like these calls will respond with 200 ONLY if 'authorization' is in the 'Access-Control-Request-Headers' header. Why? Applications might not necessarily include that. | 15:39 |
| *** rpittau is now known as rpittau|afk | 15:50 | |
| openstackgerrit | Pete Zaitcev proposed openstack/swift master: Use a less bogus credit for Melissa Ma Lei https://review.opendev.org/c/openstack/swift/+/792405 | 15:58 |
| *** fozboz has quit IRC | 16:43 | |
| *** fozboz has joined #openstack-swift | 17:07 | |
| *** TiagoPrimini has joined #openstack-swift | 17:53 | |
| timburke_ | fozboz, mainly it comes down to needing a way to identify that it's an s3api request (as opposed to a swift request). are you seeing applications using pre-signed urls or something? we might be able to look for things in query strings... not sure if a browser would include the query string in the OPTIONS request, though | 18:23 |
| fozboz | timburke_: yes exactly, the application is using a pre-signed url, and the only value it is passing in the 'Access-Control-Request-Headers' header is 'x-requested-with', which ultimately returns a 405 | 18:27 |
| timburke_ | fozboz, am i right to think that the preflight doesn't include the query string? | 18:28 |
| fozboz | No it is included | 18:33 |
| fozboz | E.g. OPTIONShttps://mycluster.com/container/object?response-content-disposition=attachment; filename*=UTF-8''ReadMe.txt&response-content-type=text/plain&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Date=20210520T170001Z&X-Amz-SignedHeaders=host&X-Amz-Expires=7200&X-Amz-Credential=0123456789abcdef/20210520/us-east-1/s3/aws4_request&X-Amz-Signature=somesignature | 18:34 |
| fozboz | Perhaps my application is handling the call incorrectly? | 18:34 |
| openstackgerrit | Tim Burke proposed openstack/swift master: tempurl: Fix PUT upload to temp url on py3 https://review.opendev.org/c/openstack/swift/+/792397 | 18:51 |
| timburke_ | fozboz, oh, cool! that gives us something to work with, then :-) | 18:52 |
| timburke_ | fwiw, i trust observed behaviors a *lot* more than my own ability to judge "correctness" when it comes to CORS ;-) | 18:53 |
| fozboz | At least I think it is. We've been troubleshooting CORS issues with different applications all week and I'm not sure which way is up anymore! | 18:54 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Remove some getattr usage in s-m-s-r https://review.opendev.org/c/openstack/swift/+/792476 | 19:16 |
| openstackgerrit | Pete Zaitcev proposed openstack/swift master: Make dark data watcher ignore the newly updated objects https://review.opendev.org/c/openstack/swift/+/788398 | 19:41 |
| zaitcev | That scared me: took a very long time for "git review" to complete. | 19:41 |
| *** fozboz has quit IRC | 19:43 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Use ContainerSharderConf class in sharder and manage-shard-ranges https://review.opendev.org/c/openstack/swift/+/792177 | 20:00 |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Add absolute values for shard shrinking config options https://review.opendev.org/c/openstack/swift/+/792182 | 20:00 |
| acoles | zaitcev - I just had same experience, slow git review | 20:01 |
| *** rcernin has joined #openstack-swift | 20:07 | |
| zaitcev | acoles: I'm going to split out the change to "absolute" criteria for the dark data watcher (e.g. only dark if no errors from container servers). | 20:12 |
| zaitcev | acoles: it has no relation to the compatibility with sharded containers, so seems like the right thing to do. | 20:12 |
| *** rcernin has quit IRC | 20:13 | |
| acoles | ok. makes sense to separate concerns | 20:13 |
| *** tdasilva_ has quit IRC | 20:32 | |
| *** thiago__ has joined #openstack-swift | 20:33 | |
| *** ChanServ sets mode: +v thiago__ | 20:33 | |
| *** fozboz has joined #openstack-swift | 20:42 | |
| *** rcernin has joined #openstack-swift | 20:50 | |
| *** rcernin has quit IRC | 21:49 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: symlink: Ensure headers are returned as a list on py3 https://review.opendev.org/c/openstack/swift/+/792485 | 22:33 |
| *** rcernin has joined #openstack-swift | 22:42 | |
| openstackgerrit | Merged openstack/swift master: tempurl: Fix PUT upload to temp url on py3 https://review.opendev.org/c/openstack/swift/+/792397 | 23:32 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!