Added Kerberos documentation

[sejunra]

What is this PR for?

Added Kerberos setup with Zeppelin

What type of PR is it?

Documentation

Is there a relevant Jira issue?

No

How should this be tested?

Check out the website

Screenshots (if appropriate)

Questions:

• Does the licenses files need update? No
• Is there breaking changes for older versions? No
• Does this needs documentation? No

[bzz]

Thank you for contribution @sejunra

Right now CI fails on the new file you have added, as it does not have an apache licence header.

 Unknown Licenses

*******************************

Unapproved licenses:

  docs/manual/conf_kerberos.md

*******************************

Would you care for adding one please?
See this example for a licence header in docs, right before {% include JB/setup %}

[Leemoonsoo]

Thanks for the documentation.
While the documentation is specific for Spark Interpreter, why not adding it under /docs/interpreter/spark.md ?

[sejunra]

@Leemoonsoo kept it on manual instead of your recommended /interpreter folder as it made more sense under /manual which is more how-tos and not related to interpreters (though it's specific to spark). If you feel strongly about the /interpreter location, I can move it.

[Leemoonsoo]

@sejunra
The docs/interpreter/spark.md supposed to have configurations for Spark Interpreter. And this document have instruction for configuring SparkInterpreter with Kerberos enabled yarn cluster. Not applied to any other interpreter nor Zeppelin server daemon. I think docs/interpreter/spark.md natural place to be.

Then the duplication that this documents made (instruction for SPARK_HOME and HADOOP_CONF_DIR) also can be removed. And user can see all necessary information in a single place.

It's up to you, but i suggest move it into docs/interpreter/spark.md or at least update doc to make it clear that the instruction is for only Spark interpreter.

[felixcheung]

@sejunra any update?

[sejunra]

sorry. thought i already pushed. done.

[felixcheung]

I think this should be ../assets/themes/zeppelin/img/docs-img/...

[sejunra]

Made all changes as requested.

[felixcheung]

looks good to me - @AhyoungRyu @Leemoonsoo ?

[Leemoonsoo]

LGTM

[AhyoungRyu]

@sejunra After applying this PR, I noticed there is already horizon line below the title because of Zeppelin's style.css.

So, you don't need to add additional <hr> tag : )

[AhyoungRyu]

I left some comments in this file. Except them, LGTM! : )

[bzz]

@AhyoungRyu thank you for a thoughtful review!
If you have a spare moment and could contribute those 2 changes, please do, as it might really speed merging this on a lot.

[AhyoungRyu]

@bzz I've addressed and pushed them : )

[felixcheung]

don't we need to escape _ in SPARK_HOME here?

[AhyoungRyu]

@felixcheung I don't know the exact reason, but that SPARK_HOME is shown properly.

[bzz]

Thank you so much for taking care!

Merging this guy if there is no more discussion.

[AhyoungRyu]

Let's merge! : )