Skip to content

[LIVY-979][Dev] A helm-based integration environment for Livy - #454

Closed
askhatri wants to merge 1 commit into
apache:masterfrom
askhatri:LIVY-979
Closed

[LIVY-979][Dev] A helm-based integration environment for Livy#454
askhatri wants to merge 1 commit into
apache:masterfrom
askhatri:LIVY-979

Conversation

@askhatri

@askhatri askhatri commented Aug 8, 2024

Copy link
Copy Markdown
Contributor

This helps deploy Livy on Kubernetes on a local machine. For debugging Livy on Kubernetes as a standalone setup, we need Apache Spark and Apache Livy deployed within the Kubernetes environment.

What changes were proposed in this pull request?

JIRA: LIVY-979

How was this patch tested?

Tested by deploying the Helm chart on a standalone machine.

@RajshekharMuchandi RajshekharMuchandi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@gyogal

gyogal commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

Thank you for the submission and reviews! However some of the newly added files are missing the Apache license headers and I am not sure if this is OK to be merged in its current form. The policy I am referring to is https://apache.org/legal/src-headers.html

@edwardcapriolo

Copy link
Copy Markdown
Contributor

A year has gone by are we gonna let the headers be the blocker?

I just packaged up livy here: https://hub.docker.com/r/ecapriolo/livy

Do you wall want to merge this or should I fork it off and maintain on my own?

@gyogal

gyogal commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

@edwardcapriolo I think the missing license headers are an issue that blocks merging (somebody who is more familiar with ASF policies or licensing please let me know if this is incorrect). If the PR could be updated and testing is successful, it would be possible to merge this and I think it would be a very useful addition.

@gyogal gyogal removed their assignment Feb 22, 2026
nileshrathi345 pushed a commit to nileshrathi345/incubator-livy that referenced this pull request Aug 10, 2026
Add dev/helmchart/ to deploy Livy, Spark History Server, RBAC, ingress,
and optional cert-manager on a local Kubernetes cluster (e.g. Docker Desktop).
Includes ASF license headers, validation script, README/Docker.md, and review
fixes: no vendored chart tgz, consistent resource naming, unified RBAC/TLS
config, and derived image tags from sparkVersion.

Builds on the original contribution in apache#454.

Co-authored-by: Asif Khatri <asif.khatri@cloudera.com>
nileshrathi345 pushed a commit to nileshrathi345/incubator-livy that referenced this pull request Aug 10, 2026
Add dev/helmchart/ to deploy Livy, Spark History Server, RBAC, ingress,
and optional cert-manager on a local Kubernetes cluster (e.g. Docker Desktop).
Includes ASF license headers, validation script, README/Docker.md, and review
fixes: no vendored chart tgz, consistent resource naming, unified RBAC/TLS
config, and derived image tags from sparkVersion.

Builds on the original contribution in apache#454.

Co-authored-by: Asif Khatri <123077165+askhatri@users.noreply.github.com>
@gyogal

gyogal commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Closing this PR in favor of a new, extended version submitted at #537

@gyogal gyogal closed this Aug 10, 2026
nileshrathi345 added a commit to nileshrathi345/incubator-livy that referenced this pull request Aug 10, 2026
Add dev/helmchart/ to deploy Livy, Spark History Server, RBAC, ingress,
and optional cert-manager on a local Kubernetes cluster (e.g. Docker Desktop).
Includes ASF license headers, validation script, README/Docker.md, and review
fixes: no vendored chart tgz, consistent resource naming, unified RBAC/TLS
config, and derived image tags from sparkVersion.

Builds on the original contribution in apache#454.

Co-authored-by: Asif Khatri <123077165+askhatri@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants