User1: morning all
User2: o/
User3: mo'in
User3: User1 https://r.chipaca.com/land_shallow_topo.jpg (to be fair it might have bitrot at this point, without me noticing :) )
User1: Chipaca, it doesn't render very well here.
User3: yeah, here either, and imagemagick now has limits in place that block it from touching the image even (without further tweaking)
User3: which was precisely the point the image was trying to make, so i won i guess?
User1: Chipaca, "I Win" :)
User3: quite
User3: User1 an interesting thing came up in the call yesterday, with narinder
User3: User1 he's going to file a bug about it, but i don't think he has yet
User3: User1 but if you look at https://github.com/narindergupta/charm-k8s-kafka/blob/master/src/charm.py
User3: User1 at expose_relation_data
User3: User1 ISTM that having to do that getnameinfo thing is not particularly friendly
User1: Chipaca, indeed. I know that there has been discussions on the Juju side of how to present a hostname when you have one
User1: It certainly is never guaranteed that there *is* a hostname for a unit
User1: And if there is one (in say, MAAS) it may not be accessible because the DNS server is local
User1: I also know of bugs like https://bugs.launchpad.net/juju/+bug/1867168
User3: User1 yeah, that sounds like exactly the issue
User3: User4 this Friday we scheduled a meeting with people writing a postgres charm (i'm assuming for k8s). Neither jam nor I work this Friday :) should we reschedule, or are you happy to carry it alone?
User1: is he in yet?
User1: I would recommend rescheduling, as that is one that I'd like to be at
User3: User1 I'll ask sohini to transfer these to me so we can move them around
User1: Chipaca, given that the bug is about the charm-k8s-mongodb it sounds very much like that bug
User3: User1 btw, are https://github.com/narindergupta/charm-k8s-kafka/ and https://github.com/narindergupta/charm-k8s-zookeeper/ on the list? (i don't see them)
User1: Chipaca, they are now :)
User3: User1 taw
User1: Chipaca, is there a notes doc about Kafka meeting?
User3: User1 i need to transfer my notes onto a doc
User3: User1 and, in future ones, start with a shared doc so we edit it as we go
User1: Chipaca, so I mentioned I was investigating creating symlinks earlier (before instantiating the charm)
User1: it is certainly 'doable' but it changes a bit
User1: namely, Events currently get their name from being assigned as an attribute to a class
User1: (install = EventSource(InstallEvent) is how we know the hook is 'install')
User1: I could change it so that the name of the associated hook is part of the base event, but it felt like a pretty significant departure
User1: so I wanted to check before I proceeded
User3: it does feel a bit nastier
User3: OTOH the know-thy-name magic might prove to be too magic at some point
User1: Chipaca, so in my head I was going to a registry of hook events so that we can easily iterate them. right now that "registry" is CharmEvents and iterating the attrs of it
User1: Chipaca, ObjectEvents.events() uses inspect.getmembers(type(self))
User1: which seems an odd way to do a registry to *me*
User3: User1 oh so it _doesn't_ use set_name-like magic
User3: my bad :)
User1: Chipaca, oh, it does
User1: it does both :0
User1: Chipaca, event_kind is used as the name, which is populated during __init__ because of calling getattr()
User1: AFAICT
User1: _create_event_link looks at 'bound_event' which has an 'event_kind' which is the name of the attribute
User1: (it could have just as easily used the 'key' portion of the key:bound_event map, but chose to just call values() and then iterate those)
User4: Muy buenos dÃ­as a todos!
User3: User4 buen dÃ­a!
User3: User4 could you review #258?
User4: Chipaca, sure
User4: Chipaca, regarding friday meeting, I can take it, but if jam wants to be there, we can reschedule it, yes
User1: morning facubatista
User4: hola jam
User4: meeting is in standup room?
User1: I added a room, but I'll happily move to standup since I added it late
User4: ah, no
User4: I'm in the other one
User1: Chipaca, coming? https://meet.google.com/pmg-hccz-tqd
User4: niemeyer, hello! any chance you leave a mup with us?
User5: User4 Sorry, machine crashed yesterday and I forgot to bring it back up.. still haven't managed to put it in a stable place
User4: niemeyer, no worries! thanks!
User4: jam, Chipaca, ...?
User4: jam, we lost you
User1: brb
User1: https://paste.ubuntu.com/p/GFFXXrwKCW/
User3: jam, facubatista, wrt ops.lib.use, in practice it'll be lib.use(), no? because 'from ops import <>' is the pattern we're encouraging
User3: or maybe even use()
User4: I like lib.use()
User3: from ops.lib import use as frameworkify
 * Chipaca hides
User1: mysql = frameworkify('mysql')
User1: nice
User3: User1 also, binding.ports[1000:1024].open()
User4: binding.ports[1000:1024].open(name='foo')
User4: binding.ports.close('foo')
User4: binding.ports[1000:1024].close()
User3: User4 where are we planning on saving 'foo'?
User4: Chipaca, some StoredState?
User3: User4 AIUI this means the ports depends on saving state in the controller
User4: Chipaca, we can not depend on that?
User4: Chipaca, we're saving the events already, right?
User3: User4 not in the controller yet
User3: it's saved in a local db
User4: Chipaca, why the ports storage would be different?
User3: User4 i might be getting this wrong but i think the pod can be replaced at any time at which time we lose? but maybe not, the local db
User3: ah no the db is not in the pod
User3: that's the problem with actions
User3: User4 ignore me, brain wongled
User3: User4 but don't ignore me _too_ much :-p
 * facubatista ignores odd chars in Chipaca sentences
User3: User4 any char < 2019 is rubbish anyway
User3: User4 ð§ð§ð§
User4: ja
User3: User4 plz don't forget to review the docs one
User4: extensions = [
User4: 'sphinx.ext.autodoc',
User4: ops.lib.use('autodoc')
User4: Chipaca, some questions inline
User4: also, I don't remember why we still don't have a proper requirements-dev.txt file with flake8, autopep8, etc
User3: User4 I don't remember what happened to your PR with that
User4: Chipaca, it's a good moment to add it
User3: User4 what happened to that PR?
User4: Chipaca, we end up not automatizing the virtualenv creation, so the dependencies ended up in the README
User4: (I landed that improvement over "you find out by trial and mistake what you need" as a middle ground)
User3: User4 you _don't_ need sphinx for dev though, rtd will build them for us
User4: Chipaca, unless you want something to be fixed, for when you want to check how it looked before and after
User4: but yes, that will not happen always
User4: Chipaca, maybe a docs/requirements.txt ?
User4: docs/requirements-build.txt ?
 * Chipaca adds a facubatista/requirements.txt
 * facubatista is very requiremented
 * Chipaca also adds a .snapcraft/you-might-be-lost.md
smooth-operator
Ubuntu IRC