/srv/irclogs.ubuntu.com/2017/04/05/#juju-dev.txt

axwhml: we can't hear you01:49
axwhml: still there?01:49
thumperhappy birthday axw :)02:12
axwthanks thumper :)02:12
axwthumper jam menn0: speaking of, I'm going out for lunch, so won't be able to make tech board today02:13
thumperack02:13
menn0axw: np02:14
babbageclunkaxw: Oh yeah, happy birthday! Hope the birthday lunch is super delish.02:31
axwbabbageclunk: thanks02:31
thumperfuck, fuck, fuckity fuck02:42
thumperdoes anyone here understand leadership as it was in 1.25?02:42
blahdeblahthumper: You really know how to brighten my day. :-)02:44
thumper:)02:44
thumperblahdeblah: don't suppose you understand it?02:44
blahdeblahI don't, but axino might. :-)02:44
thumperlooks like there should be an entry in the leases collection for it02:46
thumperbut I don't see entries there for the services I expect02:46
thumperlike...02:46
thumperthere aren't any02:46
thumperjust one for a clokc02:46
thumperclock02:46
thumperthis is crack02:47
veebersHappy Birthday axw :-)02:47
axwthanks veebers02:47
anastasiamacthumper: dunno about 1.25 but in 2., leases collection has an aaplication-leader "clock" , there is also a "clock" and a "lease" per model in that collection... these r both tagged as "singular-controller"02:49
* thumper sighs03:25
thumperFFS03:25
thumpermy old DB dump didn't have any leases03:25
thumperbut now they are there03:25
thumperwhy?...03:25
thumperI've got nothing03:25
jammenn0: btw, I'm no longer sure that deleting documents as you iterate was the cause of the problem vs the bson.M vs bson.D issue.04:41
menn0jam: could it have happened at any time?04:42
jammenn0: what I mean is that using the old "Id interface{}" meant that some % of the time documents would fail to be removed04:43
jamwhich might have been the actual cause of them not getting removed04:43
jamrather than deletions causing my iteration to be wrong.04:43
menn0jam: ok right04:44
jammenn0: so I've pushed up the changes from your review. IF you could look at it today so I can land this and then iterate on the next steps?05:15
menn0jam: looking now05:15
menn0jam: ship it on the first one05:24
menn0jam: why does the second one mention ericsnowcurrently? :)05:26
menn0jam: done05:27
jammenn0: I've seen something like that as well. Things like "this change was updated by ericsnowcurrently"07:13
jamthanks for the review07:13
=== frankban|afk is now known as frankban
menn0jam: could be the reviewboard integration as eric set that up?09:24
jammenn0: yeah, probably09:25
jammenn0: I wanted to run something by you09:25
jammenn0: I did find a few things running on a live controller09:25
jamone interesting thing09:25
jamis that it is actually reasonably to get down to 0 transactions in txns09:25
jamwhich means "pruneFactor" starts to become meaningless09:25
jamand it just causes pruning to run all the time09:25
jammenn0: yeah, I bet that's it, because it adds the reviewboard link to the pull request09:26
jammenn0: I'm thinking to add something that would make the behavior configurable / default to a minimum of 100/1000 transactions09:26
jamand maybe a maximum ignoring pruneFactor09:27
SimonKLBwill LXC containers always show hardware specs as 0 ?09:35
hoenirAfter writing a provider and a api client from scratch it's amazing how well juju is structured in interfaces.09:55
hoenirAnd how easy is to embed and compose the functionalities. I really didn't need to look up on some docs or anything like that. I just watched and readed the code... on some places it laks comments but overall a good experience.09:56
hoenirGood job done guys !09:56
hoenirWell done..09:57
hoenir*09:57
hoenirDid anyone noticed this when writing a provideR?09:57
Dmitrii-ShHi. Question: https://jujucharms.com/docs/2.1/reference-charm-hooks#leader-elected <- this doc says "If the election process is done internally to the service, other code should be used to signal the leader to Juju.". However, I don't see any hook tools to assert leadership http://paste.ubuntu.com/24319908/ So, as far as I understand, there is no manual way12:08
Dmitrii-Shto designate a leader and the doc is wrong. Does anyone know if it is supposed to be that way and if this has not been implemented for a reason?12:08
Dmitrii-Sh"Authors can use this hook to take action if their protocols for leadership, consensus, raft, or quorum require one unit to assert leadership." - this doesn't make much sense as well as if you have a network partition you may actually have multiple leaders in separated clusters. If Juju has access to both clusters, for example, it may get conflicting12:14
Dmitrii-Shrequests for leadership from that 'other code that signals internal notion of leadership to juju'12:14
=== akhavr1 is now known as akhavr
admcleodsinzui: fyi, magpie now does MTU15:34
sinzuiadmcleod: interesting. I will look soon15:34
cmarsmorning folks, could i get a review of https://github.com/juju/juju/pull/7195 please?15:38
=== frankban is now known as frankban|afk
sinzuibabbageclunk: did you see bug 168004621:04
mupBug #1680046: Bootstrap failed on maas 1.9 because invalid character '<' looking for beginning of value <bootstrap> <ci> <maas-provider> <regression> <juju:Triaged by 2-xtian> <https://launchpad.net/bugs/1680046>21:04
babbageclunksinzui: no I hadn't - looking now21:11
babbageclunksinzui: is the test running against maas 1.9.5?21:13
* sinzui looks21:15
sinzuibabbageclunk: no, 1.9.4+bzr4592-0ubuntu1~trusty1. It is scheduled to be upgraded to 1.9.5+bzr4599-0ubuntu1~14.04.1 this weekend21:17
sinzuibabbageclunk: I can make it upgrade right now21:18
babbageclunksinzui: hmm, that might be it then. I ran my smoke test against 1.9.5.21:18
babbageclunksinzui: Is breaking juju for 1.9.4  a problem?21:19
babbageclunksinzui: (I can see that it might be.)21:19
sinzuibabbageclunk: Awkward answer. the answe no we should not, but we do love the maas team when they say the only fix for a bug is to upgrade to the latest maas21:20
babbageclunksinzui: yeah, sure - it would be pretty annoying for a user who has a working 1.9.4 maas21:21
babbageclunksinzui: I'll roll back to my 1.9.4 snapshot and try to work out why it's broken against it.21:21
sinzuibabbageclunk: 1.9.5 is only 6 days old. so few will have it. Like us we don't get updated every day. BUT....21:22
sinzuiif you don't fix the issue soon, my maas will go to 1.9.5 anyway21:22
babbageclunksinzui: yay easy fix!21:22
babbageclunksinzui: j/k21:22
sinzuibabbageclunk: I am half serious myself. I think the issue will fix itself if I don't interviene with the servers update schedule21:23
babbageclunksinzui: well, I guess we can discuss more in about 5 mins21:25
anastasiamacbabbageclunk: sinzui: it's on the list \o/21:25
anastasiamac(minutes)*21:25
cmarsmorning folks, can i get a review of https://github.com/juju/description/pull/11 please?21:55
cmarshttps://github.com/juju/juju/pull/7202 will soon follow21:56
babbageclunksinzui: weirdly, I can bootstrap on 1.9.4, although it requires a lot of retries for the bootstrap to successfully ssh to the deployed controller machine22:05
babbageclunksinzui: Could I get access to the maas where this test was failing?23:03
babbageclunksinzui: no worries, I worked it out - bootstrapping against it now.23:16
sinzuibabbageclunk: you may already have,. did I send you the snippetsmfrom cloud-city to get into munna?23:16
babbageclunksinzui: I eventually remembered to look in there.23:17
sinzuibabbageclunk: are you in?23:17
babbageclunksinzui: yup yup23:18
babbageclunksinzui: ok, I see why my smoke test didn't find this now.23:25

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