rick_h_cory_fu: ty for the bundle v4 support MP's. I've asked Makyo to peek and help make sure everything looks cool and we've no suggestions from our end.16:45
cory_fuAwesome, thanks.  We need these changes to get the big data bundles passing the test runner.  I tried to make the minimal changes to include support, so definitely LMK if it could be addressed better, though I would like to see a fix out as soon as possible so we can see our bundles passing.  :)16:46
rick_h_cory_fu: understand, Makyo will look today and so can help provide feedback pretty quickly16:48
Makyocory_fu: +116:51
cory_fuExcellent.  Thanks16:51
cory_fuI do wish there were a better way to distinguish the bundle formats.  Adding a "version" or "format_version" or such field would be really n ice16:52
Makyocory_fu: agreed :/ We went through similar hoops in bundlelib.16:53
cory_fuAdding a top-level key that is required for v4+ and defaults to v3 seems like it would be reasonable to me, but that's just off the top of my head16:54
rick_h_cory_fu: we thought about it, the issue is that we're asking the user to tell us and it's a chance for a user to get it wrong16:55
rick_h_cory_fu: so it's awesome from a lib developer standpoint, but a pita for a bundle writer/user standpoint as it's one more thing to break16:55
cory_fuThere is that.16:56
=== urulama is now known as urulama__
rick_h_cory_fu: so we take the burden in the hopes the devs appreciate it and buy us beers one day :)16:56
cory_furick_h_: OTOH, there are clearly constraints that the user already has to conform, and we provide tools to help them do so (proof).  Allowing the user to be explicit about the version they *intend* seems like it would actually lead to fewer confusing errors.16:58
cory_fuI guess the current heuristics are fairly reliable due to the drastic change in the format, though16:59
rick_h_cory_fu: yea, I can be convinced we're wrong for sure. Honestly, we've not had the uptake in folks using it since we released it because they didn't "HAVE TO" so we don't have good data either way17:00
rick_h_cory_fu: but just fyi'ing on the picture there as it was thought about for sure17:00
cory_fuWell, I'm sure the lack of support in the rest of the tooling (proof, deployer, bundletester, etc) didn't help uptake, and all of that is now changing17:01
rick_h_cory_fu: <3417:01
rick_h_err <317:01
cory_fuI read that as "less than rule 34" and was a little concerned.17:01
rick_h_my typing is horrible today, must have gotten my fingers more bit up fishing than I thought this weekend :P17:02
rick_h_those bass tear up your thumbs when you lip them hah!17:03
cory_fuha, I'm sure17:03
sinzuihi bac, jcsackett, Do either of you have time to review https://code.launchpad.net/~sinzui/juju-quickstart/all-regions/+merge/26896118:54
rick_h_Makyo: ^ ?19:23
MakyoI'll take a look rick_h_ sinzui19:23
rick_h_Makyo: ty much!19:23
MakyoI'm glad we've finally settled where Amsterdam is.19:24
rick_h_Makyo: can you land that if that's cool please?19:28
Makyorick_h_: sure19:29
Makyorick_h_: Embarrassing, but I only ever used lbox to merge code on launchpad.  Do I need to do anything other than mark it as merged?19:31
rick_h_Makyo: you have to run the lbox submit thing. 19:32
rick_h_Makyo: basically if you can run tests then you bzr branch, merge it into trunk, push to  trunk. 19:32
Makyorick_h_: ack. Will take a bit to get that updated.19:32
rick_h_Makyo: understand19:32
Makyorick_h_: sinzui I did the thing! \o/  20:12
MakyoBeen forever since I worked with bzr, sorry for the delay.20:12
sinzuithank you Makyo 20:14

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