[09:06] <bthomas> morning
[09:17] <Chipaca> moin moin
[09:17] <bthomas> ahoy
[09:18] <Chipaca> speak like a pirate day was two months ago!
[09:18] <Chipaca> 🙂
[09:18] <bthomas> It is every day for me :-)
[10:00] <Chipaca> bthomas: I just @'ed you on a pull request for a database charm
[10:00] <Chipaca> bthomas: in case you were looking for examples of those things
[10:00] <bthomas> Chipaca: having a look
[10:06] <bthomas> mssql_db_client.py was useful. Thanks Chipaca. Code was quite clean.
[10:29] <bthomas> Chipaca: Isn't there still the question of how will the functionality of something like mssql_db_client.py or interface_mssql.py be exported outside the charm (meaning to a python script that does not execute in the same python environment of the charm). Will it be an Action with parameters ? Also most cloud deployments use authentication tokens for controlling database access, at least in the NoSQL world. I still don't know ho
[10:29] <bthomas> manage tokens in Charms. I am aware of RBAC in the k8s ecosystem.
[10:29] <Chipaca> bthomas: there is the question, but interestingly the answer is almost all done
[10:30] <bthomas> Charm libraries I guess.
[10:30] <Chipaca> bthomas: charm libraries indeed
[10:30] <Chipaca> bthomas: ERC and freenode seem to disagree on line lengths:
[10:30] <Chipaca> authentication tokens for controlling database access, at least in the NoSQL world. I still don't know h
 manage tokens in Charms. I am aware of RBAC in the k8s ecosystem.
[10:31] <bthomas> sorry about formatting. Will keep messages short henceforth.
[10:31] <Chipaca> bthomas: you should be able to bludgeon ERC into submission
[10:31] <bthomas> Any idea how can I learn about charm libraries ?
[10:31] <Chipaca> bthomas: ask facundo next week 🙂
[10:32] <bthomas> Chipaca: too much to do too little time. Any free time I get I would rather get virtual environments working with Emacs Elpy. I reserve time outside work for Math and Family.
[10:32] <bthomas> I will ask facundo. Thank you.
[10:32] <Chipaca> 🙂 ok
[10:55] <bthomas> Chipaca: I think my time today may be more productively spent if I just work on creating a draft pull request to fix all PEP257 convention warnings in ops. I wont mind if facundo does not want us to move to pep257 convention and this work is in vain. But if he does not object I may just manage to get something ready for him to review when he is back. Thoughts ?
[10:57]  * bthomas goes for a quick coffee while Chipaca demures :-)
[10:57] <Chipaca> bthomas: i don't mind
[10:58] <Chipaca> bthomas: and i don't have a better suggestion for you
[11:05] <bthomas> ack. switching context now
[11:35] <bthomas> Fixed all D413 errors D401 D400 D205 D204 D200 D107 D105 to go. Does not look bad at all. All smalll silly things. Quite doable.
[11:46] <DominikF> Hey  there!I'm struggling a little with writing a unit test with relations with the test harness. I wrote up my issues in this discourse post: https://discourse.charmhub.io/t/test-harness-with-custom-events/3887 could someone take a look?
[11:55] <Chipaca> DominikF: will do! added to my queue for before EOD
[11:56] <DominikF> Chipaca: thx!
[12:43] <mup> PR operator#461 opened: Pep257 <Created by balbirthomas> <https://github.com/canonical/operator/pull/461>
[13:47]  * bthomas -> lunch
[14:21] <bthomas> wow. Windows unit tests do take long indeed.
[14:57] <Chipaca> bthomas: ikr
[14:59] <bthomas> yes, indeed :)
[15:00] <bthomas> Anyway I am planning to spend the rest of the day reading ops code and in the process see if I can fill in some of the missing docstrings for __init__ and magic methods.
[15:09] <Chipaca> bthomas: __init__ doesn't need a docstring if the class has a docstring
[15:09] <Chipaca> bthomas: magic methods don't need docstrings unless they're doing something weird
[15:09] <bthomas> Chipaca: ack
[15:09] <Chipaca> like, the __div__ on Path objects can use a docstring, but a __div__ that just does division does not
[15:10] <bthomas> OK so that is why those to PEP257 were in the ignore list
[15:10] <bthomas> s/to/two/
[15:10] <bthomas> I have made no changes there. They are still in the ignore list.
[15:40]  * bthomas needs more coffee
[21:09]  * Chipaca EODs