| thumper | menn0: https://github.com/juju/juju/pull/6873 | 01:03 |
|---|---|---|
| thumper | menn0: that's the other half | 01:03 |
| menn0 | thumper: looking | 01:37 |
| thumper | menn0: just doing extra-info for model config now | 01:39 |
| menn0 | thumper: done | 01:39 |
| thumper | menn0: ta | 01:39 |
| 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:40 |
| thumper | menn0, babbageclunk: https://github.com/juju/juju/pull/6874 | 01:50 |
| babbageclunk | thumper: looking | 01:50 |
| babbageclunk | thumper: why are we making it a string rather than an object though? | 01:55 |
| babbageclunk | I guess they're happy feeding it through yaml unmarshalling themselves | 01:57 |
| babbageclunk | thumper: LGTM'd | 01:59 |
| thumper | babbageclunk: yeah | 02:05 |
| thumper | babbageclunk: string allows more flexibility and simpler checks | 02:05 |
| thumper | babbageclunk: also what I agreed with :) | 02:05 |
| 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:06 |
| 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:07 |
| 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:08 |
| thumper | ew... | 02:10 |
| thumper | $ juju model-config info | 02:10 |
| thumper | ERROR key "info" not found in {<nil> ""} model. | 02:10 |
| thumper | that's not a great error message | 02:11 |
| babbageclunk | ah well | 02:15 |
| 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:18 |
| 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:19 | |
| babbageclunk | thumper: yeah, I think you're probably right. | 02:38 |
| 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:39 |
| thumper | and have the behaviour be the same in general as the model-config | 02:40 |
| thumper | ish | 02:40 |
| thumper | babbageclunk: do you have a few minutes to chat? | 02:44 |
| thumper | I want to talk about the state pool | 02:45 |
| babbageclunk | thumper: sure. standup hangout? | 02:45 |
| thumper | sure | 02:45 |
| 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:03 |
| thumper | ack | 03:05 |
| thumper | menn0: have a good weekend | 03:05 |
| thumper | menn0: see you in just over a week | 03:05 |
| 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:14 |
| 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:15 |
| 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:16 | |
| thumper | well, that is tomorrow sorted | 04:17 |
| jam | thumper: every state object stores a list of all transactions for every model even if it only uses some of them | 04:22 |
| 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:26 |
| babbageclunk | gah, 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 | ||
| 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> | 08:43 |
| 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 | 09:07 |
| perrito666 | morning | 10:00 |
| 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:01 |
| deanman | perrito666, I have simple charm that has only centos7 defined in its metadata. | 10:08 |
| 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:09 |
| 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 :-) | 10:10 |
| === benji_ is now known as benji | ||
| === alexisb__ is now known as alexisb-afk | ||
| redir | morning juju-dev | 16:33 |
| perrito666 | redir: morning | 16:35 |
| perrito666 | redir: hey, by any chance have you ever added a config value? | 16:35 |
| perrito666 | in config/config.go? | 16:36 |
| redir | perrito666: HasDefaultSEries and PreferredSeries | 16:39 |
| redir | funcs not values... | 16:39 |
| perrito666 | redir: care to evacuate some doubts for me? | 16:39 |
| redir | I'll try | 16:40 |
| perrito666 | redir: ho? | 16:40 |
| redir | sure | 16:40 |
| perrito666 | standup ho | 16:40 |
| redir | omw | 16:40 |
| === frankban is now known as frankban|afk | ||
| === alexisb-afk is now known as alexisb | ||
| === thumper is now known as thumper-busy | ||
| redir | brb going to run a couple errands | 21:03 |
| redir | or bbiab | 21:03 |
| * thumper-busy rages against the code | 21:46 | |
| perrito666 | ah the typicall rage against the machine :p | 22:19 |
| * redir back | 22:22 | |
| thumper-busy | if this works, fair dinkum, it'll be a miracle | 22:56 |
| === thumper-busy is now known as thumper | ||
| 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:06 |
| thumper | 1:1 | 23:07 |
| * redir wonders what a fair dinkum is | 23:17 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!