Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-49750][DOC] Mention delegation token support in K8s mode #48199

Closed
wants to merge 1 commit into from

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Sep 22, 2024

What changes were proposed in this pull request?

Update docs to mention delegation token support in K8s mode.

Why are the changes needed?

The delegation token support in K8s mode has been implemented since 3.0.0 via SPARK-23257.

Does this PR introduce any user-facing change?

Yes, docs are updated.

How was this patch tested?

Review.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the DOCS label Sep 22, 2024
@pan3793
Copy link
Member Author

pan3793 commented Sep 22, 2024

maybe cc @viirya

@@ -947,7 +947,7 @@ mechanism (see `java.util.ServiceLoader`). Implementations of
`org.apache.spark.security.HadoopDelegationTokenProvider` can be made available to Spark
by listing their names in the corresponding file in the jar's `META-INF/services` directory.

Delegation token support is currently only supported in YARN mode. Consult the
Delegation token support is currently only supported in YARN and Kubernetes mode. Consult the
deployment-specific page for more information.

The following options provides finer-grained control for this feature:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is another place:

Users can exclude Kerberos delegation token renewal at resource scheduler. Currently it is only supported
on YARN.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consult the deployment-specific page for more information.

Btw, for YARN, it is true that we have a corresponding section for it as following. Do we have same one for Kubernetes? If so, can you also add the link?

The configuration is covered in the [Running Spark on YARN](running-on-yarn.html#yarn-specific-kerberos-configuration) page.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@viirya DT renewal exclusion is another feature that is only supported by YARN.

In YARN mode, the RM could take responsibility for the token renewal, in K8s

there isn't a definition of what "external renewal service" is anywhere

ref SPARK-25825

Copy link
Member Author

@pan3793 pan3793 Sep 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw, for YARN, it is true that we have a corresponding section for it as following. Do we have same one for Kubernetes?

Unlike YARN, the introduction of the K8s Kerberos support was embedded on the security page.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unlike YARN, the introduction of the K8s Kerberos support was embedded on the security page.

Do you mean the section "Secure Interaction with Kubernetes"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

@yaooqinn
Copy link
Member

It appears that StandaloneSchedulerBackend is also supported.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you, @pan3793 , @viirya , and @yaooqinn .
Let's handle this as a kind of follow up of SPARK-23257. For Standalone cluster, let's proceed to discuss seperately.

Merged to master/3.5/3.4.

dongjoon-hyun pushed a commit that referenced this pull request Sep 24, 2024
Update docs to mention delegation token support in K8s mode.

The delegation token support in K8s mode has been implemented since 3.0.0 via SPARK-23257.

Yes, docs are updated.

Review.

No.

Closes #48199 from pan3793/SPARK-49750.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit dedf5aa)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
dongjoon-hyun pushed a commit that referenced this pull request Sep 24, 2024
Update docs to mention delegation token support in K8s mode.

The delegation token support in K8s mode has been implemented since 3.0.0 via SPARK-23257.

Yes, docs are updated.

Review.

No.

Closes #48199 from pan3793/SPARK-49750.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit dedf5aa)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit b513297)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
attilapiros pushed a commit to attilapiros/spark that referenced this pull request Oct 4, 2024
### What changes were proposed in this pull request?

Update docs to mention delegation token support in K8s mode.

### Why are the changes needed?

The delegation token support in K8s mode has been implemented since 3.0.0 via SPARK-23257.

### Does this PR introduce _any_ user-facing change?

Yes, docs are updated.

### How was this patch tested?

Review.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#48199 from pan3793/SPARK-49750.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
himadripal pushed a commit to himadripal/spark that referenced this pull request Oct 19, 2024
### What changes were proposed in this pull request?

Update docs to mention delegation token support in K8s mode.

### Why are the changes needed?

The delegation token support in K8s mode has been implemented since 3.0.0 via SPARK-23257.

### Does this PR introduce _any_ user-facing change?

Yes, docs are updated.

### How was this patch tested?

Review.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#48199 from pan3793/SPARK-49750.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants