| facubatista | power is back! | 13:48 |
|---|---|---|
| facubatista | ¡Muy buenos días a todos! | 13:48 |
| * facubatista got the power! https://www.youtube.com/watch?v=j1BNcSBApOU | 13:49 | |
| facubatista | jam, hello! did you see alejdg question from yesterday? | 13:55 |
| jam | alejdg, I'd have to see how you are defining your pod spec, my guess is that you should be setting a single container pod spec, and that Juju would handle the 'count' portion. I'm not sure how you're trying to distinguish specifics for each unit | 14:12 |
| jam | thanks facubatista I had missed it. | 14:12 |
| facubatista | jam, my plan is to review all issues, delay current ones marked with milestone 0.3.0 to 0.4.0, except anything that we want to include this week | 14:14 |
| facubatista | jam, I'll check out your PR first, see if we can land it, and then maybe change build to use it, so we start including everything in the project? | 14:14 |
| jam | sgtm | 14:15 |
| facubatista | jam, travis is not happy with your branch... | 14:18 |
| jam | facubatista, thanks for the heads up, looking | 14:22 |
| jam | looks like a 3.5ism | 14:23 |
| jam | pathlib.Path *sigh* | 14:23 |
| facubatista | yes, 3.5 | 14:26 |
| facubatista | jam, after you deal with that, let's have a HO to sync? just ping me, thanks! | 14:27 |
| crodriguez__ | I've heard about harness being the new way to do unit tests with the ops framework. Is there any documentation out there about it ? | 14:36 |
| facubatista | crodriguez__, don't think so, but maybe we can provide you with examples? | 14:39 |
| crodriguez__ | facubatista, yeah, anything that could help | 14:49 |
| ballot | Hello ! | 14:50 |
| facubatista | hello ballot | 14:52 |
| crodriguez__ | any link you could share facubatista ? | 15:04 |
| facubatista | crodriguez__, 5' | 15:08 |
| facubatista | crodriguez__, https://paste.ubuntu.com/p/67Hcd5yPT3/ | 15:21 |
| facubatista | jam, let me know if we can talk at some point today so I organize myself | 15:22 |
| crodriguez__ | thank you facubatista | 15:37 |
| alejdg | jam: How would I set specific secrets for each unit? Is there a way to do that? | 16:57 |
| crodriguez_ | I don't understand harness... I thought it was supposed to mock the charm? But it looks like it is executing functions of the charm directly on my system instead. For example, I am testing harness.update_config , which triggers render_config in my charm. render_config, amongts other things, tries to create a directory in /etc/, which is denied because of lack of permissions | 18:59 |
| facubatista | crodriguez_, we'll need help from jam for that :/ | 20:14 |
| crodriguez_ | facubatista, is he on PTO? | 20:18 |
| facubatista | crodriguez_, nop, but we'll need to wait for tomorrow | 20:22 |
| crodriguez_ | Ok facubatista, thanks for the info. I might schedule a meeting/work session for that | 20:30 |
| facubatista | crodriguez_, nice | 20:32 |
| vultaire2 | @crodriguez_: Re: your questions about operator framework - if you don't mind, I'm going to paraphrase a few of the questions here | 20:46 |
| vultaire2 | I missed backscroll in this channel as I just joined | 20:47 |
| vultaire2 | Your main question was: operator framework is supposed to mock the charm, thus why is it running stuff directly on your box? ...Is that a fair way of rephrasing, or did I miss something? | 20:48 |
| drewn3ss | "For example, I am testing harness.update_config , which triggers render_config in my charm. render_config, amongts other things, tries to create a directory in /etc/, which is denied because of lack of permissions" | 20:48 |
| crodriguez_ | pretty much yeah vultaire2 | 20:48 |
| drewn3ss | yeah, you've got it, vultaire2 | 20:48 |
| * drewn3ss is wondering iif you still need to mock things like the render as we do in reactive unit tests | 20:49 | |
| * crodriguez_ wondering the same thing :) | 20:49 | |
| vultaire2 | Sorry for the delay here, jumping around a bit... | 20:51 |
| vultaire2 | So, basically: it is mocking a lot of the stuff. You could kind of say it lets you test functions of your charm code without juju running underneath it | 20:51 |
| vultaire2 | that being said, it's not any sort of virtualization or anything like that, it's not spinning up containers or anything - all code _is_ running locally on your machine | 20:52 |
| vultaire2 | with normal unit testing, you're typically running single functions directly on your development system, perhaps mocking out areas where the code does things you don't want it to (like talking to a DB, for example) | 20:53 |
| vultaire2 | basically, with unit testing you're trying to test one function, or a small area of code, in isolation from the rest of the code | 20:53 |
| vultaire2 | Now, this is normally pretty difficult with charms, especially pre-operator charms, as there's a lot of stuff running "under" them | 20:54 |
| vultaire2 | all the juju binaries and such which get called behind the scenes | 20:54 |
| vultaire2 | if you're just running unit tests and not doing a full end-to-end integration test, you typically don't want that, and thus you end up having to mock out stuff. Lots of stuff. | 20:54 |
| vultaire2 | With operator framework, a large amount of that is effectively mocked out for you | 20:55 |
| vultaire2 | Just like with non-juju unit tests, your tests run on your machine. So if your charm does things that you don't want modifying your system environment (like installing packages), you'll still need to mock those bits out | 20:56 |
| vultaire2 | ...Let me stop here. Does this make sense somewhat? | 20:57 |
| drewn3ss | so, harness isn't providing a mocked virtual environment in total, but rather mocking out the juju layer in general? | 20:57 |
| crodriguez_ | ok, thanks for the explanation. Yes that makes sense, I'll need to mock out some functions so it doesn't mess up my system haha.. | 20:57 |
| vultaire2 | yes | 20:57 |
| crodriguez_ | your video is helpful too with the examples! | 20:57 |
| crodriguez_ | I'll give it another shot and might ask more precise questions when I hit blockers | 20:58 |
| drewn3ss | crodriguez_: you might look to the unit tests I was just poking at for mock_render https://git.launchpad.net/charm-prometheus-ceph-exporter/tree/tests/unit/test_reactive_ceph_exporter.py#n95 | 20:58 |
| vultaire2 | Sounds good, please do and best of luck | 20:58 |
| drewn3ss | it's not operator, but maybe some of that code will help you mock the render | 20:59 |
| crodriguez_ | thanks drewn3ss! | 21:00 |
| facubatista | vultaire, drewn3ss, thanks! | 21:06 |
| * facubatista eods | 21:19 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!