/srv/irclogs.ubuntu.com/2020/06/02/#smooth-operator.txt

mupIssue operator#317 opened: use charm state for Juju 2.8 <Created by jetpackdanger> <https://github.com/canonical/operator/issues/317>04:52
Chipacamorning all08:29
jammorning Chipaca08:33
facubatistaMuy buenos días a todos!11:04
Chipacaa wild facundo appears!11:12
* facubatista puts some clothes on11:14
Chipacabetter11:25
Chipacafacubatista: hola :)11:25
facubatistahola :)11:25
Chipacai've just realised i spent all morning messing around with versioning :-(11:25
facubatistaChipaca, and this is the 3.1.2nd time it happes to you!11:25
facubatista*happens11:26
Chipacawell, i did do a bit of reviewing and such before diving down this rabbithole11:26
facubatistaChipaca, but at least you have a conclusion?11:26
Chipacafacubatista: I have something I like, I just need to check one more thing11:26
Chipacahm, the files github gives us at https://github.com/canonical/operator/releases is already reasonably useless :-/ do we care about them?11:28
facubatistaChipaca, for `fades`, the person doing the debian packaging takes those11:29
facubatistaChipaca, anyway, I don't care about them, not even to take the action of removing them or something11:30
Chipacaheh11:30
facubatistaChipaca, re MANIFEST.in... "LICENSE": don't we need to package it? why do we have it in the first place? shall we remove it from the project?11:31
Chipacafacubatista: in the wheel, it's already there11:32
Chipacafacubatista: in the source dist, we reference it by name, isn't that enough?11:32
Chipacai'm asking :)11:33
facubatistaChipaca, who put it in the wheel?11:33
Chipacafacubatista: wheel :)11:36
Chipacafacubatista: FWIW I don't object to it being there11:37
facubatistaChipaca, even if we don't specify to put it in the tarball?11:37
Chipacafacubatista: correct11:37
facubatistawow11:37
Chipacafacubatista: you can try it yourself :)11:37
facubatistanah, I trust you11:37
Chipacafacubatista: 'python setup.py bdist_wheel'11:37
Chipacashaaame11:37
Chipacaoh dang lunch11:37
* Chipaca rushes11:38
Chipacaso, it *looks* like github doesn't auto-add assets if you add them manually, so that might be a good way out12:13
facubatistaChipaca, what we do about LICENSE file, then?12:15
Chipacafacubatista: give me a bit12:16
Chipacajust digging into that :)12:16
Chipacabah. i should probably shelve the version stuff first :-D12:16
facubatistaack, no rush12:16
Chipacafacubatista: so one approach is to add a setup.cfg with [metadata] license_file = LICENSE.txt12:21
Chipacalicense_files*12:21
facubatistaChipaca, jam, now I pushed the files to where they belong, so I won't confuse again my directories: https://github.com/facundobatista/blog/tree/master/charm12:26
Chipacai don't have a stick big enough to shake at how much i dislike the under-documentedness of setup.py and .cfg12:30
jamChipaca, simple enough :)13:08
Chipacagrmbl13:10
Chipaca(tm)13:10
facubatistaChipaca, see, I created a clean venv, then `pip install -r requirements-dev.txt`, and I get:13:17
facubatistaInstalling collected packages: pycodestyle, toml, autopep8, mccabe, zipp, importlib-metadata, pyflakes, flake8, PyYAML, ops13:17
facubatistaI don't like that "ops" at the end13:17
facubatistaoh, mmm13:18
facubatista$ cat env/lib/python3.6/site-packages/ops.egg-link13:18
facubatista /home/facundo/canonical/operator13:18
facubatistaprobably not harmful, though13:18
Chipacayep, it's done a 'develop'13:19
* Chipaca links to https://xkcd.com/1987/ on the README.md14:03
jamChipaca, :)14:05
jamdanger of using 'sudo' to install python packages, indeed14:06
mupPR operator#318 opened: Simplified the CharmBase init signature <Created by facundobatista> <https://github.com/canonical/operator/pull/318>14:08
facubatistanever ever do "sudo pip"14:09
jamfacubatista, do "curl | sudo bash" right?14:09
facubatistaja14:10
facubatistaoh, Trello is down :/14:18
Chipacafacubatista: if trello is down, are you even working14:57
facubatistaChipaca, I'm not doing anything but pressing F5 every couple of minutes, staring at the empty browser page15:00
Chipacafacubatista: let's have our 1:1 now then :)15:01
facubatistaje15:02
facubatistaChipaca, bah, I'm ok for the 1:1 now if you prefer it15:03
Chipacafacubatista: let's do it15:06
facubatistaok, 1'15:06
facubatistaChipaca, omw15:08
Chipacasoft EOD from me. brain = ded16:43
Chipacamigt bbl if it reactivates16:43
facubatistaChipaca, should we distribute ops docs inside its package?17:10
=== facundo__ is now known as facubatista
* facundo__ eods21:04
* Chipaca eods22:24

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!