[01:03] <thumper> menn0: https://github.com/juju/juju/pull/6873
[01:03] <thumper> menn0: that's the other half
[01:37] <menn0> thumper: looking
[01:39] <thumper> menn0: just doing extra-info for model config now
[01:39] <menn0> thumper: done
[01:39] <thumper> menn0: ta
[01:40] <babbageclunk> man, I was complaining about azure's SDK documentation but their web UI is really slick.
[01:40] <menn0> babbageclunk: well at least that's something :)
[01:40] <babbageclunk> click click click
[01:50] <thumper> menn0, babbageclunk: https://github.com/juju/juju/pull/6874
[01:50] <babbageclunk> thumper: looking
[01:55] <babbageclunk> thumper: why are we making it a string rather than an object though?
[01:57] <babbageclunk> I guess they're happy feeding it through yaml unmarshalling themselves
[01:59] <babbageclunk> thumper: LGTM'd
[02:05] <thumper> babbageclunk: yeah
[02:05] <thumper> babbageclunk: string allows more flexibility and simpler checks
[02:05] <thumper> babbageclunk: also what I agreed with :)
[02:06] <babbageclunk> thumper: it does make it hard for anything else to use it at the same time, but I guess it's per-model.
[02:06] <thumper> yeah, is per model
[02:06] <thumper> I think if we are going to have multiple things attempt to store data we probably want to think this through a bit more
[02:06] <thumper> and have a different place to store it
[02:07] <thumper> and different API etc
[02:07] <babbageclunk> thumper: What are they using it for out of curiosity? He doesn't say in the bug.
[02:07] <thumper> not sure to be honest
[02:08] <thumper> conjure-up stuff
[02:08] <thumper> hmm...
[02:08] <thumper> babbageclunk: let my look at Tattrs
[02:08] <thumper> which is map[string]string
[02:10] <thumper> ew...
[02:10] <thumper> $ juju model-config info
[02:10] <thumper> ERROR key "info" not found in {<nil> ""} model.
[02:11] <thumper> that's not a great error message
[02:15] <babbageclunk> ah well
[02:18] <thumper> babbageclunk: looked at storing as Tattr, I don't think that is a goer...
[02:18] <thumper> there is too much special casing around it for it to be robust
[02:18] <thumper> like, it doesn't deal with spaces very well
[02:19] <thumper> babbageclunk: so I'm back to previous thought, that if we are wanting to store more structured user data, we do it properly with different API can collection
[02:19]  * redir eods
[02:38] <babbageclunk> thumper: yeah, I think you're probably right.
[02:39] <thumper> babbageclunk: jam mentioned annotations in the review, and I think that would be the way to go
[02:39] <thumper> add a 'juju annotation' cli
[02:39] <thumper> that would work for models or applications or units
[02:39] <thumper> or parhaps machines
[02:40] <thumper> and have the behaviour be the same in general as the model-config
[02:40] <thumper> ish
[02:44] <thumper> babbageclunk: do you have a few minutes to chat?
[02:45] <thumper> I want to talk about the state pool
[02:45] <babbageclunk> thumper: sure. standup hangout?
[02:45] <thumper> sure
[03:03] <menn0> thumper: I'm EOW (started early today). See email for the final PR for the charm URL changes that's merging right now.
[03:05] <thumper> ack
[03:05] <thumper> menn0: have a good weekend
[03:05] <thumper> menn0: see you in just over a week
[04:14] <thumper> holy crap
[04:14] <thumper> babbageclunk: every AllWatcher created creates a state pool for all models
[04:14] <thumper> babbageclunk: each GUI connection creates an all watcher
[04:15] <thumper> babbageclunk: it could just be connecting GUIs to the controller killing things
[04:15] <thumper> I'm going to refactor things so that the AllWatchers created use the single state pool
[04:16] <thumper> and looks at exposing the state pool through the facade base objects
[04:16] <thumper> this may be a bit of surgery
[04:16] <thumper> but shouldn't be too bad
[04:16]  * thumper hopes
[04:17] <thumper> well, that is tomorrow sorted
[04:22] <jam> thumper: every state object stores a list of all transactions for every model even if it only uses some of them
[04:26] <jam> thumper: state/watcher/watcher.go 'current[watchKey]int64' tracks the revno of every object seen in the transaction log
[04:26] <jam> I'd like to see them sharing that underlying watcher if possible
[04:26] <babbageclunk> thumper: ooh, that might be it.
[04:27] <babbageclunk> gah, using the resource API to update tags on azure resources requires specifying the API version per resource type. :(
[08:43] <mup> Bug #1658549 opened: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>
[08:43] <mup> Bug #1658549 changed: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>
[08:43] <mup> Bug #1658549 opened: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>
[09:07] <deanman> Morning, is there a way (even a hack) after all to be able to deploy CentOS workloads locally with LXD ? Related bug here -> https://bugs.launchpad.net/juju/+bug/1495978
[10:00] <perrito666> morning
[10:01] <perrito666> deanman: ill be gone for about an hour, but I dont think that woul work
[10:01] <perrito666> deanman: have you tried jus passing the series and see what hapens? (assumin you have centos templates)
[10:08] <deanman> perrito666, I have simple charm that has only centos7 defined in its metadata.
[10:09] <deanman> Checking logs it nags about `machine-0: 12:08:29 INFO juju.tools.lxdclient no image for ubuntu-centos7 found in https://streams.canonical.com/juju/images/releases/`
[10:10] <deanman> so sometime ago it was hinted that you could trick it by creating a new alias for a local centos7 image with the name `ubuntu-centos7` which didn't work.
[10:10] <deanman> so open to any other suggestions :-)
[16:33] <redir> morning juju-dev
[16:35] <perrito666> redir: morning
[16:35] <perrito666> redir: hey, by any chance have you ever added a config value?
[16:36] <perrito666> in config/config.go?
[16:39] <redir> perrito666: HasDefaultSEries and PreferredSeries
[16:39] <redir> funcs not values...
[16:39] <perrito666> redir: care to evacuate some doubts for me?
[16:40] <redir> I'll try
[16:40] <perrito666> redir: ho?
[16:40] <redir> sure
[16:40] <perrito666> standup ho
[16:40] <redir> omw
[21:03] <redir> brb going to run a couple errands
[21:03] <redir> or bbiab
[21:46]  * thumper-busy rages against the code
[22:19] <perrito666> ah the typicall rage against the machine :p
[22:22]  * redir back
[22:56] <thumper-busy> if this works, fair dinkum, it'll be a miracle
[23:06] <thumper> babbageclunk: mornign
[23:06] <babbageclunk> thumper: morning - how's the releaser stuff?
[23:06] <thumper> quick hangout?
[23:06] <babbageclunk> thumper: actually it's afternoon now
[23:06] <babbageclunk> thumper: sure
[23:07] <thumper> 1:1
[23:17]  * redir wonders what a fair dinkum is