[14:07:35] _joe_: if you have a minute I have a question about the guts of dbctl [14:08:13] https://gerrit.wikimedia.org/r/plugins/gitiles/operations/software/conftool/+/refs/heads/master/conftool/tests/unit/test_dbconfig.py#157 [14:08:16] why? [14:08:51] <_joe_> cdanis: right now I don't [14:09:46] I was thinking "I will add a set-candidate-master subcommand of dbctl instance, it will take ten minutes" [14:09:52] it didn't ;) [14:10:40] <_joe_> and I don't remember, that's a bit out of context right now :P [14:12:14] <_joe_> I think the reason is I wanted to have a unified write_callback, _update takes arbitrary arguments in the signature on the abstract class, and I kept that signature in the derived classes? [14:14:25] that seems likely [14:14:38] it's alright, you've given me an excuse to use functools.partial [14:18:36] obligatory reference (and audio warning) https://www.youtube.com/watch?v=1yH_j8-VVLo [14:19:46] I had no idea this existed, and some friends of mine have much love for the powerthirst video [14:19:49] ty godog [14:21:45] hahah you're welcome cdanis