| *** gyee has quit IRC | 00:07 | |
| *** mvenesio has quit IRC | 00:21 | |
| *** d0ugal_ has quit IRC | 00:33 | |
| *** d0ugal_ has joined #openstack-swift | 00:40 | |
| *** spsurya has quit IRC | 00:41 | |
| torgomatic | I wonder why X-Timestamp doesn't change when you POST to an object anyway | 00:53 |
|---|---|---|
| *** germs has quit IRC | 01:17 | |
| *** armaan has quit IRC | 01:20 | |
| *** armaan has joined #openstack-swift | 01:21 | |
| openstackgerrit | Nguyen Hai proposed openstack/python-swiftclient master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/553433 | 01:54 |
| *** d0ugal_ has quit IRC | 01:59 | |
| *** amandap_ has joined #openstack-swift | 02:01 | |
| *** amandap has quit IRC | 02:04 | |
| *** bkopilov has quit IRC | 02:08 | |
| *** d0ugal_ has joined #openstack-swift | 02:08 | |
| openstackgerrit | Merged openstack/swift master: Log deprecation warnings for account_white/blacklist https://review.openstack.org/468077 | 02:15 |
| *** SkyRocknRoll has joined #openstack-swift | 02:23 | |
| openstackgerrit | Debo Zhang proposed openstack/swift master: Fix typo https://review.openstack.org/570100 | 02:34 |
| *** amandap_ has quit IRC | 03:01 | |
| *** amandap has joined #openstack-swift | 03:07 | |
| *** germs has joined #openstack-swift | 03:17 | |
| *** germs has quit IRC | 03:22 | |
| *** bkopilov has joined #openstack-swift | 03:25 | |
| *** armaan has quit IRC | 03:50 | |
| *** armaan has joined #openstack-swift | 03:50 | |
| *** cshastri has joined #openstack-swift | 04:28 | |
| *** myrat has joined #openstack-swift | 04:38 | |
| *** links has joined #openstack-swift | 04:39 | |
| *** tdasilva has quit IRC | 04:58 | |
| *** germs has joined #openstack-swift | 05:19 | |
| *** germs has quit IRC | 05:19 | |
| *** germs has joined #openstack-swift | 05:19 | |
| *** germs has quit IRC | 05:23 | |
| *** psachin has joined #openstack-swift | 05:25 | |
| *** tdasilva has joined #openstack-swift | 05:27 | |
| *** myrat has quit IRC | 05:56 | |
| *** links has quit IRC | 06:04 | |
| *** cshastri has quit IRC | 06:09 | |
| *** myrat has joined #openstack-swift | 06:09 | |
| *** gerardgine has joined #openstack-swift | 06:18 | |
| *** gerardgine has quit IRC | 06:19 | |
| *** hseipp has joined #openstack-swift | 06:19 | |
| *** links has joined #openstack-swift | 06:20 | |
| *** armaan has quit IRC | 06:25 | |
| *** armaan has joined #openstack-swift | 06:25 | |
| *** pcaruana has joined #openstack-swift | 06:33 | |
| *** cshastri has joined #openstack-swift | 06:36 | |
| *** mikecmpbll has joined #openstack-swift | 06:40 | |
| *** cshastri has quit IRC | 06:43 | |
| *** mikecmpbll has quit IRC | 06:52 | |
| *** armaan has quit IRC | 06:52 | |
| *** armaan has joined #openstack-swift | 06:53 | |
| *** myrat has quit IRC | 06:54 | |
| *** threestrands has quit IRC | 06:57 | |
| *** rcernin has quit IRC | 06:57 | |
| *** cshastri has joined #openstack-swift | 07:01 | |
| *** myrat has joined #openstack-swift | 07:07 | |
| *** amandap has quit IRC | 07:17 | |
| *** germs has joined #openstack-swift | 07:20 | |
| *** germs has quit IRC | 07:20 | |
| *** germs has joined #openstack-swift | 07:20 | |
| *** tesseract has joined #openstack-swift | 07:20 | |
| *** germs has quit IRC | 07:25 | |
| *** geaaru has joined #openstack-swift | 07:33 | |
| *** cbartz has joined #openstack-swift | 07:44 | |
| *** mikecmpbll has joined #openstack-swift | 07:49 | |
| *** links has quit IRC | 08:01 | |
| *** d0ugal_ has quit IRC | 08:08 | |
| *** d0ugal has joined #openstack-swift | 08:08 | |
| acoles | good morning | 08:08 |
| acoles | mahatic: hi! hope you enjoy Vancouver | 08:10 |
| *** links has joined #openstack-swift | 08:14 | |
| openstackgerrit | Merged openstack/swift master: Fix typo https://review.openstack.org/570100 | 08:14 |
| *** gkadam has joined #openstack-swift | 08:17 | |
| *** amandap has joined #openstack-swift | 08:46 | |
| *** amandap has joined #openstack-swift | 08:46 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Add unit tests for replicator sync_shard_ranges https://review.openstack.org/570005 | 08:57 |
| *** armaan has quit IRC | 09:04 | |
| *** armaan has joined #openstack-swift | 09:04 | |
| *** germs has joined #openstack-swift | 09:20 | |
| *** germs has joined #openstack-swift | 09:20 | |
| *** psachin has quit IRC | 09:23 | |
| *** germs has quit IRC | 09:25 | |
| *** SkyRocknRoll has quit IRC | 09:31 | |
| *** mikecmpbll has quit IRC | 09:35 | |
| *** mikecmpbll has joined #openstack-swift | 09:40 | |
| *** bkopilov has quit IRC | 10:16 | |
| *** amandap has quit IRC | 10:25 | |
| *** amandap has joined #openstack-swift | 10:25 | |
| *** amandap has quit IRC | 10:30 | |
| *** psachin has joined #openstack-swift | 10:31 | |
| openstackgerrit | Merged openstack/swift master: Move handling of missing table outside of do_query https://review.openstack.org/570036 | 11:06 |
| *** amandap has joined #openstack-swift | 11:14 | |
| *** hseipp has quit IRC | 11:21 | |
| *** germs has joined #openstack-swift | 11:21 | |
| *** germs has quit IRC | 11:21 | |
| *** germs has joined #openstack-swift | 11:21 | |
| *** germs has quit IRC | 11:26 | |
| *** amandap has quit IRC | 11:27 | |
| *** amandap has joined #openstack-swift | 11:28 | |
| *** pcaruana|afk| has joined #openstack-swift | 11:29 | |
| *** pcaruana has quit IRC | 11:29 | |
| *** vinsh_ has joined #openstack-swift | 11:32 | |
| *** vinsh has quit IRC | 11:32 | |
| *** vinsh_ has quit IRC | 11:32 | |
| *** vinsh has joined #openstack-swift | 11:33 | |
| *** armaan has quit IRC | 11:42 | |
| *** armaan has joined #openstack-swift | 11:43 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Verify diff stat is unchanged when syncing only shard ranges https://review.openstack.org/570171 | 11:47 |
| acoles | clayg: timburke I held off +A on this https://review.openstack.org/#/c/570055/1 because clayg said he wanted to think more about it, and because ideally I'd like to see the follow on patch land with it...but someone should merge it before we cut release | 11:57 |
| patchbot | patch 570055 - swift - Stop overriding _in_sync in container replicator | 11:57 |
| openstackgerrit | Merged openstack/swift master: Add a helper context for optional connection args https://review.openstack.org/570058 | 12:01 |
| *** kei_yama has quit IRC | 12:22 | |
| openstackgerrit | Merged openstack/swift master: Use maybe_get more https://review.openstack.org/570068 | 12:28 |
| *** myrat has quit IRC | 12:29 | |
| *** cshastri has quit IRC | 12:44 | |
| *** germs has joined #openstack-swift | 13:02 | |
| *** germs has quit IRC | 13:07 | |
| *** armaan has quit IRC | 13:11 | |
| openstackgerrit | Merged openstack/python-swiftclient master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/553433 | 13:12 |
| *** cbartz has quit IRC | 13:12 | |
| openstackgerrit | melissaml proposed openstack/slogging master: fix a typo in sample.proxy-server.conf https://review.openstack.org/570189 | 13:17 |
| *** cbartz has joined #openstack-swift | 13:23 | |
| *** psachin has quit IRC | 13:32 | |
| *** bkopilov has joined #openstack-swift | 13:49 | |
| *** cbartz has quit IRC | 14:05 | |
| openstackgerrit | Merged openstack/swift master: Make the decision between primary/handoff sets more obvious https://review.openstack.org/567940 | 14:16 |
| *** mikecmpbll has quit IRC | 14:20 | |
| *** blair has quit IRC | 14:38 | |
| *** germs has joined #openstack-swift | 14:40 | |
| *** germs has quit IRC | 14:45 | |
| *** btorch has joined #openstack-swift | 14:54 | |
| *** mvenesio has joined #openstack-swift | 14:57 | |
| *** cbartz has joined #openstack-swift | 15:07 | |
| *** cbartz has quit IRC | 15:15 | |
| *** pcaruana|afk| has quit IRC | 15:33 | |
| mattoliverau | Morning | 15:36 |
| acoles | mattoliverau: o/ | 15:36 |
| mahatic | rledisez: oh okay :D | 15:45 |
| mahatic | acoles: hi! thank, I am. But you're all being missed! | 15:45 |
| mahatic | thanks* | 15:46 |
| *** gyee has joined #openstack-swift | 15:51 | |
| notmyname | good morning | 16:02 |
| timburke | good morning | 16:13 |
| notmyname | wow. quite a few patches landed on master in the last couple of days | 16:14 |
| openstackgerrit | Tim Burke proposed openstack/python-swiftclient master: Back out some version bumps https://review.openstack.org/568914 | 16:15 |
| *** links has quit IRC | 16:16 | |
| timburke | acoles: so how do you feel about trying to cache shard range info? | 16:21 |
| timburke | i feel like it'd be nice to get that container hit out of the object PUT/DELETE paths... | 16:21 |
| timburke | but i'm not sure how we can find the appropriate range without caching *all* the shard ranges... which may be impractical if your cluster has more than a few large DBs | 16:22 |
| acoles | timburke: caching the last used shard range may be a win for some workloads. But I heard clayg make a good case for the hit not being so bad - at least worth quantifying before fixing. | 16:24 |
| timburke | all right. maybe we *can* get rid of find_shard_range... | 16:25 |
| acoles | timburke: are you working on a prefix GET improvement? you've mentioned it a few times, I don't want to double up. | 16:25 |
| timburke | yeah, i can take that on. i wanted to knock out some of the smaller items i'd noticed, and wait for some patches to merge that would've caused conflicts | 16:27 |
| acoles | ok thanks | 16:27 |
| acoles | I wouldn't have got to it until next week anyway | 16:28 |
| *** pcaruana|afk| has joined #openstack-swift | 16:29 | |
| timburke | how do you feel about https://review.openstack.org/#/c/569884/ ? you were right, i had to change some unit tests... but i don't think we really *need* the ability to do direct_client requests without an x-timestamp | 16:34 |
| patchbot | patch 569884 - swift - Clean up direct_client's gen_headers | 16:34 |
| timburke | certainly, the probe tests are all still happy | 16:34 |
| *** germs has joined #openstack-swift | 16:37 | |
| *** germs has quit IRC | 16:37 | |
| *** germs has joined #openstack-swift | 16:38 | |
| acoles | timburke: IIRC I tried similar to get the user agent plumbed in, saw the unit tests fail and realised that however reasonable the change seems to us I'd never know if anyone else relies on that current behaviour | 16:46 |
| *** pcaruana|afk| has quit IRC | 16:51 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: Use sharding container epoch in shard container names https://review.openstack.org/570235 | 17:02 |
| *** cshastri has joined #openstack-swift | 17:09 | |
| timburke | of course, in *that* vein, *any* change could break consumers, and we never would have done https://github.com/openstack/swift/commit/4c11833 or https://github.com/openstack/swift/commit/4a6e0cc | 17:30 |
| timburke | i think there was some other change we did recently-ish that busted a bunch of middlewares i ship... i forget what now, though. something about moving a function from one module to another... | 17:31 |
| kota_ | good morning, I'm preparing my slides and get practice in my hotel room. | 17:32 |
| kota_ | i'll go the conference center around noon. | 17:32 |
| acoles | notmyname: no meeting today right? or...if there is I can't make it, apologies. | 17:34 |
| *** cshastri has quit IRC | 17:37 | |
| *** tesseract has quit IRC | 17:38 | |
| *** mvenesio_ has joined #openstack-swift | 17:43 | |
| *** mvenesio has quit IRC | 17:43 | |
| *** pcaruana has joined #openstack-swift | 17:55 | |
| notmyname | no swift team meeting today | 17:56 |
| notmyname | acoles: ^ | 17:56 |
| kota_ | notmyname: ok | 17:58 |
| *** armaan has joined #openstack-swift | 18:05 | |
| *** geaaru has quit IRC | 18:08 | |
| timburke | acoles: other times that we've made work for downstream consumers: https://github.com/openstack/swift/commit/2744492f3 (bye bye, get_container_memcache_key and get_account_memcache_key!) | 18:11 |
| timburke | https://github.com/openstack/swift/commit/4806434 (let's move get_listing_content_type and FORMAT2CONTENT_TYPE! surely no one's using those, right?) | 18:11 |
| timburke | i get how this sounds like an argument *against* https://review.openstack.org/#/c/569884/ but i guess my point is that when you're deeply entwined with the guts of a project, you have to expect some churn and validate that everything still works when you upgrade | 18:14 |
| patchbot | patch 569884 - swift - Clean up direct_client's gen_headers | 18:14 |
| timur | torgomatic: about the X-Timestamp -- I'm not sure. I see that it only includes the PUT time and wonder if that's related to how the objects are replicated, but am not certain. I do wonder if there would be negative consequences to making X-Timestamp return the last modified time? | 18:24 |
| timburke | pre-fast-post, they should have always been the same, yeah? | 18:25 |
| timur | correct | 18:25 |
| timur | but after fast-post, I wonder why they weren't updated? | 18:25 |
| timburke | not sure. probably something like https://bugs.launchpad.net/swift/+bug/1612991 where we just didn't notice that there was a difference | 18:28 |
| openstack | Launchpad bug 1612991 in OpenStack Object Storage (swift) "Post-as-copy does not allow a DLO manifest to be updated" [Critical,Fix released] - Assigned to Janie Richling (jrichli) | 18:28 |
| clayg | timur: oh I didn't notice you were discussing the timestamp issue in here!? KUDOS! | 18:28 |
| timur | well... I tried/am trying | 18:28 |
| clayg | I think this will be a great bug report - tdasilva is always trying to help us get better with bug triage - it'll be great! thank you for the help! | 18:29 |
| openstackgerrit | Merged openstack/swift master: Add unit tests for replicator sync_shard_ranges https://review.openstack.org/570005 | 18:42 |
| timur | clayg: https://bugs.launchpad.net/swift/+bug/1772992 | 18:56 |
| openstack | Launchpad bug 1772992 in OpenStack Object Storage (swift) "Report object's sub-second last modified timestamp" [Undecided,New] | 18:56 |
| *** gkadam has quit IRC | 19:13 | |
| *** zaitcev_ has joined #openstack-swift | 19:31 | |
| *** ChanServ sets mode: +v zaitcev_ | 19:31 | |
| *** zaitcev has quit IRC | 19:35 | |
| *** armaan has quit IRC | 19:45 | |
| *** mvenesio_ has quit IRC | 20:28 | |
| *** pcaruana has quit IRC | 20:28 | |
| *** mvenesio has joined #openstack-swift | 20:29 | |
| *** mvenesio has quit IRC | 20:33 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: Plumb includes down into _get_shard_range_rows https://review.openstack.org/569847 | 21:09 |
| notmyname | tdasilva: I just had a good chat with zaneb about py3 | 21:25 |
| notmyname | I referenced https://etherpad.openstack.org/p/Swift_py3 and he may be able to pick off some of the smaller things | 21:25 |
| notmyname | mattoliverau: tdasilva: as you're looking at py3 stuff, please try to keep that etherpad up to date | 21:27 |
| mattoliverau | notmyname: I'll try and talk a look when I have some time. | 21:28 |
| clayg | timburke: srly, the unittest in patch 569847 was SO helpful; sorry to slow you down man! | 21:37 |
| patchbot | https://review.openstack.org/#/c/569847/ - swift - Plumb includes down into _get_shard_range_rows | 21:37 |
| timburke | i *hate* fetching extra rows! especially when i ultimately only care about *one of them* | 21:39 |
| clayg | those underscores are trixy like that ;) | 21:44 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Plumb marker/end_marker down to the DB query https://review.openstack.org/570264 | 21:51 |
| timburke | clayg: similar change, but for marker/end_marker ^^^ | 21:51 |
| notmyname | mattoliverau: cschwede: meet for dinner at 6:30. at the big scuplture between the convention center and the cactus club cafe on the west side of the convention center | 21:56 |
| openstackgerrit | Tim Burke proposed openstack/swift master: Reverse shard ranges later https://review.openstack.org/570265 | 21:59 |
| timburke | ok, so who has opinions about `path` queries? | 22:03 |
| timburke | bug or feature? `?path=a` is equivalent to `?path=a/` is equivalent to `?path=a///////` | 22:04 |
| *** ediardo has joined #openstack-swift | 22:04 | |
| timburke | bug or feature? two slashes in an object name effectively hides it from path queries. there is no path value such that the object a//b will be listed | 22:05 |
| *** ediardo has left #openstack-swift | 22:11 | |
| tdasilva | notmyname: ack | 22:15 |
| tdasilva | zaitcev_: ^^^ re py3 | 22:15 |
| *** zaitcev_ is now known as zaitcev | 22:16 | |
| zaitcev | tdasilva, notmyname: roger on etherpad https://etherpad.openstack.org/p/Swift_py3 | 22:17 |
| *** amandap has quit IRC | 22:18 | |
| notmyname | timburke: path queries. an ugly hack that we cannot remove, but shouldn't document nor encourage people to use. use prefix/delimiter instead. (my gut reaction) | 22:18 |
| *** amandap has joined #openstack-swift | 22:18 | |
| torgomatic | if anyone's looking for a py3 change to review, https://review.openstack.org/563354 is small but helpful | 22:20 |
| patchbot | patch 563354 - swift - Improve path handling in proxy_logging | 22:20 |
| timburke | notmyname: well, we could at least document them *properly* -- https://github.com/openstack/swift/blob/2.17.0/swift/container/backend.py#L671-L672 is technically true, but horribly misleading | 22:20 |
| zaitcev | torgomatic, sec | 22:20 |
| *** rcernin has joined #openstack-swift | 22:21 | |
| notmyname | timburke: yeah. IDK. it's hard to see if it's a bug or feature. if it's a bug, it's almost certainly a "won't fix". so I guess that's more of a "feature". or at lease "the way things are" | 22:23 |
| *** amandap has quit IRC | 22:23 | |
| openstackgerrit | Samuel Merritt proposed openstack/swift master: Add support for PROXY protocol v1 (only) https://review.openstack.org/373563 | 22:38 |
| zaitcev | what a freaking yak | 22:45 |
| zaitcev | to review Sam's patch I wanted to test an object with a slash | 22:45 |
| zaitcev | found that my SSL certs expired on the proxy | 22:45 |
| openstackgerrit | Timur Alperovich proposed openstack/swift master: Fix SLO delete for accounts with non-ASCII names. https://review.openstack.org/570277 | 23:23 |
| openstackgerrit | Merged openstack/swift master: Let make_db_file_path accept epoch=None https://review.openstack.org/570054 | 23:41 |
| *** amandap has joined #openstack-swift | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!