=== _thumper_ is now known as thumper
stickupkidmanadart, https://github.com/juju/juju/pull/1209308:04
stickupkidachilleasa, something wrong with your check09:31
stickupkid[m[31mexpected opened-ports output to be: -e 1234/tcp 1337-1339/tcp GOT: 1234/tcp 1337-1339/tcp[m09:31
stickupkidachilleasa, strange because it's the right value as well09:32
achilleasahmmmm... it passes for me.09:32
achilleasayeah, I had to 'echo -e' it for the line-feeds to work09:32
achilleasaare you using dash?09:32
achilleasatry it with dash09:33
stickupkidha, no09:33
achilleasait would be much easier if we simply used bash :D09:33
stickupkidbecause they're meant to be transportable09:33
achilleasawhy though?09:33
achilleasait's not like someone else will be running them anyway ;-)09:33
stickupkidbut it should work in bash, dash, zsh, etc09:34
achilleasahmmm... I am not sure why that comparison fails for you though :-(09:34
achilleasacan you add a 'hexdump -C' for both values?09:34
achilleasathey should be identical09:34
achilleasaif comparisons with line-feeds do not work, we will have to md5sum them which sucks09:35
stickupkidjust running another test, will check after that09:35
stickupkidecho "${got}" | tr '\n' ' '09:37
stickupkidrip out the new line09:37
achilleasayeah, you can tr -d '\n' but that's ugly...09:38
stickupkideverything is ugly09:39
achilleasaok, let me try that to see if it works :D09:39
stickupkidi'm trying it atm09:39
stickupkidachilleasa, well that didn't work10:05
stickupkidachilleasa, expected opened-ports output to be: -e 1234/tcp1337-1339/tcp GOT: 1234/tcp1337-1339/tcp10:06
achilleasaI think I know why... waiting for my test to finish10:06
achilleasawe should really quote the exp/got parts10:06
achilleasathere is a trailing space due to the \n from juju run10:07
stickupkidson of a10:07
achilleasastickupkid: pushed a change + your patch (extra commit). It should work for you now (fingers crossed!)10:12
stickupkidachilleasa, works10:21
manadartstickupkid, achilleasa: Either of you able to review https://github.com/juju/juju/pull/12067 ?11:24
achilleasamanadart: I can look in about 15min11:25
stickupkidmanadart, updated the PR for wait-for14:34
manadartstickupkid: Ack.14:34
manadartstickupkid: https://github.com/juju/juju/pull/12074 is awesome. Check my comment.15:00
stickupkidmanadart, I saw, I do wonder about GT, LT, LTE or GTE on strings, but then you're daft if you use it on string15:10
stickupkidhml, so it begins :) https://github.com/juju/description/pull/8816:03
hmlstickupkid: w00t16:03
hmlstickupkid: what’s juju:v2 ?16:04
hmlstickupkid: nm16:05
stickupkidhml, shouldn't that be `description:v2`??16:05
hmlstickupkid: brain fart, i thought it was going into juju/juju not description.  ;-)16:05
stickupkidhml, yeah but still, i think that's wrong16:06
hmljuju is using v2 for description right now16:06
hmlstickupkid: quick one: https://github.com/juju/juju/pull/1209516:14
hmlstickupkid: approved 8816:15
stickupkidhml, approved yours... this is what I wanted ages ago16:16
stickupkidget rid of the hack16:16
hmlyes!  the data was finally available…16:16
=== jose- is now known as jose
JackStormSo with the later versions juju-gui is baked in, and "juju gui" confirms the new dashboard is avalible on my controller after bootstraping...but how do you expose the dashboard now? (I could do it OOB, but would rather not start one offing things) since it's not an app that expose understands19:08
bradmI'm seeing some weird error messages from juju after I've bootstrapped when I'm trying to add new machines to the environment:21:56
bradm2020-10-06 06:13:13 ERROR juju.core.raftlease store.go:265 timeout waiting for Command(ver: 1, op: claim, ns: singular-controller, model: 347130, lease: fbab0193-797f-42f0-8146-7d3eb70d642c, holder: machine-0) to be processed21:56
bradm2020-10-06 17:08:06 WARNING juju.worker.httpserver log.go:181 http: TLS handshake error from x.y.z.18:59472: EOF21:57
bradmthe TLS handshake error appears every time a machine tries to download the agent binaries21:57
bradmjust filed https://bugs.launchpad.net/juju/+bug/1898802 about it, if anyone has any ideas I'd appreciate it.23:55

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