[00:29] <rick_h_> How goes vinodhini ?
[00:40] <babbageclunk> anastasiamac: PTAL https://github.com/juju/juju/pull/9078?
[00:40]  * anastasiamac looking
[00:41] <babbageclunk> ta
[00:41] <babbageclunk> thumper: trivial review? https://github.com/juju/pubsub/pull/8
[00:42] <vinodhini> hi rick_h i am good.
[00:42] <vinodhini> I fixed all minor test issues.
[00:43] <vinodhini> If u can take a look please.
[00:43] <anastasiamac> babbageclunk: so neat! loved it :D
[00:43] <babbageclunk> I'm going to put that quote on the cover of the paperback when it comes out!
[00:44] <babbageclunk> Thanks anastasiamac
[00:45] <babbageclunk> hmm, good question about 2.4 - rick_h_, what do you think? https://github.com/juju/juju/pull/9078
[00:46] <vinodhini> rick_h : regarding the error handling issues.Where you able to test that same scenarios in GUI ?
[00:48] <rick_h_> babbageclunk: hmm I don't think oci is in 2.4
[00:48] <babbageclunk> rick_h_: ah, ok - then that makes it simpler. I'll just look
[00:49] <rick_h_> vinodhini: I didn't. What are we looking for or expecting?
[00:50] <babbageclunk> rick_h_: yeah you're right - the package is there but it's all just NotImplemented stubs.
[00:51] <rick_h_> babbageclunk: cool
[00:52] <vinodhini> rick_h: approval for PR. Plus regarding the errorhandling  issues u have mentioned. Yes. there is a delay until the whole setup comes up fully.
[00:53] <rick_h_> vinodhini: feedback inbound
[00:54] <rick_h_> vinodhini: yes, so thumper had some ideas around some of those, implementation-wise
[00:54] <rick_h_> vinodhini: so the version thing I think needs to be right for that PR other than that thanks so much for the great updates
[00:56] <vinodhini> ok. then i would like to discuss regarding that delay issues today.
[00:57] <vinodhini> let me fix the version so i can get that merged.
[00:58] <vinodhini> rick_h: isnt better not to have version in that error message so there wont be any misinterpretation.
[00:59] <rick_h_> vinodhini: that's ok for the moment.
[01:00] <rick_h_> vinodhini: do me one favor though, make sure to call it the "bundle export feature"
[01:00] <rick_h_> vinodhini: as any juju 2.x supports deploying bundles
[01:00] <rick_h_> vinodhini: just to be clear to users
[01:01] <vinodhini> sure then i will remove the version and update the message.
[01:03] <rick_h_> vinodhini: awesome
[01:04] <vinodhini> rick_h: i have push the commit now.
[01:04] <vinodhini> so, we are done with cosmetic changes related to export-bundle.
[01:05] <vinodhini> We have now: 3 pending
[01:06] <vinodhini> delay and error handling, series, machine constraints
[01:06] <rick_h_> vinodhini: k, approved
[01:06] <vinodhini> rick_h: ty
[01:06] <rick_h_> vinodhini: are you also able to get the good update in during your day so we can lock down 2.4.2?
[01:06] <rick_h_> Sorry, goose, not good
[01:07] <vinodhini> rick_h: goose - not good ?
[01:09] <rick_h_> vinodhini: so thumper says that we need to update the goose dependency with your fixes in the 2.4 branch to be part of the release we're locking up tomorrow morning.
[01:12] <vinodhini> rick_h: i have updated the goose dep with my changes i mean after merging goose stuffs and merged the juju PR: https://github.com/juju/juju/pull/9049
[01:13] <rick_h_> vinodhini: ok, so 2.4 branch is all caught up with your goose changes?
[01:13] <vinodhini> yes.
[01:14] <vinodhini> thumper: i have updated the juju side PR which got merged after the goose chnages landed with dependencies.tsv update.
[01:15] <vinodhini> Apologise i didnt inform but i did it.
[01:15] <rick_h_> vinodhini: cool ignore me then. Sounds good.
[01:16] <vinodhini> rick_h: sure. Its ok to double check.
[01:20] <vinodhini> thumper: that PR : https://github.com/juju/juju/pull/9076 - export-bundle in dev mode should go with 2.4.2 correct ?
[01:21] <vinodhini> i have answered ur question this morning. Could u please take a look when u get time ?
[02:05] <thumper> vinodhini: commented on PR, I'd like us to keep the file
[02:06] <vinodhini> thumper: sure
[02:06] <vinodhini> reverting now.
[02:18] <vinodhini> thumper: pushed to commit.
[02:18] <thumper> ack
[02:18] <thumper> approved
[02:18] <vinodhini> thumper: ty
[02:37] <thumper> bugger...
[02:37] <thumper> that thing that I knew was going to be a problem in the future is a problem
[02:38]  * thumper thinks how to fix with minimum impact
[02:38] <thumper> babbageclunk: busy?
[02:40] <vinodhini> i believe rick_h is offline. thumper: when u get time before you leave for the day i would like to discuss regarding the other issues in export-bundle.
[02:40] <thumper> vinodhini: sure
[02:40] <thumper> I can do now if you like
[02:40] <vinodhini> I have updated the doc with few questions which rick_h also can read thru
[02:40] <vinodhini> ok.
[02:41] <vinodhini> ho ?
[02:42] <thumper> yep, in 1:1
=== jhebden is now known as jhebden-lunch
[03:20] <babbageclunk> thumper: sorry, totally missed your ping
[03:20] <babbageclunk> is now good?
[03:23] <thumper> yep
[03:23] <thumper> babbageclunk: ^^
[03:23] <babbageclunk> saw that one!
[03:24] <thumper> babbageclunk: I'm in our 1:1
[03:50] <anastasiamac> thumper: we have some fairly old PRs (say from Jan this year)... do we still want them around?
[08:21] <magicaltrout> is it me or is JAAS really sad this morning?
=== ashipika is now known as ashipika|afk
[11:12] <rick_h_> magicaltrout: not sure just waking up.
[11:18] <rick_h_> magicaltrout: passed the report along. Any details?
=== cory_fu_ is now known as cory_fu
[14:06] <fnordahl> for a bundle without a `machines` section, is there a way to decide in which order the machines will be allocated for charms?
[14:25] <rick_h_> fnordahl: in what order? no. It just means juju will ask for machines that meet the application constraints and take what the underlying cloud gives it
=== salmankhan1 is now known as salmankhan
[19:47] <magicaltrout> if anyone is still here can you give https://github.com/juju/layer-index/pull/42/files a merge before you all bugger off for the weekend
[19:48]  * rick_h_ wonders what an oozie is
[19:49] <rick_h_> magicaltrout: "The repository for this project is empty" :(
[19:51] <magicaltrout> pfft
[19:51] <magicaltrout> i was padding it out so someone could merge it :P
[19:51] <magicaltrout> there is now a relation
[19:51] <rick_h_> magicaltrout: :)
[19:51] <magicaltrout> in answer to your question, oozie is a big data workflow manager
[19:52] <magicaltrout> in the bigtop stack
[19:52] <rick_h_> magicaltrout: ok, hit the merge button
[19:52] <rick_h_> magicaltrout: hope I don't get in any trouble :P
[19:53] <magicaltrout> thanks. Just finishing up our hue charm
[21:09] <kwmonroe> magicaltrout: did you mean to have #41 closed in favor of #42? https://github.com/juju/layer-index/pull/41 -- i see both included oozie, but jdbc was already there?  i'm so confused.
[21:58] <magicaltrout> don't think it matters kwmonroe
[21:59] <magicaltrout> your user tip from yesterday worked a treat thanks
[22:43] <magicaltrout> sweet and a working spark config for the thrift server
[22:43] <magicaltrout> i'll have a couple of PR's for you on monday kwmonroe to complain about and throw back in my face