sparkiegeekhi, if I SetAnnotations with a Tag pointing at unit X and then juju destroy-unit X, it seems the annotation "survives". Is that expected?09:07
sparkiegeekmy expectation was that the annotation would die along with the unit09:08
sparkiegeekbut it seems immortal :)09:08
dimiternsparkiegeek, annotations are kept in a separate mongo collection, but it should be indeed removed when the unit is destroyed09:39
dimiternsparkiegeek, there's code for that, and if the unit's not in state anymore this is a bug worth reporting09:39
sparkiegeekdimitern: what do you mean by  "unit's not in state"?09:41
dimiternsparkiegeek, i mean can you access the unit somehow or it reports not found?09:42
dimitern(does it show up somewhere, perhaps with an error in status?)09:42
sparkiegeekdimitern: doesn't show anywhere. I'll file a bug?09:43
dimiternsparkiegeek, thanks!09:45
sparkiegeekdimitern: https://bugs.launchpad.net/juju-core/+bug/1276976 for reference09:58
_mup_Bug #1276976: Juju annotations are immortal <landscape> <juju-core:New> <https://launchpad.net/bugs/1276976>09:58
dimiternsparkiegeek, cheers :)09:58
mgzrogpeppe: (and anyone else around) weekly standup?10:05
rogpeppemgz: thanks10:05
mattywrogpeppe, would you have a moment now or later to discuss the api and juju destroy-environment?10:25
rogpeppein a call right now, but in a little bit, sure10:25
rogpeppemattyw: ^10:25
mattywrogpeppe, ok cool10:26
mattywrogpeppe, thanks10:26
rogpeppemattyw: just finished10:43
rogpeppemattyw: hangout?10:43
rogpeppemgz, dimitern, natefinch: review of these fixes to juju restore would be appreciated: https://codereview.appspot.com/6058004311:23
mgzrogpeppe: sure11:23
dimiternrogpeppe, looking11:23
rogpeppemgz, dimitern: thanks11:23
rogpeppemgz, dimitern: i just verified it live, BTW11:24
dimiternrogpeppe, reviewed11:27
dimiternrogpeppe, can you look at this? https://codereview.appspot.com/59620043/11:27
rogpeppedimitern: thanks, will do11:27
rogpeppedimitern: i don't quite understand how TestInstanceInfo ever passed11:31
rogpeppedimitern: was a branch pushed without running the tests, do you think?11:32
dimiternrogpeppe, there's some trickery with initially setting the dns name11:32
dimiternrogpeppe, that's likely yes11:32
rogpeppedimitern: ah yes, looks like i am the culprit, oops11:33
dimiternrogpeppe, ;)11:34
rogpeppedimitern: LGTM, and i'd suggest that you could submit it now, as it fixes trunk11:34
dimiternrogpeppe, I don't have rights to submit it11:37
rogpeppedimitern: ok, i could probably do that11:37
dimiternrogpeppe, wait!11:38
dimiternrogpeppe, won't that screw up my whole pipeline?11:38
rogpeppedimitern: how so?11:38
dimiternrogpeppe, well, this is the first CL, and the others depend on it11:38
dimiternrogpeppe, if you land it how can i land the rest?11:38
dimiternrogpeppe, hmm.. I suppose we can set it to Merged once you land it11:39
rogpeppedimitern: are you using bzr pipelines?11:39
dimiternrogpeppe, and ohh.. there's no bot to check the order, so it should be fine11:39
rogpeppedimitern: yeah, i don't think there should be a problem11:39
dimiternrogpeppe, yep, they are indispensable!11:39
dimiternrogpeppe, esp. with the case of goamz where the trunk rarely changes, so I can line up a whole bunch of related pipes and don't care about making it too long and having to merge trunk all the way11:40
rogpeppedimitern: yeah11:41
=== marcoceppi_ is now known as marcoceppi
adeuringcould somebody please have a look here: https://codereview.appspot.com/6063004313:49
natefinchadeuring: reviewed14:05
adeuringnatefinch: thanks14:10
natefinchadeuring: np14:12
dimiternrogpeppe, https://codereview.appspot.com/60620043 - last CL so far, if you can take a look14:25
dimiternmgz, ^^14:25
rogpeppedimitern: looking14:26
dimiternrogpeppe, mgz, also I'd appreciate one last look at the previous CLs, after I changed the live tests to be more resilient: https://codereview.appspot.com/49930045/  https://codereview.appspot.com/54690048/  https://codereview.appspot.com/54570048/14:27
dimiternrogpeppe, cheers!14:27
mgzdimitern: I'll go through the series again now14:27
dimiternmgz, thanks14:27
dimiternmgz, rogpeppe, final one, I promise :) - it's simpler than the rest https://codereview.appspot.com/54210047/16:09
mgzdimitern: :)16:13
marcoceppican anyone answer questions about juju set-env and ssh keys?16:43
dimiternmarcoceppi, there's an authorized keys plugin to manage ssh keys now16:43
marcoceppidimitern: ohohoh! link?16:43
dimiternmarcoceppi, sorry, it's not a plugin, it's a command: juju authorized-keys16:44
marcoceppidimitern: amazing, thank you16:44
dimiternmarcoceppi, http://paste.ubuntu.com/6886194/16:45
rogpeppenotify: make(chan struct{}, 1),18:37
rogpeppenatefinch: any chance you could propose utils/voyeur as its own CL? i've got a test suite i quite want to use it in18:38
natefinchrogpeppe: sure18:38
rogpeppenatefinch: oh yes, you wanted another chat about stuff, didn't you18:38
rogpeppenatefinch: have you got a moment now?18:39
natefinchrogpeppe: yeah18:39
rogpeppenatefinch: https://plus.google.com/hangouts/_/calendar/am9obi5tZWluZWxAY2Fub25pY2FsLmNvbQ.mf0d8r5pfb44m16v9b2n5i29ig?authuser=118:39
hazmatanybody know why we install bridge-utils?20:24
hazmatfor every machine20:24
natefinchhazmat: I believe to support networking between lxc containers20:25
hazmatnatefinch, seems like a bug then.. we should only be installing it when we're using lxc containers.. like we do for lxc itself20:26
natefinchhazmat: we install lxc only after you request deploying an lxc container to a machine?20:26
hazmatnatefinch, yes.. else its just waste of time/space20:27
natefinchhazmat:  true.  Then, yes, probably it's a bug, unless there's some other use that we need it for.  Not really my area of expertise.  I know there was some work being done about supporting multiple networks, no idea if that might have something to do with it.20:27
hazmathmm.. looks like the log for 2297 (committed yesterday) has it20:29
hazmatmaas-specific hack20:29
hazmatseems like that should only be installed if provider == maas20:30
natefinchhazmat: ahh yeah, the commit message says why it's always installed when lxc isn't... but still should only be done if we're on maas.20:32
rogpeppehazmat: yeah, that's my fault21:52
rogpeppehazmat: but it isn't necessarily a maas-specific hack21:52
rogpeppehatch: (except for the fact that networking between lxc containers only works under maas currently)21:53
rogpeppehazmat: ^21:53
ahasenackguys, any idea what's up with this error on 1.16.5? I'm getting it frequently now, to the point I can't bootstrap23:46
ahasenackWARNING failed to write bootstrap-verify file: cannot make S3 control bucket: A conflicting conditional operation is currently in progress against this resource. Please try again.23:46
ahasenackI trashed all jenv files i could find, then changed my control bucket name, and still can't bootstrap23:46
ahasenackthis is aws on us-west-223:46

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