rick_h_ | How goes vinodhini ? | 00:29 |
---|---|---|
babbageclunk | anastasiamac: PTAL https://github.com/juju/juju/pull/9078? | 00:40 |
* anastasiamac looking | 00:40 | |
babbageclunk | ta | 00:41 |
babbageclunk | thumper: trivial review? https://github.com/juju/pubsub/pull/8 | 00:41 |
vinodhini | hi rick_h i am good. | 00:42 |
vinodhini | I fixed all minor test issues. | 00:42 |
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:43 |
babbageclunk | Thanks anastasiamac | 00:44 |
babbageclunk | hmm, good question about 2.4 - rick_h_, what do you think? https://github.com/juju/juju/pull/9078 | 00:45 |
vinodhini | rick_h : regarding the error handling issues.Where you able to test that same scenarios in GUI ? | 00:46 |
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:48 |
rick_h_ | vinodhini: I didn't. What are we looking for or expecting? | 00:49 |
babbageclunk | rick_h_: yeah you're right - the package is there but it's all just NotImplemented stubs. | 00:50 |
rick_h_ | babbageclunk: cool | 00:51 |
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:52 |
rick_h_ | vinodhini: feedback inbound | 00:53 |
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:54 |
vinodhini | ok. then i would like to discuss regarding that delay issues today. | 00:56 |
vinodhini | let me fix the version so i can get that merged. | 00:57 |
vinodhini | rick_h: isnt better not to have version in that error message so there wont be any misinterpretation. | 00:58 |
rick_h_ | vinodhini: that's ok for the moment. | 00:59 |
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:00 |
vinodhini | sure then i will remove the version and update the message. | 01:01 |
rick_h_ | vinodhini: awesome | 01:03 |
vinodhini | rick_h: i have push the commit now. | 01:04 |
vinodhini | so, we are done with cosmetic changes related to export-bundle. | 01:04 |
vinodhini | We have now: 3 pending | 01:05 |
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:06 |
vinodhini | rick_h: goose - not good ? | 01:07 |
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:09 |
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:12 |
rick_h_ | vinodhini: ok, so 2.4 branch is all caught up with your goose changes? | 01:13 |
vinodhini | yes. | 01:13 |
vinodhini | thumper: i have updated the juju side PR which got merged after the goose chnages landed with dependencies.tsv update. | 01:14 |
vinodhini | Apologise i didnt inform but i did it. | 01:15 |
rick_h_ | vinodhini: cool ignore me then. Sounds good. | 01:15 |
vinodhini | rick_h: sure. Its ok to double check. | 01:16 |
vinodhini | thumper: that PR : https://github.com/juju/juju/pull/9076 - export-bundle in dev mode should go with 2.4.2 correct ? | 01:20 |
vinodhini | i have answered ur question this morning. Could u please take a look when u get time ? | 01:21 |
thumper | vinodhini: commented on PR, I'd like us to keep the file | 02:05 |
vinodhini | thumper: sure | 02:06 |
vinodhini | reverting now. | 02:06 |
vinodhini | thumper: pushed to commit. | 02:18 |
thumper | ack | 02:18 |
thumper | approved | 02:18 |
vinodhini | thumper: ty | 02:18 |
thumper | bugger... | 02:37 |
thumper | that thing that I knew was going to be a problem in the future is a problem | 02:37 |
* thumper thinks how to fix with minimum impact | 02:38 | |
thumper | babbageclunk: busy? | 02:38 |
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:40 |
vinodhini | ho ? | 02:41 |
thumper | yep, in 1:1 | 02:42 |
=== jhebden is now known as jhebden-lunch | ||
babbageclunk | thumper: sorry, totally missed your ping | 03:20 |
babbageclunk | is now good? | 03:20 |
thumper | yep | 03:23 |
thumper | babbageclunk: ^^ | 03:23 |
babbageclunk | saw that one! | 03:23 |
thumper | babbageclunk: I'm in our 1:1 | 03:24 |
anastasiamac | thumper: we have some fairly old PRs (say from Jan this year)... do we still want them around? | 03:50 |
magicaltrout | is it me or is JAAS really sad this morning? | 08:21 |
=== ashipika is now known as ashipika|afk | ||
rick_h_ | magicaltrout: not sure just waking up. | 11:12 |
rick_h_ | magicaltrout: passed the report along. Any details? | 11:18 |
=== cory_fu_ is now known as cory_fu | ||
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:06 |
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 | 14:25 |
=== salmankhan1 is now known as salmankhan | ||
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:47 |
* rick_h_ wonders what an oozie is | 19:48 | |
rick_h_ | magicaltrout: "The repository for this project is empty" :( | 19:49 |
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:51 |
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:52 |
magicaltrout | thanks. Just finishing up our hue charm | 19:53 |
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:09 |
magicaltrout | don't think it matters kwmonroe | 21:58 |
magicaltrout | your user tip from yesterday worked a treat thanks | 21:59 |
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 | 22:43 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!