[00:28] rick_h: heya. about to go into standup, will be free to chat in a little while [00:30] axw: k [00:31] babbageclunk: standup? [00:31] oops [01:03] rick_h: ok, reading your notes now [01:04] axw: k, I'd like to see what I'm missing for --attach to work so I can move forward [01:04] axw: the rest is feedback/notes just on using stuff [01:05] rick_h: I don't understand why --attach-storage isn't working for you. it's working for me, and I'm pretty sure wallyworld used it in his demo [01:05] axw: is there a feature flagged or something? [01:06] Oh...maybe I didn't use the snap path. Let me double check. [01:06] rick_h: no, there was one but I removed it ages ago [01:06] yeah, worked for me in demo IIANM [01:07] externalreality: reviewed PR, see if my comments make sense [01:09] axw: yea, that's it. Apologies. I have multiple Juju on here and forgot to use the right /snap/bin path [01:10] I'm not sure why I didn't notice it through all the notes/etc [01:10] axw: let me know if any of the other notes are unclear or you want to chat about [01:11] axw: I assume this works on OpenStack and AWS, does it work anywhere else? [01:11] rick_h: AWS, GCE, OpenStack, Azure [01:11] axw: <3 awesome [01:11] rick_h: oh, and LXD :) [01:12] rick_h: there's some new functionality which I've commented on in the doc, which is only supported on AWS and LXD atm: releasing and importing storage [01:12] axw: yea, I've got to get the zsh thing setup to play with that side some. [01:12] axw: k, ty [01:12] rick_h: you can now do this: juju remove-storage --no-destroy pgdata/0. that'll remove it from juju, but leave the EBS volume (or whatever) alone [01:12] axw: k, cool [01:13] rick_h: you can then: juju import-filesystem ebs vol-123456 pgdata, into a new controller. you'll end up with pgdata/N with that EBS volume [01:13] rick_h: still need to implement import-volume [01:13] axw: hmm, wouldn't it need to know the charm to make sure the title is ok for the declared storage location? [01:14] axw: e.g. can I just call it whatever vs pgdata? [01:14] axw: and if so how would I use it as I note in the docs the attach-storage doesn't take the extra arg like deploy --storage does? [01:14] we talked about that, but for now the import name needs to match the charm storage name [01:14] rick_h: you can call it whatever. it's a bit rough, I'm not sure what the best UX is there. but it needs to be able to support importing before deploying the app [01:14] wallyworld: right, but how's that validated on import then if you don't specify the charm name? [01:15] it's not right now [01:15] this was all done last minute [01:15] axw: ah, yea, I think that's why i'm wondering if the arg needs to not get dropped in --attach-storage vs --storage [01:15] rick_h: yes, probably [01:15] wallyworld: k, well just the questions/issues I saw in playing. Totally understand that it's WIP and such [01:15] np, the feedback is awesome [01:16] rick_h: +1, thank you [01:16] we know though there's more to do [01:16] ok, I'll play with more tomorrow then and demo on the juju show tomorrow afternoon. [01:16] yay [01:16] anything else I think of I'll add to the doc [01:16] ty for the time axw and wallyworld [01:16] rick_h: and the cmr blog issue? [01:16] wallyworld: replied to your email [01:16] i think we should go ahead, the macaroon stuff is internal [01:17] wallyworld: I'll make one more run by uros just to make sure that it's not a case of two folks hearing two different things from the same conversation. [01:17] wallyworld: but I can steer a path I'm sure [01:17] sounds good ty [04:05] * babbageclunk goes for a run [04:42] babbageclunk: for whenever, here's a followup to the pr from yesterday - offer connection info now displayed in status https://github.com/juju/juju/pull/7698 [05:06] wallyworld: back now but I want to get some things finished off first [05:07] no hurry at all [05:07] just whenever [05:09] wallyworld: cool cool [05:42] axw: ping? [06:08] axw: here's a small one to fix something that was shitting me with juju offers (was on the todo list) https://github.com/juju/juju/pull/7699 [06:36] babbageclunk: sorry, pong [06:36] wallyworld: looking [06:37] wallyworld: why not just make it a model command? [06:38] wallyworld: you can still get a controller API for a model command, if that's why you didn't === frankban|afk is now known as frankban [07:31] axw: it can't be a model comand because it talks to a controller facade [07:31] wallyworld: wallyworld: you can still get a controller API for a model command, if that's why you didn't [07:31] hmmm, ok, i'll look into it [07:32] wallyworld: see: ModelCommand.NewControllerAPIRoot [07:33] ok, i had a feeling there was something else amiss, but maybe not. there was a lot of churn at one point with all the facades [08:14] Ha ha, spent the afternoon writing Python and suddenly I'm leaving off all the closing braces. [08:21] :) [16:21] Hello juju... Chloe Grace wanted to say hi, https://goo.gl/photos/wVPHohn5MLbf684TA [16:21] redir: !!!!!!! [16:21] hope all is well in juju-land [16:22] redir: congrats sir! All the best to the little one and momma [16:22] thank you, rick_h :) [16:22] redir: very very happy for you [16:22] me too, she's awesome [16:23] :) === frankban is now known as frankban|afk === externalreality_ is now known as externalreality