/srv/irclogs.ubuntu.com/2017/09/20/#juju-dev.txt

* babbageclunk goes for a run02:04
thumperotp04:02
axwthumper: robot again04:17
thumperit normally settles down04:18
thumpernot sure why it does this04:18
thumperaxw, jam: I'm going04:24
babbageclunkaxw: ping?04:47
axwbabbageclunk: pong05:05
babbageclunkaxw: hey - just tried an lxc-lxd upgrade with an IS chap - it returned an error about unknown config key lxd.hook.mount05:07
babbageclunkaxw: how do you think I should handle that?05:07
babbageclunkI haven't been able to find anything on lxd mount hooks on the web.05:08
axwbabbageclunk: lxd.hook.mount? or lxc.hook.mount?05:08
babbageclunkaxw: oops, the latter05:09
axwbabbageclunk: can you check what the hook is? I haven't come across that one in my testing05:09
babbageclunkFrom here https://linuxcontainers.org/lxc/manpages/man5/lxc.container.conf.5.html it's a script that runs after mounting is done but before pivot_root, whatever that is.05:10
axwbabbageclunk: yeah, this is on bootstack? I think I found a charm that's setting that in lxc config05:11
babbageclunkaxw: yup - I think they're using it to set static routes in the containers.05:12
=== frankban|afk is now known as frankban
axwrogpeppe1: sorry, my PR is a bit messy. I would appreciate your thoughts on the general approach10:14
axwit started out as a targeted PR, then morphed along the way10:14
rogpeppe1axw: i just looked at it in passing tbh.10:14
rogpeppe1axw: i don't really understand it, i'm afraid, or quite what the eventual motivation is10:14
axwrogpeppe1: ok. motivations are in the description, which I take I didn't express well :)10:15
axwrogpeppe1: gotta eat dinner and then go out, will try again tomorrow10:16
rogpeppe1axw: ok, i think i understand the motivation a bit more now10:18
rogpeppe1axw: but... if we moved to something like Raft, would we really keep all the Op crap?10:18
axwrogpeppe1: not the underlying ones, no. but the high level operations, yes - they're the ones that would map to log entries10:19
axwrogpeppe1: i.e. DestroyUnitOperation et al. would be made serializable, and would be a log entry10:19
axwthat's my vague idea atm anyway10:19
rogpeppe1axw: ok, that seems reasonable10:20
axwthere'd need to be another sort of entry for ops in a transaction10:20
rogpeppe1axw: but for mongo, i'm not entirely convinced that arbitrary operations can be composed10:20
axwrogpeppe1: no, they can't, but we do it already... :)10:21
rogpeppe1axw: this will only make it worse, right?10:21
rogpeppe1axw: because people will *think* they can compose a NewService op with a DestroyUnit op, for example10:22
axwrogpeppe1: hmm, I'm not sure. I guess it would encourage it for future operations. for existing ones, I don't think they're going to be any worse10:22
axwI mean State.AddApplication adds both application and unit ops; if we compose them outside, yo uget the same result10:22
rogpeppe1axw: anyway, i'd like to see a more focused version of the PR so that it's clear what changes are as a result of the introduction of ModelTransaction10:23
axwrogpeppe1: ok, will try and do that tomorrow10:23
rogpeppe1axw: yup, that's true for AddApplication specifically, and it's tested that way10:23
axwfair point. bounded badness10:24
rogpeppe1axw: but i'm concerned that if we provide arbitrary composition, that things will fail easily10:24
rogpeppe1axw: also, it's very easy to blow the txn doc size limit10:24
rogpeppe1axw: (we already do in some cases)10:24
* axw out10:27
=== frankban is now known as frankban|afk

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