babbageclunkFun test fix: https://github.com/juju/juju/pull/1101400:01
lucidoneIt doesn't look like the juju gui supports SAAS, and the UI crashes if you try to show relations on an application that is related to a SAAS application .. Is https://bugs.launchpad.net/juju-gui the place to report a bug?00:03
babbageclunklucidone: yes, that would be the right place to start (although it might turn out to be a juju bug under the hood)00:30
lucidonebabbageclunk: Cheers, this bug in particular is a js error "TypeError: Cannot read property 'serviceName' of undefined", so I imagine it's in the gui code. Supporting SAAS in juju gui itself feels more like a feature request00:32
babbageclunklucidone: yeah, that makes sense00:33
wallyworldbabbageclunk: state pr lgtm with a small fix to error processing01:05
babbageclunkwallyworld: cool, thanks01:05
lucidoneLooked like there was more activity on github issues (and I was blocked by ubuntuone login not working on launchpad..) so I've posted to github https://github.com/juju/juju-gui/issues/402301:17
babbageclunklucidone: oops, sorry for the bum steer there01:53
lucidonebabbageclunk: No worries! I did see a 2019 bug report there, so didn't appear to be totally defunct01:54
timClicksis it possible to serve resources locally, or is the charm store required?02:29
lucidoneAs in $ juju deploy /path/to/charm ?02:29
timClickslucidone: I am developing a charm that needs a resource02:30
timClicksbut I don't want to run "charm attach" yet to upload it to the charm store (it's not ready)02:30
lucidoneAh right02:31
* lucidone reads the docs for resources02:31
timClicksit looks like I can use `juju attach-resource`02:32
ec0thumper: if you're around in about 30 minutes I should be free to catch up on that Cisco upgrade02:58
babbageclunktimClicks: I'm late to the party but I think you can use deploy with --resource to attach the resource at deploy-time as well (rather than doing it after the fact)03:03
timClicksbabbageclunk: yeah you can :)03:04
palGood Morning Juju!07:58
nammn_de1ptal, small one: https://github.com/juju/juju/pull/1101109:25
nammn_de1*ptal, please? :D09:25
nammn_de1stickupkid: morning, thats what i meant yesterday with forcing in init. Sadly we cannot use supercommand fields, thus no commonflags. https://github.com/juju/cmd/pull/7109:26
nammn_de1stickupkid: wanted to discuss that before adding more tests ect.09:26
stickupkidmanadart, ping09:31
manadartstickupkid: Pong.09:32
manadartOMW. 2 secs.09:33
nammn_de1stickupkid: hows that comment? https://pastebin.canonical.com/p/xq2ShFXnvY/09:36
stickupkidnammn_de1, ok, that'll do09:36
nammn_de1stickupkid: great, updated pr09:40
nammn_de1stickupkid manadart rick_h: just realized as i was going through the branches test why they fail on 2.7. Do we want to backport this? https://github.com/juju/juju/pull/10796/09:50
nammn_de1nvm me, seems like they are..09:51
nammn_de1stickupkid: for the pr https://github.com/juju/juju/pull/11011 i updated the comments+naming as you said, additionally i added the fix to make sure that there is no race condition for the acceptancetest test-branches10:04
manadartnammn_de1: Yes, back-port it.10:39
nammn_de1manadart: apologies, i just realised this already has been the case. Lucky me10:44
nammn_de1while at it, manadart stickupkid: ptal https://github.com/juju/juju/pull/11011 ?10:45
nammn_de1stickupkid: if you have time later we can talk/look at the init change here https://github.com/juju/cmd/pull/71 . Ping me up if you up for it11:30
nammn_de1how do i enable all featureflags?11:31
nammn_de1is it exporting `export JUJU_DEV_FEATURE_FLAGS="" `?11:31
nammn_de1manadart: changed it, you will hate me, but i rebased as this change was more trivial, hope you don't mind:  https://github.com/juju/juju/pull/11011/files11:34
stickupkidmanadart, CR on this one please :D https://github.com/juju/description/pull/6911:57
nammn_de1stickupkid: ping me if you are around for my python questions and/or the init cmd thingy14:07
stickupkidnammn_de1, do it now?14:08
nammn_de1stickupkid sure, daily?14:08
manadartstickupkid: https://github.com/juju/juju/pull/11020. I am heading to HO.14:26
nammn_de1stickupkid: what was the output of doing it in init again? How do we want to proceed? https://github.com/juju/cmd/pull/7115:26
nammn_de1rick_h: im a bit struggling to be able to connect to the allwatchersapi through python, any pointers for me?  `WatchAllModels` returns an id of  an allwatcher which iam supposed to use with something15:45
nammn_de1just cant find something easiliy15:45
rick_hnammn_de1:  sec, otp for a bit can help look afterwards15:58
nammn_de1rick_h: thanks! stickupkid: is there a way to set a context for a facade connection?16:03
nammn_de1as it seems the watcher_id is taken from a context param, which i cannot give for the watcher.next() call16:03
stickupkidnammn_de1, have a look at what the go code, see how it uses it, that's generally what I'd do...16:04
nammn_de1cr someone? https://github.com/juju/juju/pull/1102116:16
nammn_de1stickupkid: they only api things  we can call through python have to be behind facades, right?16:59
nammn_de1so this call https://github.com/juju/juju/blob/6ea20273f7c540d93b1075d77aafe39e70770b33/api/controller/controller.go#L186, which is returning the real watcher, is not directly possible and we might have a facade equivalent construct somewhere?17:00
stickupkidnammn_de1, not at all, each go client knows about it's facade it's going to use. It just abstracts that information away, the python code has no such idea.17:02
nammn_de1stickupkid: thanks and dammnit17:02
stickupkidnammn_de1, see https://github.com/juju/juju/blob/6ea20273f7c540d93b1075d77aafe39e70770b33/api/controller/controller.go#L3617:03
nammn_de1stickupkid: doesnt this suck in this case? Would need to add a new facade for this to work in python, right?17:03
stickupkidnammn_de1, nope, we export all facades for python to consume17:04
nammn_de1stickupkid: i need to understand how the attributes are set in python17:04
stickupkidnammn_de1, there is a set of facades we don't export https://github.com/juju/python-libjuju/blob/master/juju/client/facade.py#L2117:05
stickupkidnammn_de1, maybe it's that one17:05
nammn_de1stickupkid: this quick hack did it for me: https://pastebin.canonical.com/p/sBskSH7QdQ/17:28
stickupkidnammn_de1, just remove "ClientFacade" from this https://github.com/juju/python-libjuju/blob/master/juju/client/facade.py#L21 and then follow this guide https://discourse.jujucharms.com/t/python-libjuju/155317:30
nammn_de1stickupkid: I will keep that in mind,ta17:32
timClicksbabbageclunk: ping, question in discourse for your perhaps? https://discourse.jujucharms.com/t/application-level-data-questions/244722:35
babbageclunktimClicks: thanks! I'll have a look in a bit22:36

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