Wednesday, 2020-10-21

*** gyee has quit IRC00:11
*** gyee has joined #openstack-swift00:25
*** ChanServ changes topic to "OpenStack Swift object storage | Logs: http://eavesdrop.openstack.org/irclogs/%23openstack-swift/ | Meetings: https://wiki.openstack.org/wiki/Meetings/Swift | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Ideas: https://wiki.openstack.org/wiki/Swift/ideas"00:39
-openstackstatus- NOTICE: The Gerrit service at review.opendev.org is back up and running; for outage details see analysis here: http://lists.opendev.org/pipermail/service-announce/2020-October/000011.html00:39
*** gyee has quit IRC01:12
*** mikecmpbll has quit IRC01:15
*** dsariel has quit IRC03:31
*** psachin has joined #openstack-swift03:32
*** dsariel has joined #openstack-swift03:39
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-swift04:33
*** m75abrams has joined #openstack-swift05:25
*** dsariel has quit IRC06:17
*** dsariel has joined #openstack-swift06:18
*** renich has quit IRC06:26
*** rpittau|afk is now known as rpittau06:42
*** hoonetorg has joined #openstack-swift06:51
*** openstackgerrit has quit IRC07:38
*** rcernin has quit IRC07:39
*** rcernin has joined #openstack-swift08:09
*** rcernin has quit IRC08:14
*** rcernin has joined #openstack-swift08:28
*** rcernin has quit IRC08:56
*** mikecmpbll has joined #openstack-swift09:46
*** aluria has quit IRC10:28
*** aluria has joined #openstack-swift10:30
*** aluria has quit IRC10:38
*** aluria has joined #openstack-swift10:39
*** mikecmpbll has quit IRC10:58
*** dsariel has quit IRC12:30
*** zamba has quit IRC13:03
*** zamba has joined #openstack-swift13:11
*** tdasilva has quit IRC13:15
*** tdasilva has joined #openstack-swift13:15
*** ChanServ sets mode: +v tdasilva13:15
*** psachin has quit IRC13:19
*** fingo has joined #openstack-swift14:19
*** mikecmpbll has joined #openstack-swift14:55
*** klamath_atx has joined #openstack-swift14:58
*** openstackgerrit has joined #openstack-swift15:24
openstackgerritTim Burke proposed openstack/swift master: Be willing to send hashes as JSON  https://review.opendev.org/75863815:24
openstackgerritTim Burke proposed openstack/swift master: Compress REPLICATE responses  https://review.opendev.org/75863915:24
openstackgerritTim Burke proposed openstack/swift master: py3: Fix status comparison  https://review.opendev.org/75907515:25
*** tdasilva_ has joined #openstack-swift15:27
*** ChanServ sets mode: +v tdasilva_15:27
*** tdasilva has quit IRC15:30
*** m75abrams has quit IRC15:51
*** noonedeadpunk has quit IRC15:51
*** rpittau is now known as rpittau|afk16:04
*** noonedeadpunk_ has joined #openstack-swift16:12
timburkehuh. so in update_deleted, we only mark suffixes invalid if the sync was successful: https://github.com/openstack/swift/blob/2.26.0/swift/obj/replicator.py#L515-L52416:23
timburkebut in update, we do it all the time: https://github.com/openstack/swift/blob/2.26.0/swift/obj/replicator.py#L694-L70116:24
*** mikecmpbll has quit IRC16:24
timburkei wonder *why*. i'm kind of tempted to push the rehash logic into the rsync function, in light of https://bugs.launchpad.net/swift/+bug/181870916:25
openstackLaunchpad bug 1818709 in OpenStack Object Storage (swift) "object replicator update_deleted post ssync REPLICATE request considered harmful" [Undecided,New]16:25
*** mikecmpbll has joined #openstack-swift16:26
*** noonedeadpunk_ has quit IRC16:33
*** noonedeadpunk has joined #openstack-swift16:39
*** mikecmpbll has quit IRC17:25
*** mikecmpbll has joined #openstack-swift17:26
openstackgerritTim Burke proposed openstack/swift master: Add option to REPLICATE to just invalidate hashes  https://review.opendev.org/75863618:08
*** gyee has joined #openstack-swift18:09
openstackgerritTim Burke proposed openstack/swift master: s3api: Tolerate absolute-form request targets  https://review.opendev.org/75911918:11
ormandjon deletes/bulk deletes, i would expect tombstone files are getting created and the purge is actually happening after the reclaim_age by the replicator and so forth. is there any reason for a bulk delete to be 'slow' as such? we're seeing lots of clients timeout at 120s+ on bulk delete operations (granted, 10k items)19:13
timburkeormandj, a bulk delete turns into a whole bunch of normal deletes on the backend -- there's some concurrency allowed, but the defaults are somewhat low, as it's a potential abuse vector20:15
timburkesee https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L983-L986 for the swift api, and https://github.com/openstack/swift/blob/master/etc/proxy-server.conf-sample#L531-L533 for s3api20:16
clayg@timburke I think on update_deleted we're reverting handoffs - when adding capacity we can get rejected by rsync for concurrency and then if we end up doing a REPLICATE call that's super dumb20:24
timburkeclayg, can/should we skip it on failure for update?20:26
claygprimary syncing isn't an area i worry too much about optimizing20:26
claygthe biggest problem with the REPLICATE call is there's no concurrency control - lock N per disk, 1 per part with a timeout/5XX/stats would be ideal20:28
claygin theory the hash calculation is lockless so we can support hash invalidation while hashing - but I think once we moved to the .invalid file we actually have a lot more options for how we shape and throttle the i/o we use there20:28
mattoliverauNot sure if we're meeting today, but i have to take Zoe in for a day care orientation early (for some reason, I guess so parents can work?) so might not really be around. Ptg next week, looking forward to it!20:33
timburkealmost meeting time!20:55
kota_morning20:59
acoleskota_: hello!20:59
seongsoochomorning o/21:00
kota_wow acoles! how's going?21:00
acolesgood thanks21:00
rledisezwelcome back acoles! :)21:00
* kota_ reads the backlog. wow acoles is now at NVIDIA.21:01
acolesrledisez: thanks21:01
kota_welcome back!21:02
acoleskota_: thank you21:04
ormandjtimburke: yeah, saw those variables, still didn't expect it to be quite _this_ slow, but we'll play with tuning those21:07
mattoliverauWhat! \o/ acoles! Best news ever!21:08
acoleshi mattoliverau21:08
zaitcevIt's like that time when I went to work on Ceph.21:08
zaitcevIn a year's time I quit and went back to Swift.21:08
acoleswith swift you can go missing but eventually you'll get replicated :)21:10
tdasilva_join #openstack-meeting21:11
*** tdasilva_ is now known as tdasilva21:11
*** mikecmpbll has quit IRC21:29
mattoliverauLol, even the devs are eventually consistent21:30
*** mikecmpbll has joined #openstack-swift21:31
timburkeswift cores: possibly degraded, but always durable21:31
mattoliverauLol21:31
acolesbut prone to inconsistency21:39
timburkejust please tell me you didn't format your brain prior to coming back ;-)21:41
tdasilvamaybe just quarantined some data21:44
clayg😍22:02
acoleslove it ... "thank you all for coming, and thank you for working on swift!" ... some things haven't changed22:03
zaitcevBut Chuck is now a VP at a grocery store chain.22:03
tdasilvasome things haven't changed....22:07
timburkeit's a good ending to a meeting :-)22:15
*** rcernin has joined #openstack-swift22:51

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!