| lucidone | Hi, when running juju add-k8s $cluster; juju bootstrap $cluster; against a k8s cluster in openstack. The controller service comes up with a ClusterIP. As such, it fails when it tries to contact the juju controller / api server becuse the address is inaccessible. I'm probably blind / doing something wrong, but is there a config setting to make it use a spec.type=LoadBalancer instead? | 02:27 |
|---|---|---|
| wallyworld | lucidone: there's a new 2.7 feature https://discourse.jujucharms.com/t/whats-new-in-juju-k8s-for-2-7/2300 | 02:32 |
| wallyworld | you can specify a bootstrap option | 02:32 |
| wallyworld | hpidcock: the final PR to remove charmstore.v5 https://github.com/juju/juju/pull/10988 | 02:33 |
| hpidcock | ok | 02:34 |
| lucidone | wallyworld: ah, magic! thanks | 02:37 |
| lucidone | --config controller-service-type=loadbalancer worked a charm ;) | 02:45 |
| thumper | I'd love a review of my ratelimit PR: https://github.com/juju/juju/pull/10987 | 02:56 |
| hpidcock | thumper: having a look | 04:07 |
| thumper | hpidcock: thanks | 04:54 |
| achilleasa | manadart: I fixed the broken test (https://github.com/juju/juju/pull/10979/commits/dcd69a17088c7d19bf5bcf95e7e025b661440dcc) and I think I got all your suggested changes in 10979; can you take another look? | 08:58 |
| manadart | achilleasa: Yep. | 08:59 |
| achilleasa | manadart: also, I got rid of some not needed code in the instancepoller facade which triggered the original type-casting error we were seing (this line: https://github.com/juju/juju/pull/10979/commits/2393a6d3d1ce273564080359a62d46090b952ceb#diff-d177f61f881e38ac7ccb857814927e3fR118) | 09:01 |
| nammn_de1 | manadart stickupkid up for a small review? https://github.com/juju/juju/pull/10985 | 10:44 |
| nammn_de1 | rick_h: regarding the patch we were talking about yesterday | 12:52 |
| nammn_de1 | ^ | 12:52 |
| rick_h | nammn_de1: did you get the feedback from thumper? | 13:00 |
| rick_h | I mentioned it on a call with him and he was going to take a peek | 13:00 |
| nammn_de1 | rick_h: yes I did, I tried to work his feedback in, but needed to adjust a few places different | 13:05 |
| nammn_de1 | I need make the err silent as manadart has mentioned | 13:05 |
| rick_h | nammn_de1: gotcha, sorry yea just catching up on email this morning | 13:07 |
| nammn_de1 | manadart: on your comment: https://github.com/juju/juju/pull/10985#issuecomment-562123107 | 13:17 |
| nammn_de1 | That are the options I see, any else in your mind? | 13:17 |
| stickupkid | hml, can you check this one, this is the one that breaks all the tests :( https://github.com/juju/juju/pull/10990/files | 14:29 |
| hml | stickupkid: looking | 14:29 |
| hml | stickupkid: is there face to palm asci art? | 14:29 |
| hml | for me? | 14:30 |
| stickupkid | generally Patrick Stewart isn't it? | 14:30 |
| stickupkid | https://i.kym-cdn.com/entries/icons/original/000/000/554/picard-facepalm.jpg | 14:30 |
| hml | errr… don’t know that one | 14:30 |
| hml | ah, i was thinking asci, not jpg | 14:31 |
| stickupkid | interestingly I've met him on multiple occations, depends if you're a trekkie or not | 14:31 |
| hml | doh | 14:31 |
| stickupkid | https://www.google.com/search?q=facepalm+ascii&sxsrf=ACYBGNQ_Sdi-AUddeexe3IF_Ujw4YkKalw:1575556212640&source=lnms&tbm=isch&sa=X&ved=2ahUKEwiH0oKj3J7mAhXgURUIHfSOCC8Q_AUoAXoECA0QAw&biw=1272&bih=635#imgrc=-uejGHWQzbfg2M: | 14:31 |
| hml | stickupkid: seen a lot of it. | 14:31 |
| hml | wow | 14:31 |
| hml | stickupkid: posted 2 comments | 14:37 |
| rick_h | hml: can I bug you back in daily around the error message for the neutron thingy please? | 14:38 |
| stickupkid | hml, done https://github.com/juju/juju/pull/10990 | 14:45 |
| hml | stickupkid: approved | 14:57 |
| stickupkid | ta | 14:58 |
| hml | anyone know if there’s a juju command to get the <commands> allowed with juju-run? or it is just in docs? | 15:29 |
| hml | found it | 15:31 |
| timClicks | we received a 35x increase in anonymous pageviews on our discourse yesterday | 19:09 |
| rick_h | timClicks: :/ | 19:10 |
| timClicks | not sure what the significance is; but it does throw everything out | 19:11 |
| rick_h | yea, I bet | 19:13 |
| timClicks | blog post about juju on the ubuntu blog! https://admin.insights.ubuntu.com/2019/12/05/web-application-development-with-juju-charms/ | 20:16 |
| thumper | hml: here's a quick PR for you https://github.com/juju/juju/pull/10992 | 20:33 |
| thumper | timClicks: can we catch up now? | 20:33 |
| thumper | timClicks: Friday meetings have been all screwed up since daylight savings | 20:33 |
| timClicks | thumper: sure, 2 mins | 20:33 |
| hml | thumper: looking | 20:35 |
| thumper | hml: came across them trying to land my other branch | 20:35 |
| hml | thumper: we seem to be hitting more intermittent again recently. nice to see a fix for one. | 20:36 |
| hml | or two | 20:36 |
| wallyworld | hml: tiny https://github.com/juju/bundlechanges/pull/58 pretty please | 21:32 |
| hml | wallyworld: looking | 21:32 |
| hml | wallyworld: juju:master? | 21:33 |
| wallyworld | hml: parse rror? | 21:34 |
| hml | wallyworld: looks like your pr is trying to merge into master instead of develop | 21:35 |
| wallyworld | this is bundlechanges | 21:35 |
| wallyworld | a different repo | 21:35 |
| hml | wallyworld: ah | 21:36 |
| wallyworld | we need to use bakery.v2 in juju and it's a friggin rabbit warren to get all the dependencies sorted | 21:37 |
| hml | wallyworld: definitely! we’ve all(?) been down that path a run screaming at one time or another. :-D | 21:37 |
| wallyworld | i'm living it :-( for the past week | 21:38 |
| hml | wallyworld: :-( | 21:38 |
| wallyworld | finally got charmstore.v5 all removed | 21:38 |
| wallyworld | now hitting issues with go dep | 21:38 |
| wallyworld | ty fore review | 21:39 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
