wallyworldkelvinliu: no hurry, this PR fixes a couple of action test races https://github.com/juju/juju/pull/1061201:25
=== exsdev0 is now known as exsdev
kelvinliuwallyworld: yep looking01:31
kelvinliuwallyworld: lgtm, just small question. thanks01:48
wallyworldkelvinliu: that's the issue - using defer is insufficient because the Stop() does not occur soon enough -it needs to happen immediately when Exec() returns01:49
wallyworldthe defers are there to ensure Stop() is called in the case of early exit01:49
wallyworldand that's why also Stop() was made idempotent01:50
kelvinliuok, ic01:52
wallyworldhopefully that makes sense01:52
wallyworldunder --race it can lead to incorrect capture of stdout01:52
wallyworldthe original local only exec used a non-defer Stop()01:53
wallyworldi changed to defer and that broke01:53
kelvinliuis it ok to combine Stdout n StdoutLogger?01:53
wallyworldin the adaptor struct?01:54
wallyworlddon't quite follow, quick HO?01:54
hpidcocksmall one https://github.com/juju/juju/pull/1061302:45
timClickswallyworld: is there a spec for user friendly actions IDs in juju/names?02:51
hpidcock<application>-<number> or <UUID>03:09
anastasiamachpidcock: anyone for? review of 10613?04:35
anastasiamacfwiw i disagree witht the original implementation -that check againsta an arbitrary regular expression should really be a chack against names pkg valid definition of action 'id' (whichever version of that pkg was shipped with that Juju)04:42
anastasiamacotherwise it's an ongoing headache to keep changing this reg exp to match our improved understand of id... or as per ur pr making it braod enough to work for all scenarios...04:43
wallyworldanastasiamac: the CI test in Python does replicate the same check as is done in the juju names pkg. the names pkg was recently updated but the python test wasn't05:54
anastasiamacwallyworld: i understand but it has a copy of reg exp, rather than use the actual pkg that Juju was shipped with... anyway05:55
wallyworldone is Go, the other is Pythin though05:55
wallyworldgawd, why do I keep typing "puthin"05:56
wallyworldfark, fail05:56
wallyworld"pythin" even05:56
wallyworldi like test that break when we change implementation as it forces you to reevaluate the correctness of the test etc05:56
anastasiamacwith all my typos, i got it :) thank you for understanding me over the years \o/05:57
wallyworldi need an autocorrect for it05:57
wallyworldbut i have have stand up arguments with other people in the past who disagree with me on that aspect of test writing05:58
hpidcockwallyworld: pythin or pythick?06:05
SeubertEHi there everyone! I'm new to using Juju and trying to set it up on my home lab. I had a few questions that came up while reading the docs (https://jaas.ai/docs/getting-started-with-juju). First, do I need to use JAAS with juju? Or can I run juju all on my home network?06:19
wallyworldSeubertE: hi there. you can run juju locally using LXD containers (no need for JAAS to get started). The current docs do not reflect that so well. These are being updated to give a much better getting started experience. So long as you have snap installed LXD, you can "juju bootstrap lxd" to get up and running06:27
SeubertEThanks wallyworld, that's what I've been doing so far, but got confused since the docs kind of push JAAS in getting started. I've also been having issues with the gui, would it be better to just post on discourse?06:30
wallyworldthat would work yeah, so then others can benfit from the answers06:31
SeubertEAwesome, thanks, I'll post my issue there!06:32
rick_hworks here, but not in another channel geeze13:34
jamguild can I get a review of https://github.com/juju/juju/pull/10615 (its quite small)14:03
stickupkidjam, should this be on 2.6 as well, or are we not bothered?14:04
timClicksdoes it make sense to say something like this: "21:25
timClicksCharms can define "storage" endpoints. Clouds provide "storage pools", tied to the capabilities of the underlying provider."?21:25
timClicksin particular, do charms define storage endpoints? I'm trying to figure out the best way to express what's defined in metadata.yaml21:26
timClicksanswering self - the terminology we use currently is a storage label.21:29
anastasiamactimClicks: yeah i would not call them "endpoints" :)21:58
wallyworldtimClicks: generically, clouds tend to have ways to provision storage, either as a block device, or a filesystem. some clouds use a "storage pools" concept but at the end of the day, juju models it as asking the cloud for either a block device or filesystem22:26
=== exsdev0 is now known as exsdev

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