| *** timburke has joined #openstack-swift | 00:08 | |
| *** ChanServ sets mode: +v timburke | 00:08 | |
| *** baojg has quit IRC | 00:18 | |
| *** baojg has joined #openstack-swift | 00:19 | |
| *** benj_- has joined #openstack-swift | 00:53 | |
| *** benj_ has quit IRC | 00:54 | |
| *** benj_- is now known as benj_ | 00:54 | |
| openstackgerrit | Pete Zaitcev proposed openstack/swift master: Watchers: add a documentation page https://review.opendev.org/c/openstack/swift/+/766378 | 01:51 |
|---|---|---|
| openstackgerrit | Pete Zaitcev proposed openstack/swift master: Watchers: add a documentation page https://review.opendev.org/c/openstack/swift/+/766378 | 02:07 |
| *** timburke has quit IRC | 03:03 | |
| *** openstackgerrit has quit IRC | 03:22 | |
| *** psachin has joined #openstack-swift | 03:45 | |
| *** gyee has quit IRC | 04:00 | |
| *** camelCaser has quit IRC | 04:59 | |
| *** camelCaser has joined #openstack-swift | 05:02 | |
| *** dasp has quit IRC | 05:18 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-swift | 05:33 | |
| *** dasp has joined #openstack-swift | 05:34 | |
| *** klamath_atx has quit IRC | 06:11 | |
| *** rpittau|afk has quit IRC | 06:11 | |
| *** rpittau|afk has joined #openstack-swift | 06:12 | |
| *** timburke has joined #openstack-swift | 06:34 | |
| *** ChanServ sets mode: +v timburke | 06:34 | |
| *** timburke has quit IRC | 06:43 | |
| *** evrardjp has quit IRC | 06:49 | |
| *** evrardjp_ has joined #openstack-swift | 06:49 | |
| *** renich has joined #openstack-swift | 07:09 | |
| *** rcernin has quit IRC | 07:10 | |
| *** rcernin has joined #openstack-swift | 07:29 | |
| *** timburke has joined #openstack-swift | 08:06 | |
| *** ChanServ sets mode: +v timburke | 08:06 | |
| *** rcernin has quit IRC | 08:07 | |
| *** rpittau|afk is now known as rpittau | 08:17 | |
| *** rcernin has joined #openstack-swift | 08:56 | |
| *** dsariel has joined #openstack-swift | 09:01 | |
| *** rcernin has quit IRC | 09:13 | |
| *** mvalsecc has quit IRC | 09:15 | |
| *** openstackgerrit has joined #openstack-swift | 09:52 | |
| openstackgerrit | Merged openstack/swift master: ssync: Tolerate more hang-ups https://review.opendev.org/c/openstack/swift/+/744270 | 09:52 |
| *** rcernin has joined #openstack-swift | 10:57 | |
| *** rcernin has quit IRC | 12:13 | |
| *** lxkong has quit IRC | 12:26 | |
| *** baojg has quit IRC | 12:46 | |
| *** baojg has joined #openstack-swift | 12:47 | |
| *** zigo has joined #openstack-swift | 13:53 | |
| openstackgerrit | Alistair Coles proposed openstack/swift master: swift-manage-shard-ranges: add repair and analyze commands https://review.opendev.org/c/openstack/swift/+/765624 | 14:24 |
| *** psachin has quit IRC | 14:28 | |
| openstackgerrit | Illes Elod proposed openstack/swift stable/stein: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/swift/+/766489 | 14:59 |
| openstackgerrit | Illes Elod proposed openstack/swift stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/swift/+/766491 | 15:01 |
| openstackgerrit | Illes Elod proposed openstack/swift stable/queens: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/swift/+/766494 | 15:05 |
| openstackgerrit | Illes Elod proposed openstack/swift stable/pike: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/swift/+/766495 | 15:09 |
| openstackgerrit | Illes Elod proposed openstack/swift stable/pike: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/swift/+/766495 | 15:12 |
| *** renich has quit IRC | 15:39 | |
| *** tdasilva_ has joined #openstack-swift | 15:54 | |
| *** ChanServ sets mode: +v tdasilva_ | 15:54 | |
| *** tdasilva has quit IRC | 15:55 | |
| *** thiago__ has joined #openstack-swift | 15:56 | |
| *** ChanServ sets mode: +v thiago__ | 15:56 | |
| *** tdasilva_ has quit IRC | 15:59 | |
| *** tdasilva_ has joined #openstack-swift | 16:00 | |
| *** ChanServ sets mode: +v tdasilva_ | 16:00 | |
| *** thiago__ has quit IRC | 16:03 | |
| zaitcev | timburke: are you around? | 17:44 |
| zaitcev | timburke: Does any of your watchers use df in see_object()? | 17:44 |
| zaitcev | timburke: actually, never mind. I see that your EC watcher does with open(df._data_file, 'rb') as fp: header = fp.read(80) | 17:46 |
| timburke | i am! and yeah, i was in the midst of writing a commet about that :-) | 17:56 |
| timburke | i'm not clear about what the pain point is in having the df get passed, though -- can you help me understand? | 17:57 |
| zaitcev | I don't want plugins to have too much visibility inside the diskfile. It's one thing to do this in tree, but watchers are supposed to be useful to operators who write them out in the field. Letting them access df will lead them to depend on its layout. | 17:59 |
| zaitcev | For example, the metadata in particular - you may have a member df.metadata, but you never know what it actually is. Maybe it's the metadata you want, maybe not. | 17:59 |
| zaitcev | So I wanted to get rid of it. Sam didn't have df. | 18:00 |
| zaitcev | But, it's too useful | 18:00 |
| zaitcev | I think I'll add some guidelines to the follow-up doc. I already posted it, but Gerrit is not working... It's there in dependency list. | 18:01 |
| zaitcev | I think we can sacrifice the policy index at least. But I am not sure. | 18:01 |
| *** rpittau is now known as rpittau|afk | 18:04 | |
| *** zaitcev is now known as zaitcev|afk | 18:11 | |
| timburke | fair enough. fwiw, the main thing i wanted out of it was the actual path to the .data file -- quaratine's optional for my ec stats (i could just return early instead) | 18:13 |
| timburke | hmm... maybe we could replace df with a data_file arg that's just the path? then if a watcher wants to quarantine, have it raise DiskFileQuarantined (which the auditor can then catch and handle properly) | 18:17 |
| openstackgerrit | Merged openstack/swift master: Increase gate timeouts from 1hr to 1.5hr https://review.opendev.org/c/openstack/swift/+/766105 | 20:11 |
| *** rcernin has joined #openstack-swift | 20:21 | |
| *** zaitcev|afk is now known as zaitcev | 20:22 | |
| *** tdasilva_ has quit IRC | 20:37 | |
| *** tdasilva_ has joined #openstack-swift | 20:39 | |
| *** ChanServ sets mode: +v tdasilva_ | 20:39 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: Dark data probe test follow-up https://review.opendev.org/c/openstack/swift/+/766549 | 20:59 |
| *** lxkong has joined #openstack-swift | 21:02 | |
| *** rcernin has quit IRC | 21:31 | |
| *** rcernin has joined #openstack-swift | 21:34 | |
| openstackgerrit | Tim Burke proposed openstack/swift master: watchers: Only pass data_file_path via see_object https://review.opendev.org/c/openstack/swift/+/766612 | 21:37 |
| openstackgerrit | Merged openstack/swift master: Stop explicitly handling StopIteration in proxy-logging https://review.opendev.org/c/openstack/swift/+/763251 | 21:48 |
| *** rcernin has quit IRC | 21:51 | |
| *** rcernin has joined #openstack-swift | 21:53 | |
| *** thiago__ has joined #openstack-swift | 22:30 | |
| *** ChanServ sets mode: +v thiago__ | 22:30 | |
| *** tdasilva_ has quit IRC | 22:32 | |
| *** thiago__ has quit IRC | 22:45 | |
| *** tdasilva_ has joined #openstack-swift | 22:45 | |
| *** ChanServ sets mode: +v tdasilva_ | 22:45 | |
| *** baojg has quit IRC | 23:22 | |
| *** baojg has joined #openstack-swift | 23:25 | |
| *** baojg has quit IRC | 23:52 | |
| *** adeberg has quit IRC | 23:53 | |
| *** adeberg has joined #openstack-swift | 23:53 | |
| *** baojg has joined #openstack-swift | 23:53 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!