[01:18] <_mup_> ensemble/new-hook-semantics-1-departed-hook r182 committed by jim.baker@canonical.com [01:18] <_mup_> Addressed review points [01:20] <_mup_> ensemble/new-hook-semantics-1-departed-hook r183 committed by jim.baker@canonical.com [01:20] <_mup_> Merged trunk [01:25] <_mup_> ensemble/service-unit-state-upgrade-support r194 committed by kapil.thangavelu@canonical.com [01:25] <_mup_> merge changes from review [01:48] <_mup_> ensemble/trunk r188 committed by kapil.thangavelu@canonical.com [01:48] <_mup_> merge service-unit-state-upgrade-support [r=niemeyer][f=750193] [01:48] <_mup_> Enable state api methods on units and services to support upgrade, via [01:48] <_mup_> an setting, getting, and watching an upgrade flag on unit states. [02:32] <_mup_> ensemble/trunk-merge r183 committed by kapil.thangavelu@canonical.com [02:32] <_mup_> merge trunk [04:01] <_mup_> ensemble/new-hook-semantics-2-joined-hook r183 committed by jim.baker@canonical.com [04:01] <_mup_> Merged new-hook-semantics-1-departed-hook and resolved conflicts [04:15] <_mup_> ensemble/new-hook-semantics-1-departed-hook r184 committed by jim.baker@canonical.com [04:15] <_mup_> PEP8 [04:15] <_mup_> ensemble/new-hook-semantics-2-joined-hook r184 committed by jim.baker@canonical.com [04:15] <_mup_> Addressed review points [04:16] <_mup_> ensemble/new-hook-semantics-2-joined-hook r185 committed by jim.baker@canonical.com [04:16] <_mup_> Merged new-hook-semantics-1-departed-hook and resolved conflicts [04:23] <_mup_> ensemble/new-hook-semantics-2-joined-hook r186 committed by jim.baker@canonical.com [04:23] <_mup_> Removed unnecessary parens [04:26] <_mup_> ensemble/new-hook-semantics-3-remove-change-hook r184 committed by jim.baker@canonical.com [04:26] <_mup_> Merged new-hook-semantics-2-joined-hook and resolved conflicts [04:31] <_mup_> ensemble/new-hook-semantics-1-departed-hook r185 committed by jim.baker@canonical.com [04:31] <_mup_> Merged trunk [04:40] <_mup_> ensemble/formula-upgrade-cli r197 committed by kapil.thangavelu@canonical.com [04:40] <_mup_> update to latest parse formula id signature. [05:01] <_mup_> ensemble/trunk r189 committed by jim.baker@canonical.com [05:01] <_mup_> merge new-hook-semantics-1-departed-hook [r=niemeyer,hazmat][f=740569] [05:01] <_mup_> Upon a departed event, invokes -relation-changed hook, then [05:01] <_mup_> -relation-departed hook. A subsequent branch merge will remove [05:01] <_mup_> the invocation of the -relation-changed hook. [05:02] <_mup_> ensemble/new-hook-semantics-2-joined-hook r187 committed by jim.baker@canonical.com [05:02] <_mup_> Merged trunk [05:06] <_mup_> ensemble/trunk r190 committed by jim.baker@canonical.com [05:06] <_mup_> merge new-hook-semantics-2-joined-hook [r=niemeyer][f=744723] [05:06] <_mup_> Upon a joined event, invokes -relation-joined hook, then [05:06] <_mup_> -relation-changed hook. [05:07] <_mup_> ensemble/new-hook-semantics-3-remove-change-hook r185 committed by jim.baker@canonical.com [05:07] <_mup_> Merged trunk [05:12] <_mup_> ensemble/trunk r191 committed by jim.baker@canonical.com [05:12] <_mup_> merge new-hook-semantics-3-remove-change-hook [r=niemeyer][f=744724] [05:12] <_mup_> Upon a departed event, no longer invoke -relation-changed hook. [05:14] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r187 committed by jim.baker@canonical.com [05:14] <_mup_> Merged trunk and resolved conflicts [05:49] ensemble natty image if anyone is curious, ami-d06b96b9 [05:55] jimbaker, is that fix for the formulas on trunk checked-in? [06:45] <_mup_> ensemble/formula-upgrade-cli r198 committed by kapil.thangavelu@canonical.com [06:45] <_mup_> add a dry run upgrade option, and a separate exception for newer formulas not found. [06:59] <_mup_> ensemble/formula-upgrades-spec r185 committed by kapil.thangavelu@canonical.com [06:59] <_mup_> remove per unit relation upgrade discussion and hook-api into new futures section [07:02] <_mup_> ensemble/trunk r192 committed by kapil.thangavelu@canonical.com [07:02] <_mup_> merge formula-upgrades-spec [r=bcsaller,jimbaker,niemeyer][f=731519] [07:02] <_mup_> Spec documentation of an upgrade-formula command, and accompanying [07:02] <_mup_> formula support. [07:07] <_mup_> Bug #754318 was filed: add register cleanup utility to cli < https://launchpad.net/bugs/754318 > [13:26] hazmat: Hi what's that ensemble natty ami do ? [13:30] s/what's/what does/ [13:58] kim0: It's the base image in which formulas run [13:59] niemeyer: that is used for "bootstrp" right [13:59] kim0: It's used to run everything [13:59] oh I see === deryck is now known as deryck[lunch] === niemeyer is now known as niemeyer_lunch === niemeyer_lunch is now known as niemeyer [17:00] bcsaller: Are you in Canonical's IRC server? [17:05] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r188 committed by jim.baker@canonical.com [17:05] <_mup_> Merged trunk [17:06] gustavo: not currently [17:06] bcsaller: Can you please join it [17:13] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r189 committed by jim.baker@canonical.com [17:13] <_mup_> Addressed review points [17:16] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r190 committed by jim.baker@canonical.com [17:16] <_mup_> Removed hook stubs in light of optional hook support as r185 of trunk [17:20] <_mup_> ensemble/trunk r193 committed by kapil.thangavelu@canonical.com [17:20] <_mup_> merge formula-upgrade-cli [r=niemeyer][f=750304] [17:20] <_mup_> A new ensemble subcommand for upgrading formulas, via local [17:20] <_mup_> repository search, formula publishing, and marking service [17:20] <_mup_> units for upgrade. === deryck[lunch] is now known as deryck [17:40] <_mup_> ensemble/bashified-wordpress-mysql-examples r178 committed by jim.baker@canonical.com [17:40] <_mup_> Merged trunk and resolved conflicts [18:02] <_mup_> ensemble/bashified-wordpress-mysql-examples r179 committed by jim.baker@canonical.com [18:02] <_mup_> Updated examples to use new hook semantics [18:05] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r191 committed by jim.baker@canonical.com [18:05] <_mup_> Merged trunk [18:25] hazmat, r193 broke trunk for me for both natty and meerkat. when i checked out trunk to r192, it runs on both natty and meerkat for me. (fwiw, i also tried rebooting) [18:30] hazmat, this also occurs w/ a fresh checkout of trunk [18:32] jimbaker, broke how? [18:32] hazmat, unit test suite [18:33] jimbaker, a particular test? [18:33] first error: test_file_storage_returns_same_storage ... [ERROR] - then cascades from there [18:34] jimbaker, thanks.. checking [18:37] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r192 committed by jim.baker@canonical.com [18:37] <_mup_> Removed references to ENSEMBLE_CHANGE from docs [18:41] i just found out that the dc local government also shutsdown on a federal shutdown, 10% local taxes go straight to the federal treasury to be allocated back.. or not. getting back zero on the dollar. [18:42] a good case for dc statehood [18:43] jimbaker, it looks like a zk connection exhaustion, esp with the timing [18:47] hazmat, any good options here? [18:47] jimbaker, yeah.. i'm making one.. i filed a ticket on it last night.. bug 754318 [18:47] <_mup_> Bug #754318: add register cleanup utility to cli < https://launchpad.net/bugs/754318 > [18:47] jimbaker, basically just a function to register connections generically for cleanup when the cli exits [18:51] hazmat, so we regain connections with a client.close? [18:52] hazmat, should we also play around with maxClientCnxns in the zk config? [18:54] jimbaker, i've done the latter a bit earlier.. i think we're up to 32 connections at the moment, they recently changed the default upstream to increase it. really the solution is to not have tests leak connections, automatic cleanup for the cli, would solve most of it, perhaps a forsenic tool as well for checking for dangling open connections from a test [18:54] most of the other tests manage their own connection [18:54] s/most/all [18:55] the cli tests do internal construction, so the automatic cleanup registration when getting a connection there should solve it, afaics [18:56] hazmat, i changed maxClientCnxns=0 for ZK; with that the tests run except for ensemble.providers.tests.test_dummy.DummyProviderFileStorageTest.test_file_storage_returns_same_storage - the original failing test for me [18:56] =0 means unlimited of course [18:57] jimbaker, yeah.. i fixed that as well [18:57] hazmat, i think the cleanup mech sounds good, but maybe do =0 for now so trunk works [18:58] jimbaker, sounds good.. let me put together a minimal trunk diff [18:59] jimbaker, can you give a +1 on this trivial patch.. https://pastebin.canonical.com/45886/ [19:00] hazmat, trying it out [19:03] hazmat, +1 [19:04] hazmat, we should add to bug 754318 that ideally we can get the max connections down to a small number, just to ensure cleanup in the case you mentioned [19:04] <_mup_> Bug #754318: add register cleanup utility to cli < https://launchpad.net/bugs/754318 > [19:05] hazmat, if that makes sense to you, i will add that in to the text of the bug [19:05] jimbaker, sounds good [19:05] <_mup_> ensemble/trunk r194 committed by kapil.thangavelu@canonical.com [19:05] <_mup_> up max client connections for test zk, to temporarily prevent connection exhaustion, fix typo in a dummy provider test. [r=jimbaker][trivial] [19:05] bcsaller, jimbaker, niemeyer standup? [19:05] yeah [19:05] hazmat, sounds good, let me jump on skype [19:06] hazmat: I would like to skip it if that's ok, unless you have something for debate specifically [19:06] i need to go run some errands in a few before the gov shutdown [19:06] I'm working on reviews [19:06] and have to finish that today [19:06] shutdown does seem imminent [19:06] niemeyer, i did have one.. [19:06] and i was actually planning to go to the nearby national park this weekend :( [19:06] niemeyer, but it can wait.. its the notion that upgrade-formula cli interupption is an inconsistent state, i noted on the merge proposal, but it never got discussed [19:07] hazmat: Ok, can we please delay that until Monday? [19:07] thats fine with me [19:07] there's a larger discussion on some other things, but i'll kick it off in response to kim0's email [19:07] niemeyer, sounds good [19:07] hazmat: I'd like to be able to focus on the problem you'll bring up [19:07] kapil and I talked over some changes on my branch and I have some revisions to do [19:07] hazmat: But don't want to let my current mental state go away [19:08] maybe skip standup then? [19:08] sounds like it will be Monday [19:08] my big issue just got resolved, now i can get back to releasing my branches [19:09] well, release one more branch, plus get another one ready for review :) [19:09] sounds good, i'm off to the post office, back in a bitg [19:09] In that case I'm going to get some food. [19:34] <_mup_> ensemble/new-hook-semantics-4-remove-env-var r193 committed by jim.baker@canonical.com [19:34] <_mup_> Merged trunk [19:39] <_mup_> ensemble/trunk r195 committed by jim.baker@canonical.com [19:39] <_mup_> merge new-hook-semantics-4-remove-env-var [r=niemeyer][f=740330] [19:39] <_mup_> Removes the setting of the ENSEMBLE_CHANGE environment variable. [19:54] Hrm.. I really need to be able to make principia work w/ the current ensemble trunk. :-P [19:55] * SpamapS feels it will take a lot of effort. :-/ [19:56] SpamapS, you may want to take a look at lp:~jimbaker/ensemble/bashified-wordpress-mysql-examples, which i just pushed up [19:56] jimbaker: Yeah I will do that. [19:56] this branch incorporates trunk as it is now, including changes to relation-get and new hook semantics. it's actually not too bad [19:57] We're trying to get ensemble to work w/ openstack btw [19:57] SpamapS, now that we are deploying against natty, i'm going to change it to use augtool [19:57] not sure if trunk has this problem, but in the old rev in the PPA you have to hack the code to set the ec2 and s3 uri's [19:58] SpamapS, this bug is still outstanding, not certain if this will have impact on that: bug 725082 [19:58] <_mup_> Bug #725082: DNS error when specifying a different S3 URI < https://launchpad.net/bugs/725082 > [20:20] jimbaker: yeah we ran into that right at the beginning. :-( [21:35] jimbaker? [21:44] <_mup_> ensemble/state-machine-enhancements r199 committed by kapil.thangavelu@canonical.com [21:44] <_mup_> statemachine success transition support [21:56] <_mup_> ensemble/state-machine-enhancements r200 committed by kapil.thangavelu@canonical.com [21:56] <_mup_> add support for transition specified actions [22:01] Heading off for some exercising.. will be back later [22:07] niemeyer, got a moment? [22:07] too late? [22:07] jimbaker, bcsaller ? [22:07] i wanted to bounce an idea regarding upgrade state transitions on error [22:14] <_mup_> ensemble/state-machine-enhancements r201 committed by kapil.thangavelu@canonical.com [22:14] <_mup_> pep8 [22:32] <_mup_> Bug #755062 was filed: statemachine needs support for success transitions and custom actions < https://launchpad.net/bugs/755062 >