[00:00] kelvinliu: it wasn't added in the rc branch [00:01] only the 2.8 branch [00:01] ah, ic [00:08] wallyworld: did you want me to quickly look at both issues and land in 2.8? [00:08] hpidcock: if you had time that would be grand [00:08] np [02:01] does the '--map-machines' option need to be used when deploying with both a bundle and an overlay? i'm thinking that the bundle will define the existing machines and then the overlay needs the option. or is an existing machine completely "pre-deploy"? [04:25] wallyworld: https://github.com/juju/juju/pull/11720 [04:26] ended up chasing my tail, thought I saw really small binaries of 40mb etc, for some reason was unable to repro [04:31] looking [04:33] hpidcock: lgtm ty [04:33] i shold merge directly [04:34] i'll let the check run pass [04:46] hpidcock: merged [05:09] stupid question sorry... is --device just a k8s thing? [05:32] timClicks: yes === arif-ali_ is now known as arif-ali === salmankhan1 is now known as salmankhan [09:01] achilleasa: QA is failing for me on #11716 [09:01] Bug #11716: psycopg: new changes from Debian require merging [09:02] Stupid mup. The PR, not LP. [09:02] mup-- [09:06] juju#11716 [09:06] * Chipaca guesses [09:06] * Chipaca guesses poorly [09:06] manadart: ho? [09:07] achilleasa: OMW [09:50] manadart: pushed the acpi fix to all kvm PRs; taking a look at the failed QA now [10:11] CR anyone https://github.com/juju/juju/pull/11715 [10:13] stickupkid: I'll do it, then swing back to achilleasa's stuff. [10:15] manadart, run this https://github.com/juju/juju/pull/11715#issuecomment-645285581 if you intend to run it all [10:15] manadart, life is too short [10:25] stickupkid: Running now. [10:35] stickupkid: Done. You still looking at mine? [10:35] manadart, nearly done [11:00] manadart: hmmm... so the remove call is skipped because lxd reports the nic type as "broadcast" instead of "bridged"... [11:02] achilleasa: Yeah, I added some Criticalfs here was surprised they weren't hit. So it's bugging out earlier. [11:02] also, the lxd from snap on focal seems to remove the entry from the bridge as expected... [11:02] yay :D [11:02] so I need to tweak some things and re-test on bionic [11:03] achilleasa: I suspect this difference goes hand-in-hand with the profile changes. [11:04] So we get 3.0.3 on Bionic and the latest Snap on Focal. [11:04] I will amend my comments accordingly [11:27] manadart: updated the PR and verified it works on bionic (I also move the maybeRemove call a bit higher up); can you take another look? [11:27] achilleasa: Yep, saw. Upgrading now. [11:34] manadart: if you set juju.core.networking level to DEBUG you should see a log of the call to remove the veth [11:40] achilleasa: Yep, that one's done. Onto the next. [12:23] achilleasa: You try and merge the wrong PR? [12:24] manadart: yeah. cancelled the merge once I spotted it... too many open tabs [12:24] the right one just landed [15:32] stickupkid: i think i know why height and width are not unmarshalling in Media… they can return integer or null [15:33] stickupkid: pieced that info out of http://api.snapcraft.io/docs/charms.html [15:34] hml, any reason why we read the resp.Body twice? [15:35] stickupkid: not that i can think of. [15:35] stickupkid: missed that in putting together the spike [15:35] hml, as we're not doing auth yet, I'm going to cull most of the do command and bakery client for now [15:36] hml, we can add as required (which I'm sure will be soon) [15:36] stickupkid: will it be harder to move back in when we do? [15:36] hml, nah, just align to an interface [15:36] stickupkid: rgr [15:37] hml, so the http client and the bakery client both conform to `func (Client) Do(request *http.Request) (http.Response, error)` [15:38] hml, so as long as we do that, we can inject what ever we want [15:38] stickupkid: ack === narindergupta is now known as narinderguptamac === petevg_ is now known as petevg === zeestrat_ is now known as zeestrat === bloodearnest_ is now known as bloodearnest === balloons7 is now known as balloons [21:26] wallyworld: Hey, for CMR, is there a way to get the remote side's model UUID? [21:34] wallyworld: Actually, I should step back. What I really want to know is when a CMR is created on AWS, how are the SGs modified so that the two units can talk to each other? Are specific rules added to the unit SGs? [21:40] does the juju/gnuflags package support modifying a Var once it's been added to the flagset? [21:42] the juju deploy command includes flags from add-unit directly.. I would like to add " (charm only)" to their help lines when someone execs `juju help deploy` [21:55] wallyworld: Nm, tested and confirmed that it's the specific rules on the unit SG, which makes sense. [22:41] thumper: I keep wanting to make juju help topics a thing again [22:41] juju help charm-urls [22:45] timClicks: I'm ok with that... [22:52] I'm going in circles trying to make 'juju help deploy' shorter. It's getting easier to read, but longer. [23:48] wallyworld: https://github.com/juju/juju/pull/11723 [23:49] ok [23:49] tlm: add bug to PR description, make as in progress... [23:49] ah yep [23:51] tlm: just to test end-end, might be work hacking an existing charm and try it, as we've seen once or twice a missing bit in practice, just to be sure etc [23:51] it looks ok, but you never know [23:52] any suggestions what what charm to hack etc ? [23:52] i use a mariadb one i have unpacked locally. ~juju/mariadb-k8s [23:53] cs:~juju/mariadb-k8s [23:53] just hack the extra line in the yaml