=== max__ is now known as Guest38174 | ||
niemeyer | Good morning all | 09:42 |
---|---|---|
mup__ | Issue # closed: operator#98, operator#113, operator#122, operator#123, operator#156, operator#158, operator#161, operator#165, operator#166, operator#169, operator#171, operator#172, operator#174, operator#175, operator#178, operator#179, operator#180, operator#185, operator#186, operator#187, | 10:06 |
mup__ | operator#188, operator#189, operator#198, operator#200, operator#205, operator#206, operator#208, operator#211, operator#214, operator#215, operator#219, operator#220 | 10:06 |
mup__ | PR # closed: operator#135, operator#190, operator#196, operator#203, operator#209, operator#210, operator#212, operator#216, operator#218, operator#221 | 10:06 |
mup__ | Issue # opened: operator#98, operator#113, operator#122, operator#123, operator#156, operator#158, operator#161, operator#165, operator#166, operator#169, operator#171, operator#172, operator#174, operator#175, operator#178, operator#179, operator#180, operator#185, operator#186, operator#187, | 10:06 |
mup__ | operator#188, operator#189, operator#198, operator#200, operator#205, operator#206, operator#208, operator#211, operator#214, operator#215, operator#219, operator#220 | 10:06 |
mup__ | PR # opened: operator#135, operator#190, operator#196, operator#203, operator#209, operator#210, operator#212, operator#216, operator#218, operator#221 | 10:06 |
=== mup__ is now known as mup_ | ||
=== max__ is now known as Guest75689 | ||
niemeyer | Dmitrii-Sh: The change you added to Chipaca's #203 doesn't make much sense given the PR purpose | 11:08 |
mup_ | PR #203: ops, test: revert the EventSet change, make EventsBase ObjectEvents <Created by chipaca> <https://github.com/canonical/operator/pull/203> | 11:08 |
Dmitrii-Sh | niemeyer: that's for a different repo & PR | 11:11 |
niemeyer | Dmitrii-Sh: I don't understand the comment given what I just said | 11:11 |
niemeyer | Dmitrii-Sh: I'm talking about *that* change in *that* PR | 11:11 |
Dmitrii-Sh | https://github.com/canonical/operator/pull/203/commits - there's only one commit there, I didn't make any changes | 11:12 |
mup_ | PR #203: ops, test: revert the EventSet change, make EventsBase ObjectEvents <Created by chipaca> <https://github.com/canonical/operator/pull/203> | 11:12 |
niemeyer | Dmitrii-Sh: I see, thanks | 11:13 |
niemeyer | Dmitrii-Sh: Indeed I misread the GitHub comments as being a change in that PR.. confusing | 11:13 |
Dmitrii-Sh | np, references are not very explicit, I agree | 11:14 |
niemeyer | Yeah, it clearly says "<nick> added commit ab3b to <branch>" | 11:15 |
niemeyer | So we need to go see which branch it is | 11:15 |
niemeyer | Dmitrii-Sh: Is #210 about a new interface or an existing interface that existing real world charms are using? | 11:17 |
mup_ | PR #210: tls-certificates interface (requires side, existing interface) <🆕 interface> <Created by dshcherb> <https://github.com/canonical/operator/pull/210> | 11:17 |
niemeyer | Dmitrii-Sh: The summary and the labels disagree | 11:17 |
Dmitrii-Sh | tls-certificates is about the existing one | 11:17 |
niemeyer | Not sure if the idea of the label was to flag things being added to the repo or things being created anew | 11:18 |
niemeyer | We'll need to catch up tomorrow about that once everyone is back | 11:18 |
Dmitrii-Sh | right, haven't noticed that the "interface" label has a description saying "A new interface!" | 11:19 |
facubatista | Muy buenos dÃas a todos! | 11:21 |
niemeyer | facubatista: Heya! | 11:21 |
facubatista | hola niemeyer! | 11:21 |
niemeyer | Lunchero | 11:35 |
facubatista | jam, hola! I just approved #209 but please see a comment in the test part, thanks! | 11:45 |
mup_ | PR #209: Enable stderr logging if JUJU_DEBUG is set <Created by jameinel> <https://github.com/canonical/operator/pull/209> | 11:45 |
facubatista | mup_, thanks :) | 11:45 |
jam | facubatista, hiya, thanks! I thought you were off today | 12:13 |
facubatista | jam, nop, thu & fri last week; Chipaca is off today | 12:14 |
jam | ah, gotcha | 12:14 |
jam | facubatista, actually, the main asserts are outside of the patch, but I went ahead and removed the extra assert that was inside, because it functionally is redundant with the rest. | 12:21 |
facubatista | perfect | 12:21 |
mup_ | PR operator#209 closed: Enable stderr logging if JUJU_DEBUG is set <Created by jameinel> <Merged by jameinel> <https://github.com/canonical/operator/pull/209> | 12:30 |
mup_ | Issue operator#200 closed: Install stderr log handler when running under debug-hooks <Created by stub42> <Closed by jameinel> <https://github.com/canonical/operator/issue/200> | 12:31 |
jam | nice to see mup working | 12:40 |
jam | but why mup_ ? | 12:40 |
jam | I guess only one mup on freenode at a time | 12:40 |
niemeyer | No, if works across channels, networks, and even protocols.. | 12:41 |
niemeyer | It's the same mup here and in Signal, for example | 12:41 |
niemeyer | But I've been hacking on it recently as mentioned.. this is the one running on my laptop.. soon the other one will go away and this will be moved to a proper place | 12:42 |
jam | sorry, I meant one bot named 'mup' and running the new one will use a different name until you have it worked out. | 13:21 |
niemeyer | Ah, yeah, that's it | 13:21 |
niemeyer | Dmitrii-Sh: Are you off today? | 13:33 |
Dmitrii-Sh | niemeyer: no, but I got transferred to a different team | 13:34 |
niemeyer | Dmitrii-Sh: Are you coming to the standup? | 13:35 |
Dmitrii-Sh | no, I declined the invite for today | 13:37 |
mup_ | Issue operator#222 opened: Test harness: network_get support <enhancement> <Created by dshcherb> <https://github.com/canonical/operator/issue/222> | 14:23 |
mup_ | PR operator#210 closed: tls-certificates interface (requires side, existing interface) <🆕 interface> <Created by dshcherb> <Closed by niemeyer> <https://github.com/canonical/operator/pull/210> | 14:54 |
mup_ | PR operator#216 closed: tcp-load-balancer interface (provides and requires sides, completely new interface) <🆕 interface> <Created by dshcherb> <Closed by niemeyer> <https://github.com/canonical/operator/pull/216> | 14:54 |
mup_ | PR operator#135 closed: Operator Framework charm writing guide <Created by VariableDeclared> <Closed by niemeyer> <https://github.com/canonical/operator/pull/135> | 14:59 |
* facubatista -> lunch | 15:12 | |
* facubatista is back | 16:28 | |
* facubatista brb | 17:55 | |
facubatista | Docs for the debugger: https://discourse.juju.is/t/live-debugging-at-python-level/2914 | 19:35 |
* facubatista eods | 21:00 | |
niemeyer | facubatista: Have a good evening | 21:00 |
facubatista | niemeyer, thanks! | 21:01 |
mup_ | Issue operator#223 opened: Operator fails on Juju 2.7.0 and below <Created by chris-sanders> <https://github.com/canonical/operator/issue/223> | 22:07 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!