thumpermenn0: https://github.com/juju/juju/pull/687301:03
thumpermenn0: that's the other half01:03
menn0thumper: looking01:37
thumpermenn0: just doing extra-info for model config now01:39
menn0thumper: done01:39
thumpermenn0: ta01:39
babbageclunkman, I was complaining about azure's SDK documentation but their web UI is really slick.01:40
menn0babbageclunk: well at least that's something :)01:40
babbageclunkclick click click01:40
thumpermenn0, babbageclunk: https://github.com/juju/juju/pull/687401:50
babbageclunkthumper: looking01:50
babbageclunkthumper: why are we making it a string rather than an object though?01:55
babbageclunkI guess they're happy feeding it through yaml unmarshalling themselves01:57
babbageclunkthumper: LGTM'd01:59
thumperbabbageclunk: yeah02:05
thumperbabbageclunk: string allows more flexibility and simpler checks02:05
thumperbabbageclunk: also what I agreed with :)02:05
babbageclunkthumper: it does make it hard for anything else to use it at the same time, but I guess it's per-model.02:06
thumperyeah, is per model02:06
thumperI think if we are going to have multiple things attempt to store data we probably want to think this through a bit more02:06
thumperand have a different place to store it02:06
thumperand different API etc02:07
babbageclunkthumper: What are they using it for out of curiosity? He doesn't say in the bug.02:07
thumpernot sure to be honest02:07
thumperconjure-up stuff02:08
thumperbabbageclunk: let my look at Tattrs02:08
thumperwhich is map[string]string02:08
thumper$ juju model-config info02:10
thumperERROR key "info" not found in {<nil> ""} model.02:10
thumperthat's not a great error message02:11
babbageclunkah well02:15
thumperbabbageclunk: looked at storing as Tattr, I don't think that is a goer...02:18
thumperthere is too much special casing around it for it to be robust02:18
thumperlike, it doesn't deal with spaces very well02:18
thumperbabbageclunk: 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 collection02:19
* redir eods02:19
babbageclunkthumper: yeah, I think you're probably right.02:38
thumperbabbageclunk: jam mentioned annotations in the review, and I think that would be the way to go02:39
thumperadd a 'juju annotation' cli02:39
thumperthat would work for models or applications or units02:39
thumperor parhaps machines02:39
thumperand have the behaviour be the same in general as the model-config02:40
thumperbabbageclunk: do you have a few minutes to chat?02:44
thumperI want to talk about the state pool02:45
babbageclunkthumper: sure. standup hangout?02:45
menn0thumper: I'm EOW (started early today). See email for the final PR for the charm URL changes that's merging right now.03:03
thumpermenn0: have a good weekend03:05
thumpermenn0: see you in just over a week03:05
thumperholy crap04:14
thumperbabbageclunk: every AllWatcher created creates a state pool for all models04:14
thumperbabbageclunk: each GUI connection creates an all watcher04:14
thumperbabbageclunk: it could just be connecting GUIs to the controller killing things04:15
thumperI'm going to refactor things so that the AllWatchers created use the single state pool04:15
thumperand looks at exposing the state pool through the facade base objects04:16
thumperthis may be a bit of surgery04:16
thumperbut shouldn't be too bad04:16
* thumper hopes04:16
thumperwell, that is tomorrow sorted04:17
jamthumper: every state object stores a list of all transactions for every model even if it only uses some of them04:22
jamthumper: state/watcher/watcher.go 'current[watchKey]int64' tracks the revno of every object seen in the transaction log04:26
jamI'd like to see them sharing that underlying watcher if possible04:26
babbageclunkthumper: ooh, that might be it.04:26
babbageclunkgah, using the resource API to update tags on azure resources requires specifying the API version per resource type. :(04:27
=== frankban|afk is now known as frankban
mupBug #1658549 opened: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>08:43
mupBug #1658549 changed: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>08:43
mupBug #1658549 opened: Security issue: jujud is not owned by a user on the system <juju-core:New> <https://launchpad.net/bugs/1658549>08:43
deanmanMorning, 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/149597809:07
perrito666deanman: ill be gone for about an hour, but I dont think that woul work10:01
perrito666deanman: have you tried jus passing the series and see what hapens? (assumin you have centos templates)10:01
deanmanperrito666, I have simple charm that has only centos7 defined in its metadata.10:08
deanmanChecking 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:09
deanmanso 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
deanmanso open to any other suggestions :-)10:10
=== benji_ is now known as benji
=== alexisb__ is now known as alexisb-afk
redirmorning juju-dev16:33
perrito666redir: morning16:35
perrito666redir: hey, by any chance have you ever added a config value?16:35
perrito666in config/config.go?16:36
redirperrito666: HasDefaultSEries and PreferredSeries16:39
redirfuncs not values...16:39
perrito666redir: care to evacuate some doubts for me?16:39
redirI'll try16:40
perrito666redir: ho?16:40
perrito666standup ho16:40
=== frankban is now known as frankban|afk
=== alexisb-afk is now known as alexisb
=== thumper is now known as thumper-busy
redirbrb going to run a couple errands21:03
rediror bbiab21:03
* thumper-busy rages against the code21:46
perrito666ah the typicall rage against the machine :p22:19
* redir back22:22
thumper-busyif this works, fair dinkum, it'll be a miracle22:56
=== thumper-busy is now known as thumper
thumperbabbageclunk: mornign23:06
babbageclunkthumper: morning - how's the releaser stuff?23:06
thumperquick hangout?23:06
babbageclunkthumper: actually it's afternoon now23:06
babbageclunkthumper: sure23:06
* redir wonders what a fair dinkum is23:17

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