/srv/irclogs.ubuntu.com/2015/04/06/#juju-dev.txt

=== kadams54 is now known as kadams54-away
dimiternmorning all06:26
dimitern(just in case it's not just me today :)06:27
natefinchdimitern: morning.  I'm here, but only just for a minute or two before bed06:27
dimiternnatefinch, hey, btw do you need another review on that ensure availability API branch?06:28
natefinchdimitern: yes please.  Just pushed a bunch of changes.  There's a few comments from Andrew that I just can't get to before I fall over tonight, but mostly I think it's fine.06:28
natefinchhttp://reviews.vapour.ws/r/1299/06:29
dimiternnatefinch, ok, np - looking06:29
natefinchway past bedtime for me.  Good night06:29
dimiterngood night natefinch !06:30
dimiternjam, I guess you're not here today06:35
jamdimitern: I'm going to be a bit late for our 1:1, I have to take my son to a friends house. I'll ping you when I get back.06:42
dimiternjam, sure, np06:43
* dimitern steps out for ~30m09:18
perrito666morning09:22
jamdimitern: sorry I forgot to ping you back. Do you have any pressing items to discuss? How are you feeling about prep for next week?09:33
* dimitern is back09:54
dimiternjam, hey09:54
dimiternjam, perhaps we can have a quick chat now if you're available?09:55
dimiternmorning perrito66609:55
dimiternperrito666, aren't you off today?09:59
perrito666dimitern: nope, I have been on vacations for two weeks10:05
perrito666but everyone else seems to be off10:05
dimiternperrito666, yeah :)10:06
=== lp|away is now known as lazyPower
=== ericsnow is now known as ericsnow_afk
=== ericsnow_afk is now known as ericsnow
natefinchwwitzel3: standup?14:33
natefinchdimitern: about utils_RandomPassword ... I intentionally use an unusual format w/ the underscore, since it's replacing utils.RandomPassword... I find it makes it easy to tell what function is being replaced by this variable when you're just looking at the code.  otherwise I could just call it randomPassword()... but I like having an easy way to tell what it's replacing without having to go to definition on it.14:55
dimiternnatefinch, well what's wrong with utilsRandomPassword ?:)14:56
natefinchdimitern: it's still not as obvious that the first part is part of the package name... the underscore really separates it out, and it still looks a lot like the original namespaced form: var utils_RandomPassword = utils.RandomPassword14:57
natefinchdimitern: but if people really don't like it, it's not a huge deal.14:58
dimiternnatefinch, I still rather not have underscores in code, but it that's the only thing remaining and others are ok with it, I'll accept that14:59
dimiterns/but it/but if/15:00
natefinchdimitern: heh... there's a ton of other stuff remaining :)15:00
natefinchdimitern:  I'm not sure what you meant when you said there was inconsistent indenting with the txn.Ops.  I just copied the other three functions that were formatted exactly like mine (as far as I can tell).15:23
dimiternnatefinch, in a call, will get back to you15:31
dimiternnatefinch, ah, so I though my comment there made it clear,15:43
natefinchdimitern: not to me, at least :)15:44
dimiternnatefinch, I think you either use []txn.Op{{\nC:XYZ,\n,...,\n}, {\n,C:UVT,\n...,\n}} or []txn.Op{\n{\nC:XYZ,\n...,\n},\n{\nC:UVT,\n...,\n}\n,}\n15:45
dimiternnatefinch, does it make more sense?15:46
natefinchdimitern: I think I'm doing that first one15:47
natefinchor are you asking me to un-indent the stuff inside?15:48
dimiternnatefinch, sorry, let me have a look at the review again15:48
dimiternnatefinch, ugh, I've seen it now - it's fine, so please ignore my comment and I'll drop the issue15:50
natefinchok :)  good, I thought I was going crazy or something, which is entirely possible running on 4 hours sleep :)15:50
dimitern:)15:50
* dimitern reached EOD15:52
cheryljericsnow: you around?15:59
ericsnowcherylj: yep15:59
cheryljericsnow: I'm working on bug 1439447 that we talked about last Friday and I need to get some advice...  natefinch had an idea that we could specify a new config option to indicate whether or not to use the proxy when contacting the bootstrap node16:00
mupBug #1439447: tools download in cloud-init should not go through http[s]_proxy <cloud-installer> <landscape> <juju-core:In Progress by cherylj> <juju-core 1.23:In Progress by cherylj> <https://launchpad.net/bugs/1439447>16:00
cheryljericsnow: and I've been digging through things for a while and I'm wondering if this flag should be part of proxy.Settings since it does relate to how we use the proxy...16:01
cheryljproxy.Settings is part of juju/utils16:02
cheryljand it would break backwards compatibility, but we branched utils because of the license changes I made anyway...16:02
ericsnowcherylj: I thought about that too16:03
ericsnowcherylj: no_proxy already covers this sort of16:03
ericsnowcherylj: if it supported IP ranges it might be good enough16:04
cheryljericsnow: yeah, and another thought I had was to just add the state server IPs to the no_proxy list when this new flag is set16:04
ericsnowcherylj: yep16:04
cheryljericsnow: then if we add new state servers, or remove them, then we need to prune the list16:04
cheryljericsnow: and if the flag is unset, do we remove them?  What if the user had manually added them and we just don't know?16:05
ericsnowcherylj: proxy.Settings could also grow a new field like "ProxyPrivateAddresses"16:05
cheryljericsnow: yeah, that was my thought16:06
ericsnowwith that setting we wouldn't need to add addresses to no_proxy16:06
cheryljericsnow: are there other places (other than this curl command) that we would need to change to check this?  I'm not sure if there are other places communication is happening with the state servers through the proxy16:06
ericsnowcherylj: looks like several packages use proxy.Settings: utils/apt/apt.go and a few places in juju core16:08
cheryljericsnow: okay, I'll take a look at those places16:11
cheryljericsnow: so you think adding the new field to proxy.Settings sounds good?  I think it will be the clearest way to add this config option.16:12
ericsnowcherylj: ideally the proxy.Settings methods would handle that new field properly by adding the appropriate IP address16:17
ericsnowcherylj: however, without IP range support that probably isn't doable16:17
cheryljcherylj: I had also thought that in environs/config/config.go where we return a new proxy.Settings, we could check there if this flag was set, and add the IP addresses to the no_proxy list before returning16:18
ericsnowcherylj: definitely16:19
cheryljI'm not sure if that list is used in any permanent way16:19
cheryljor if ProxySettings() is called every time we need the proxy info16:19
ericsnowcherylj: it would be nice, though, if we had an implicit solution; that way every use of proxy.Settings doesn't need to handle the new field16:20
cheryljyeah16:21
ericsnowcherylj: for now we could just add the IP address of each state server on a private IP address to Settings.NoProxy16:28
ericsnowcherylj: or prepend "HTTP_PROXY= HTTPS_PROXY= " to that one curl command if the IP in the curl command is private16:29
ericsnowcherylj: in the interest of solving the 1.23 bug16:30
ericsnowcherylj: then open a bug against 1.24 for the better solution16:30
cheryljericsnow: yeah, I wasn't sure what the timeframe looked like for this bug.16:31
ericsnowcherylj: we could probably argue for deferring it to 1.23.1, but it would be nice to have a fix for 1.23.0. :)16:32
cheryljericsnow: it's easy to determine if we're not the bootstrap node, and we could prepend the "HTTP_PROXY= ..." in that case.  Do I need to check if the IP is also private?16:32
ericsnowcherylj: it depends on whether or not cloudinit for a new HA instance pulls from another state server16:34
ericsnowcherylj: I'm guessing we can simply key off of w.mcfg.Bootstrap16:36
cheryljericsnow: that's what I was thinking16:36
ericsnowcherylj: we would need to check whether or not that is set for HA instances16:36
=== jog_ is now known as jog
ericsnownatefinch: do you know if MachineConfig.Bootstrap (as used by cloudinit) is set for HA instances?16:41
natefinchericsnow: I do not.   I'd have to go look it up16:42
ericsnownatefinch: no worries16:42
natefinchgotta run for a bit, back in an hour-ish.16:44
=== natefinch is now known as natefinch-afk
cheryljericsnow: I don't think that it is set.  Just did a little test with ensure-availability and it wasn't...16:50
cheryljI gotta take a quick lunch break.  brb16:50
ericsnowcherylj: k16:50
=== natefinch-afk is now known as natefinch
perrito666brb18:11
natefinch>     >     >> >>> On Apr 5, 2015, at 11:28 AM, Someone with an annoying plaintext email program replied....18:12
wwitzel3lol18:13
cheryljericsnow: do you know how can I get the script output like what's included in that bug to see the actual commands being run?18:52
ericsnowcherylj: look for cloudinit.log on the new instance18:53
ericsnowcherylj: I don't recall exactly where the cloudinit script is written18:54
cheryljok18:54
* perrito666 had to go replace his fron't door lock after he had to dismantle the door to get out of the house19:20
perrito666mondays...19:21
mupBug #1433577 changed: Vivid unit tests need to pass <ci> <test-failure> <vivid> <juju-core:Fix Released by ericsnowcurrently> <juju-core 1.23:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1433577>19:22
mupBug #1435860 changed: certSuite.TestNewDefaultServer failure <ci> <regression> <test-failure> <unit-tests> <juju-core:Fix Released by ericsnowcurrently> <juju-core 1.22:Fix Released by ericsnowcurrently> <juju-core 1.23:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1435860>19:22
mupBug #1435974 changed: Copyright information is not available for some files <juju-core:Fix Released by cherylj> <juju-core 1.22:Fix Released by cherylj> <juju-core 1.23:Fix Released by cherylj> <https://launchpad.net/bugs/1435974>19:22
mupBug #1437040 changed: unit test failure: TestNewDefaultServer.N40_github_com_juju_juju_cert_test.certSuite <ci> <unit-tests> <juju-core:Fix Released by ericsnowcurrently>19:22
mup<juju-core 1.22:Fix Released by ericsnowcurrently> <juju-core 1.23:Fix Released by ericsnowcurrently> <https://launchpad.net/bugs/1437040>19:22
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== _thumper_ is now known as thumper
natefinchthumper: got a minute?20:44
thumpernatefinch: otp with alexisb20:44
natefinchthumper: k20:44
mupBug #1439880 changed: Container's interfaces are all on private networks instead of host's eth0 network <lxc> <maas-provider> <network> <oil> <juju-core:Triaged by dimitern> <https://launchpad.net/bugs/1439880>21:40
thumpernatefinch: ok, here now21:47
thumpernatefinch: do I need coffee?21:47
* thumper goe to get breakfast21:51
axw_katco: Internet just dropped out, I guess because of the weather. Just FYI, in case I don't turn up at the standup23:05
=== axw_ is now known as axw
=== kadams54 is now known as kadams54-away
katcoaxw: k ty23:16
katcoaxw: since it's just us, we can probably just coordinate over irc. i have a few questions for you, but they can wait until later (after my dinner)23:19
katcoaxw: i'll just catch up with you over irc after dinner23:22
axwkatco: :/  sorry23:29
axwkatco: we can try again if you like. otherwise may be best to email me23:30
thumperaxw, katco: seems like your team is as small as mine this week :)23:31
axwcosy23:32
thumperaxw: makes for quick standup calls :)23:37
* thumper relocates for better design thinking space23:38
katcothumper: lol yep23:59
katcoaxw: i'll send you an email23:59

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