*** openstack has joined #openstack-sahara | 12:40 | |
*** tosky has quit IRC | 12:48 | |
*** adrienverge has joined #openstack-sahara | 12:50 | |
openstackgerrit | Nikita Konovalov proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 12:53 |
---|---|---|
*** miqui has joined #openstack-sahara | 12:58 | |
*** _crobertsrh is now known as crobertsrh | 13:01 | |
*** shakamunyi has joined #openstack-sahara | 13:02 | |
*** shakamunyi has quit IRC | 13:06 | |
openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Added docs for running Sahara in distributed mode https://review.openstack.org/124742 | 13:20 |
*** tosky has joined #openstack-sahara | 13:29 | |
*** tmckay has joined #openstack-sahara | 13:48 | |
elmiko | dmitryme: hey, trying to follow the conversation between you and tosky from earlier, what's the issue? | 13:52 |
tmckay | I read back, too, I'll look at the patches | 13:59 |
tmckay | SergeyLukjanov, I can probably have https://bugs.launchpad.net/sahara/+bug/1357615 done today. It is half-written :) But, waiting for Kilo-1 is probably fine too | 14:00 |
uvirtbot | Launchpad bug 1357615 in sahara "[DOC] Plugin SPI method get_edp_engine is not documented" [Medium,In progress] | 14:00 |
tmckay | I noticed on Friday that it had been sitting for a while, so I re-assigned to myself | 14:00 |
*** shakamunyi has joined #openstack-sahara | 14:02 | |
*** shakamunyi has quit IRC | 14:07 | |
elmiko | tosky, dmitryme, https://bugs.launchpad.net/sahara/+bug/1375278 | 14:11 |
uvirtbot | Launchpad bug 1375278 in sahara "[DOC] Installation instructions for RDO are out of date" [Undecided,New] | 14:11 |
elmiko | i'll try and put together a patch today | 14:11 |
tosky | \o/ | 14:11 |
*** shakamunyi has joined #openstack-sahara | 14:32 | |
*** k4n0 has quit IRC | 14:34 | |
stannie | hi, hmm when updated the plugin capability matrix I was wondering if the HDP plugin was also able to provide full scalability? In the current matrix it's written as "Scale Up", I thought it was working with both, anyone has some info about this please? | 14:43 |
*** shakamunyi has quit IRC | 14:46 | |
*** shakamunyi has joined #openstack-sahara | 15:03 | |
crobertsrh | Question for all: Do we want Spark and CDH to be added to the plugin list by default (in configs)? I asked Friday, but everyone (almost) was gone. | 15:03 |
elmiko | stannie: i've tested scale up but never down for hdp | 15:04 |
*** Longgeek has quit IRC | 15:20 | |
tmckay | crobertsrh, was spark around for icehouse? I forget | 15:22 |
tmckay | I'm almost completely unfamiliar with CDH | 15:23 |
* tmckay goes to check stable icehouse branch | 15:23 | |
openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Removed mentions of Sahara Dashboard https://review.openstack.org/124683 | 15:26 |
crobertsrh | I don't think it was | 15:26 |
crobertsrh | Parts might have been there though | 15:26 |
crobertsrh | I tried CDH on Friday....I got parts of it to come up...the ambari-like manager seems nice | 15:27 |
tmckay | crobertsrh, spark is not in stable/icehouse | 15:27 |
crobertsrh | Right, but should we put it in default configs for Juno? | 15:27 |
elmiko | dmitryme: nice change on creating the common section | 15:28 |
dmitryme | elmiko: thank you | 15:28 |
tmckay | that's what I'm thinking about. It was added in May ... and we have EDP. I say "sure" -- let gerrit decide | 15:28 |
crobertsrh | Ok, I'll go for it :) | 15:29 |
tmckay | crobertsrh, can you add a simple CR just to add spark? And maybe a dependent one for CDH? I have no input on CDH, never tried it | 15:29 |
crobertsrh | Already doing it | 15:29 |
tmckay | great, thanks | 15:29 |
crobertsrh | I can make separate CR for spark and CDH in the event that CDH isn't ready for prime time. | 15:30 |
*** adrienverge has quit IRC | 15:30 | |
dmitryme | crobertsrh: I think that is a good idea | 15:31 |
crobertsrh | dmitryme: same for CDH? Or just for Spark? | 15:31 |
dmitryme | I would say for both | 15:31 |
dmitryme | but I am not sure if we want to enable CDH | 15:32 |
dmitryme | at least we don’t have docs for it yet | 15:32 |
crobertsrh | Ok, I'll do both, but in separate patches. If people hate CDH, we can veto it in gerrit | 15:32 |
crobertsrh | True | 15:32 |
tmckay | hater :) | 15:33 |
tmckay | brb | 15:33 |
crobertsrh | I probably need to see a doc for it since my cluster didn't fully come up when I tried it out. | 15:33 |
openstackgerrit | Chad Roberts proposed a change to openstack/sahara: Adding Spark to the list of default plugins https://review.openstack.org/124786 | 15:51 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara: Updating RDO installation documentation https://review.openstack.org/124788 | 16:09 |
SergeyLukjanov | tmckay, hey, I'm ok with merging fix re plugin spi missed doc, but it's not a blocker for juno release | 16:13 |
tmckay | SergeyLukjanov, okay. Working on it now, should be done today. A lot of methods to document :) | 16:14 |
SergeyLukjanov | tmckay, crobertsrh, could you please review https://review.openstack.org/#/c/124788/1 ? | 16:15 |
SergeyLukjanov | tmckay, ack, thank you | 16:15 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara: Updating RDO installation documentation https://review.openstack.org/124788 | 16:18 |
SergeyLukjanov | tmckay, what do you think about https://review.openstack.org/#/c/124786/1 & | 16:27 |
tmckay | SergeyLukjanov, spark is new, but it was merged originally in May. I've had a good experience with it working with EDP. I think it's good to highlight that Sahara can do more than only Hadoop. | 16:30 |
SergeyLukjanov | tmckay, absolutely agree with you | 16:30 |
tmckay | we probably should build up integration tests for it in Kilo, though. I think they are light | 16:30 |
tmckay | we're missing scaling | 16:32 |
tmckay | and EDP only does sparkPi, I think. We definitely need more when we have spark/swift | 16:33 |
SergeyLukjanov | tmckay, yeah, but I think it's ok to enable by default | 16:33 |
tmckay | agreed | 16:33 |
tmckay | SergeyLukjanov, not sure about CDH, though. crobertsrh was asking about enabling that by default, too | 16:33 |
tmckay | I'm unfamiliar with it | 16:34 |
openstackgerrit | Nikita Konovalov proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 16:34 |
SergeyLukjanov | tmckay, I'm mostly sure about CDH to keep it disabled by default | 16:34 |
tmckay | k | 16:34 |
openstackgerrit | Nikita Konovalov proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 16:34 |
SergeyLukjanov | tmckay, it's not well used even by us | 16:34 |
SergeyLukjanov | tmckay, and there were a lot issues with it in sahara-ci | 16:34 |
SergeyLukjanov | tmckay, so, I hope to enable it in Kilo | 16:34 |
tmckay | okay, I agree then for sure | 16:35 |
*** tosky has quit IRC | 16:43 | |
*** bradd1 has joined #openstack-sahara | 16:56 | |
*** bradd1 has left #openstack-sahara | 16:56 | |
openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Added docs for running Sahara in distributed mode https://review.openstack.org/124742 | 16:57 |
openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Removed mentions of Sahara Dashboard https://review.openstack.org/124683 | 17:00 |
SergeyLukjanov | tmckay, crobertsrh, elmiko, please take a look on this patchs ^^ | 17:02 |
crobertsrh | will do | 17:02 |
*** ViswaV has joined #openstack-sahara | 17:05 | |
*** ViswaV has quit IRC | 17:10 | |
*** akuznetsov has joined #openstack-sahara | 17:17 | |
elmiko | SergeyLukjanov: ack | 17:24 |
*** ViswaV has joined #openstack-sahara | 17:30 | |
tmckay | have to run to the airport to retrieve wife and daughter, back soon :) | 17:31 |
crobertsrh | take a car | 17:32 |
*** ViswaV has quit IRC | 17:35 | |
*** ViswaV has joined #openstack-sahara | 17:56 | |
*** ViswaV_ has joined #openstack-sahara | 17:57 | |
*** ViswaV has quit IRC | 17:57 | |
*** mattf is now known as _mattf | 17:57 | |
*** ViswaV_ has quit IRC | 17:59 | |
*** saurabhs has joined #openstack-sahara | 18:24 | |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Removed mentions of Sahara Dashboard https://review.openstack.org/124683 | 18:27 |
*** _mattf is now known as mattf | 18:36 | |
*** ViswaV has joined #openstack-sahara | 19:01 | |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Added docs for running Sahara in distributed mode https://review.openstack.org/124742 | 19:01 |
*** ViswaV has quit IRC | 19:02 | |
*** ViswaV has joined #openstack-sahara | 19:03 | |
*** ViswaV has quit IRC | 19:03 | |
openstackgerrit | Chad Roberts proposed a change to openstack/sahara: Adding CDH to the list of default plugins https://review.openstack.org/124835 | 19:07 |
tmckay | crobertsrh, earlier SergeyLukjanov and I chatted about CDH. The idea was to add it in kilo, maybe not quite stable | 19:13 |
crobertsrh | Yeah, I think I gathered that it wouldn't go for J | 19:14 |
SergeyLukjanov | tmckay, will you be available in an hour? | 19:14 |
tmckay | crobertsrh, I know, we said let gerrit vote. Just a heads up. | 19:14 |
tmckay | SergeyLukjanov, yes | 19:14 |
crobertsrh | No prob. | 19:14 |
* tmckay runs down review list again | 19:15 | |
SergeyLukjanov | tmckay, yup, please :) | 19:15 |
tmckay | SergeyLukjanov, at this rate my EDP spi won't be complete today :) Almost, but not quite | 19:16 |
tmckay | can we merge tomorrow, or does it have to be today? | 19:16 |
SergeyLukjanov | tmckay, I think it's ok | 19:16 |
SergeyLukjanov | tmckay, I mean it's ok if we'll not have it in J | 19:16 |
tmckay | k | 19:16 |
tmckay | the bug will link the change | 19:16 |
SergeyLukjanov | tmckay, I'd like to approve all the changes in next ~1 hour | 19:16 |
tmckay | so, someone can find it | 19:16 |
tmckay | ok | 19:17 |
SergeyLukjanov | and it'll be the rc1 and only critical fixes could be applied after it | 19:17 |
tmckay | dmitryme, are you there? | 19:17 |
SergeyLukjanov | tmckay, alazarev is now updating patchs for him ;) | 19:17 |
SergeyLukjanov | it's already 11 pm for dmitryme, so, I'm not sure that he's still online | 19:17 |
tmckay | ok. alazarev, we seem to have lost the common section changes in dmitryme's patch set #2 on https://review.openstack.org/#/c/124683/ | 19:18 |
tmckay | I don't know if this was on purpose. I don't think so. | 19:19 |
tmckay | since dmitry was the champion of the common section :) | 19:19 |
*** alazarev has joined #openstack-sahara | 19:21 | |
elmiko | tmckay: i have to imagine that was just missed somehow | 19:22 |
tmckay | (03:18:42 PM) tmckay: ok. alazarev, we seem to have lost the common section changes in dmitryme's patch set #2 on https://review.openstack.org/#/c/124683/ | 19:22 |
tmckay | SergeyLukjanov, why did you -1 Workflow "Add CONTRIBUTING.rst" patches? | 19:30 |
tmckay | alazarev, are you there? I can fix https://review.openstack.org/#/c/124742/, just extra word | 19:32 |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Removed mentions of Sahara Dashboard https://review.openstack.org/124683 | 19:35 |
SergeyLukjanov | tmckay, I already have enough -1s on it ;) | 19:35 |
SergeyLukjanov | tmckay, need to update them | 19:35 |
tmckay | heh, okay | 19:35 |
openstackgerrit | Trevor McKay proposed a change to openstack/sahara: Added docs for running Sahara in distributed mode https://review.openstack.org/124742 | 19:36 |
tmckay | I fixed dmitry's extra word | 19:36 |
tmckay | elmiko, ^^ +1 please | 19:36 |
elmiko | tmckay: ack | 19:38 |
tmckay | SergeyLukjanov, +2 and approve ^^ :) | 19:38 |
openstackgerrit | A change was merged to openstack/sahara: Adding Spark to the list of default plugins https://review.openstack.org/124786 | 19:38 |
elmiko | tmckay: make sure to hit this one again, https://review.openstack.org/#/c/124683/2 | 19:39 |
tmckay | k, I'm there! | 19:39 |
tmckay | NikitaKonovalov, are you there? | 19:44 |
tmckay | https://review.openstack.org/#/c/124711/5 needs a rebase and a fix to comment. I can do it if you are not there .... :) | 19:44 |
crobertsrh | Any other outstanding issues that I can assist with? | 19:47 |
tmckay | SergeyLukjanov, will changes merge successfully if they are dependent on outdated change? Or do they have to be rebased? https://review.openstack.org/#/c/124711/ | 19:47 |
tmckay | crobertsrh, just running down the review list, trying to get stuff approved | 19:48 |
crobertsrh | Ok. Let me know if you need eyes on anything. | 19:48 |
tmckay | ack, thanks | 19:48 |
openstackgerrit | Trevor McKay proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 19:52 |
SergeyLukjanov | tmckay, it should be ok if no merge conflicts | 19:53 |
tmckay | k, I just fixed commit comment, I'll mark approved again | 19:53 |
elmiko | cool | 19:54 |
tmckay | SergeyLukjanov, I think that's it | 19:54 |
tmckay | everything else has red Xs or multiple -1 | 19:55 |
tmckay | SergeyLukjanov, you want help fixing Contributing? I think it's only grammar | 19:55 |
elmiko | i don't think i'll need to rebase the rdo change, we'll see | 19:55 |
SergeyLukjanov | tmckay, if you have some free time - I'll be happy | 19:56 |
tmckay | elmiko, yeah, we can approve after Saratov is asleep ;-) | 19:56 |
tmckay | SergeyLukjanov, sure, np. It's late for you. | 19:56 |
elmiko | tmckay: i can take some of those too if you want to split it up | 19:56 |
* tmckay goes to fix contributing CRs | 19:56 | |
SergeyLukjanov | tmckay, I'm in CA now | 19:56 |
SergeyLukjanov | so, just 1 pm :0 | 19:56 |
tmckay | ah. Still. 3 hours :) | 19:56 |
tmckay | oh, that's right, the other way. lol | 19:57 |
SergeyLukjanov | but I'm double booked by meetings for the next two weeks :( | 19:57 |
elmiko | lol | 19:57 |
tmckay | hmm, ok. we'll fix | 19:57 |
tmckay | elmkio, claim one | 19:57 |
SergeyLukjanov | tmckay, thank you | 19:57 |
tmckay | https://review.openstack.org/#/c/123899/ | 19:57 |
tmckay | elmiko, that's mine ^^ | 19:57 |
elmiko | tmckay: https://review.openstack.org/#/c/123904/ mine | 19:57 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara-image-elements: Add CONTRIBUTING.rst https://review.openstack.org/123904 | 20:00 |
elmiko | tmckay: https://review.openstack.org/#/c/123905/ | 20:01 |
openstackgerrit | Trevor McKay proposed a change to openstack/python-saharaclient: Add CONTRIBUTING.rst https://review.openstack.org/123899 | 20:01 |
tmckay | elmiko, you beat me, my branches were messed up :) | 20:01 |
elmiko | doh! | 20:02 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara-specs: Add CONTRIBUTING.rst https://review.openstack.org/123905 | 20:02 |
elmiko | tmckay: https://review.openstack.org/#/c/123903/ | 20:03 |
tmckay | I'll just let you finish and I'll approve them :)\ | 20:03 |
SergeyLukjanov | tmckay, dmitryme just find an issue with https://review.openstack.org/#/c/124711/5/doc/source/userdoc/installation.guide.rst,cm | 20:03 |
tmckay | k | 20:03 |
SergeyLukjanov | tmckay, the item # should 3, not 9 | 20:04 |
openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 20:04 |
SergeyLukjanov | tmckay, need your approve :) | 20:04 |
dmitryme | actually , here is it fixed ^^ | 20:04 |
dmitryme | * it is | 20:04 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara-extra: Add CONTRIBUTING.rst https://review.openstack.org/123903 | 20:04 |
tmckay | this is getting a little crazy :) | 20:05 |
elmiko | totally | 20:06 |
elmiko | tmckay: https://review.openstack.org/#/c/123902/ | 20:06 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara-dashboard: Add CONTRIBUTING.rst https://review.openstack.org/123902 | 20:08 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/sahara: [DO NOT MERGE] Test commit https://review.openstack.org/124855 | 20:09 |
elmiko | tmckay: https://review.openstack.org/#/c/123900/ | 20:10 |
tmckay | elmiko, go go go | 20:10 |
openstackgerrit | Michael McCune proposed a change to openstack/sahara: Update CONTRIBUTING.rst https://review.openstack.org/123900 | 20:11 |
elmiko | ok, i think that's all of em | 20:11 |
tmckay | alright, cool | 20:13 |
elmiko | crobertsrh: lol, taking the easy -1 on the DONOTMERGE. i like the cut of your jib sir ;) | 20:14 |
crobertsrh | Just doing my part to make sure no yahoos +1 it blindly :) | 20:14 |
tmckay | https://review.openstack.org/123906 | 20:14 |
tmckay | https://review.openstack.org/123906 | 20:15 |
tmckay | SergeyLukjanov, I think these are the only 2 doc bugs left that need fixing ^^ | 20:15 |
SergeyLukjanov | tmckay. thx, I'll fix it later | 20:16 |
tmckay | okay. I'll check later for needed +2, if any. I'll be around for an hour or so, then a birthday party, then back | 20:16 |
SergeyLukjanov | tmckay, oh, today is your b-day? | 20:17 |
tmckay | oh, not me. My nephew who is visiting from Texas | 20:17 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/sahara: Update custom hacking checks https://review.openstack.org/123906 | 20:17 |
SergeyLukjanov | tmckay. ok:) | 20:17 |
*** ViswaV has joined #openstack-sahara | 20:18 | |
tmckay | mattf, hey, we need a +2 and approve on https://review.openstack.org/#/c/123906/. Sergey made it, so ... :) | 20:19 |
*** ViswaV has quit IRC | 20:20 | |
*** ViswaV has joined #openstack-sahara | 20:20 | |
*** ViswaV has quit IRC | 20:21 | |
*** ViswaV has joined #openstack-sahara | 20:28 | |
*** ViswaV has quit IRC | 20:28 | |
*** ViswaV has joined #openstack-sahara | 20:29 | |
*** ViswaV has quit IRC | 20:30 | |
*** miqui has quit IRC | 20:46 | |
*** crobertsrh is now known as _crobertsrh | 20:48 | |
mattf | tmckay, done | 20:48 |
tmckay | mattf, great, thanks | 20:50 |
tmckay | SergeyLukjanov, okay, I'll check back a few times tonight | 21:43 |
SergeyLukjanov | tmckay, I'll monitor it too | 21:44 |
SergeyLukjanov | tmckay, thx | 21:44 |
*** mattf is now known as _mattf | 21:59 | |
openstackgerrit | A change was merged to openstack/sahara: Added docs for running Sahara in distributed mode https://review.openstack.org/124742 | 22:02 |
openstackgerrit | A change was merged to openstack/sahara: Removed mentions of Sahara Dashboard https://review.openstack.org/124683 | 22:02 |
*** bradd1 has joined #openstack-sahara | 22:42 | |
SergeyLukjanov | any volunteers to rebase https://review.openstack.org/#/c/124788/ ? | 22:46 |
SergeyLukjanov | alazarev, tmckay, elmiko | 22:46 |
alazarev | SergeyLukjanov: I'll do | 22:47 |
SergeyLukjanov | alazarev, ack, th | 22:47 |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Updating RDO installation documentation https://review.openstack.org/124788 | 22:55 |
*** _mattf is now known as mattf | 23:06 | |
openstackgerrit | A change was merged to openstack/sahara: Update custom hacking checks https://review.openstack.org/123906 | 23:11 |
* SergeyLukjanov alazarev, please rebase this one too - | 23:16 | |
SergeyLukjanov | https://review.openstack.org/#/c/124711/ | 23:16 |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 23:17 |
alazarev | SergeyLukjanov done | 23:17 |
openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara: Sahara UI panels configuration docs updated https://review.openstack.org/124711 | 23:38 |
openstackgerrit | A change was merged to openstack/sahara-image-elements: Add CONTRIBUTING.rst https://review.openstack.org/123904 | 23:44 |
openstackgerrit | A change was merged to openstack/sahara-extra: Add CONTRIBUTING.rst https://review.openstack.org/123903 | 23:44 |
openstackgerrit | A change was merged to openstack/sahara-dashboard: Add CONTRIBUTING.rst https://review.openstack.org/123902 | 23:45 |
openstackgerrit | A change was merged to openstack/sahara-specs: Add CONTRIBUTING.rst https://review.openstack.org/123905 | 23:46 |
*** shakamunyi has quit IRC | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!