anastasiamacbabbageclunk: PTAL https://github.com/juju/juju/pull/11131 as discussed00:13
babbageclunkT'ing an L00:13
stubskay: If you remove all the units, but don't remove the application, then the Juju leadership state hangs around. When you add new units to the application, they have no way of telling if they need to wait for what the leadership settings tell them is the master, or if the data is stale. Well, they can now thanks to the goal state feature, but that didn't exist when that code was written.00:33
stubIt might be a mojo (or juju-deployer) bug, where it assumes removing all the units is equivalent to removing the application. It has come up a few times with mojo.00:35
anastasiamacstub: ah! thanks for explanation - that makes sense :D00:45
babbageclunkanastasiamac: approved02:42
anastasiamacbabbageclunk: \o/04:23
stickupkidanyone know if these files are still generated?10:21
stickupkidand https://github.com/juju/juju/blob/2.7/service/windows/zpassword_windows.go10:21
stickupkidboth are flagging up with import errors in the linter, but I'm trying to work out how they're generated10:22
stickupkidturns out the new goimport in latest golang just finds they're not correct10:49
achilleasastickupkid: were the linter errors in your PR caused by the lint_go.sh bits that your last commit tweaks?11:07
stickupkidonly in tests11:08
stickupkidvery strange11:08
nammn_destickupkid achilleasa: just for my understanding. The settings collection is responsible to safe config setting of a model?  E.g. changes to config mysql are reflected there?12:05
skaystub: I've removed all of the applications are re-run my spec but it still can't tell it's the leader. Is there a bug about that? Meanwhile, I guess someone could delete and recreate the model for me?13:23
skaywhen I run remove-application on postgresql it's not removing anything. I have to call it with --force and --no-wait13:58
skayoh wait, it just takes freaking forever.13:58
rick_hskay:  yes, it could take forever depending on the underlying cloud, state of things, etc14:03
rick_hskay:  --force will tell it to try to do things right, but if you have to start cheating in potentially bad ways"14:03
skayrick_h: might that explain why I got the problem with the messed up leader state?14:05
skayrick_h: in the scrollback I asked about my situation. I've been testing a mojo spec and when I want to rerun it I remove all the applications. I thought postgresql was stuck somehow because of how long it was taking14:06
skayrick_h: so I've been using --force --no-wait on it and other things14:06
nammn_demanadart: was doublechecking with achilleasa. It seems that the method `AllSpaces()` seems to use `AllAddresses` which uses the addresses collection, which in aws, do not contain the public ip's. Therefore I would change that to use `machine.Addresses()`14:38
nammn_dedoes that make sense?14:39
manadartnammn_de: The public IPs are shadow addresses. Those are not reasoned about in terms of spaces, so it shouldn't matter for this purpose.14:47
nammn_demanadart: quick ho?14:48
manadartnammn_de: I'm there.14:49
skayok, the answer to my problem was that there were still instances running even though juju status did not show them14:51
skayI deleted hte instances and the postgresql unit became leader14:51
achilleasanammn_de: that is ipaddresses collection14:51
nammn_derick_h: I updated the description. Talked to manadart about why a machine-count=0 can happen. That's because I happen to bind after a deploy.15:17
nammn_deAdditionally, it seems that we do not care about the public part of the addresses, therefore we just skip that for space information. I updated the QA section in the PR and added a comment. Seems like no change to the code regarding this is needed.15:17
achilleasahml: what do you think about adding a dirty check to the in-memory state map in the uniter's context? We can set it each time we write/delete something and clear when flushing. We could avoid needless roundtrips to the controller if the state has not been mutated15:39
skayI have a puzzling situation with the autocert charm. I'm using mojo to deploy it, and the charm is not getting configured with values in the services file20:05
skaymojo uses juju-deployer20:05
skayI can assign the values by hand using juju config20:06

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