Commit Graph

911 Commits

Author SHA1 Message Date
Bo Yang b723367d79
Support exposing extra TCP ports in Spark Driver via K8s Ingress (#1998)
* Add driverIngressOptions in SparkApplication CRD

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>

* Update chart version to 1.3.0

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>

* Update helm chart README

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>

* Fix make detect-crds-drift

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>

* Update api-docs.md

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>

---------

Signed-off-by: Bo (AIML) Yang <bo_yang6@apple.com>
2024-05-20 19:42:48 +00:00
Peter McClonski 153537e82d
Fixes a bug with dynamic allocation forcing the executor count to be 1 even when minExecutors is set to 0 (#1979)
Signed-off-by: Peter McClonski <mcclonski_peter@bah.com>
2024-05-17 22:24:45 +00:00
Andrej Kyselica 1d123caa97
Remove outdated PySpark experimental warning in example (#2014)
Signed-off-by: Andrej Kyselica <andrejky@microsoft.com>
2024-05-09 21:52:45 +00:00
Matthew Rossi a1efb79a4f
Update Spark Job Namespace docs (#2000)
Signed-off-by: Matthew Rossi <mattw.rossi@gmail.com>
2024-05-07 15:58:43 +00:00
Peter McClonski d87c6855ff
Fix examples (#2010)
Signed-off-by: Peter Jablonski <mcclonski.peter@gmail.com>
2024-05-07 15:10:43 +00:00
Cian Gallagher 333ee0bcb2
feat: add support for service labels on driver-svc (#1985)
* feat: add support for service labels on driver-svc

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* docs: update helm docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* fix: undo changes to api-docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* docs: update api-docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* fix: update appVersion

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* ci: remove step to check api change

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* docs: update helm-docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* docs: update helm-docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* fix: update app version

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

* docs: update helm docs

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>

---------

Signed-off-by: Cian Gallagher <cian.gallagher@zendesk.com>
2024-04-26 00:44:46 +00:00
t3mi 8fc405822d
fix(chart): remove operator namespace default for job namespaces value (#1989)
Signed-off-by: t3mi <t3mi@users.noreply.github.com>
2024-04-24 16:41:19 +00:00
Andrey Velichkevich bf2eccefef
Fix Docker Hub Credentials in CI (#2003)
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
2024-04-24 14:49:19 +00:00
Jacob Salway ccb3ceb54b
Add emptyDir sizeLimit support for local dirs (#1993)
* Add emptyDir sizeLimit support

Signed-off-by: Jacob Salway <jacob.salway@rokt.com>

* Bump appVersion and add sizeLimit example

Signed-off-by: Jacob Salway <jacob.salway@rokt.com>

---------

Signed-off-by: Jacob Salway <jacob.salway@rokt.com>
2024-04-19 22:10:20 +00:00
Vara Bonthu 4f26fe7b58
fix: Removed `publish-image` dependency on publishing the helm chart (#1995)
Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>
2024-04-19 07:06:47 +00:00
Vara Bonthu 4d4f1c5f8c
fix: Update Github workflow to publish Helm charts on chart changes, irrespective of image change (#1992)
* Update workflow to publish Helm charts on chart changes, irrespective of image updates

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* fixed the chart name with prefix

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

---------

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>
2024-04-19 04:31:47 +00:00
Duc Nguyen f6c9d64cd6
chore: Add Timo to user list (#1615) 2024-04-19 04:17:47 +00:00
Yi Chen 5e9c0b4013
Update spark operator permissions for CRD (#1973)
Signed-off-by: Yi Chen <github@chenyicn.net>
2024-04-17 21:07:30 +00:00
Aran Shavit c01e91da63
fix spark-rbac (#1986)
* remove non-existent field

Signed-off-by: Aran Shavit <aranshavit@gmail.com>

* bump version

Signed-off-by: Aran Shavit <aranshavit@gmail.com>

* README

Signed-off-by: Aran Shavit <aranshavit@gmail.com>

---------

Signed-off-by: Aran Shavit <aranshavit@gmail.com>
2024-04-17 19:49:27 +00:00
Andrey Velichkevich dede19052e
Use Kubeflow Docker Hub for Spark Operator Image (#1974)
* Use Kubeflow Docker Hub for Spark Operator Image

Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Bump Helm Chart version

Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

---------

Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
2024-04-17 00:10:26 +00:00
Andrii Chubatiuk 1d3e385706
fix: fixed serviceaccount annotations (#1972)
* fixed serviceaccount annotations

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* bump chart version

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* docs fix

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

---------

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-16 21:07:23 +00:00
Andrii Chubatiuk e9dbdfb971
fix: upgraded k8s deps (#1983)
Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-16 20:22:52 +00:00
Ajay Kemparaj 362b812355
chore: remove k8s.io/kubernetes replaces and adapt to v1.29.3 apis (#1968)
Signed-off-by: ajayk <ajaykemparaj@gmail.com>
2024-04-16 17:01:52 +00:00
Yi Chen 6ded3acd1c
Add some helm chart unit tests and fix spark service account render failure when extra annotations are specified (#1967)
* Add helm unit tests

Signed-off-by: Yi Chen <github@chenyicn.net>

* Fix: failed to render spark service account when extra annotations are specified

Signed-off-by: Yi Chen <github@chenyicn.net>

* Update developer guide

Signed-off-by: Yi Chen <github@chenyicn.net>

* Bump helm chart version

Signed-off-by: Yi Chen <github@chenyicn.net>

---------

Signed-off-by: Yi Chen <github@chenyicn.net>
2024-04-15 20:32:52 +00:00
Vara Bonthu 69e9841352
feat: Doc updates, Issue and PR templates are added (#1970)
* Issue templates are added to the repo

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* removed Google CLA requirement

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Updated ghcr.io registry references in the workflow

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Added Pull request template

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Updated Main README.md with Kubeflow header and new Slack channel link

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Removed the License header and it will be replaced with Kubeflow guidelines

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Revert "Removed the License header and it will be replaced with Kubeflow guidelines"

This reverts commit b892f5c7fa0398cff8b85f961bd292313ef47953.

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* Readme line revert for gcp docs

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* pre-commit run -a updates

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* fixed the helm lint issue by upgrading the Helm chart version

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* fixed docker image tag and updated chart docs (#1969)

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

* rebase from master

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>

---------

Signed-off-by: Vara Bonthu <vara.bonthu@gmail.com>
Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
Co-authored-by: Andrii Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-14 06:24:49 +00:00
Andrii Chubatiuk 39b79d733b
fixed docker image tag and updated chart docs (#1969)
Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-14 05:59:49 +00:00
Andrii Chubatiuk 76a7947816
publish chart independently, incremented both chart and image versions to trigger build of both (#1964)
* publish chart independently, incremented both chart and image versions to trigger build of both

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* bump chart version

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

---------

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-13 03:19:48 +00:00
Yi Chen 179ad8b1cf
Update helm chart README (#1958)
Signed-off-by: Yi Chen <github@chenyicn.net>
2024-04-13 03:18:48 +00:00
Zev Isert 59badbfed9
fix: add containerPort declaration for webhook in helm chart (#1961)
* fix: add containerPort declaration for webhook in helm chart

Signed-off-by: Zev Isert <dev@zevisert.ca>

* docs: update helm chart readme

Signed-off-by: Zev Isert <dev@zevisert.ca>

* fix: copied helm value should be for webhook

Signed-off-by: Zev Isert <dev@zevisert.ca>

* build: bump helm chart to 1.2.3

Signed-off-by: Zev Isert <dev@zevisert.ca>

* style: undo unrelated editor autoformatting

Signed-off-by: Zev Isert <dev@zevisert.ca>

---------

Signed-off-by: Zev Isert <dev@zevisert.ca>
Co-authored-by: Mason Legere <masonlegere@gmail.com>
2024-04-11 17:45:47 +00:00
Andrii Chubatiuk 4fd6a54b3c
added id for a build job to fix digests artifact creation (#1963)
* added id for a build job to fix digests artifact creation

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* configure user for helm publish

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

---------

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-11 16:52:47 +00:00
Andrii Chubatiuk 835281d4d3
support multiple namespaces (#1955)
* support multiple namespaces

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* bump helm chart version

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

---------

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-11 04:46:47 +00:00
Zev Isert 7e34e7ea1e
chore: replace GoogleCloudPlatform/spark-on-k8s-operator with kubeflow/spark-operator (#1937)
Signed-off-by: Zev Isert <dev@zevisert.ca>
2024-04-08 18:01:27 +00:00
Aakcht 1591668b2f
Chart: add patch permissions for spark operator SA to support spark 3.5.0 (#1884)
Signed-off-by: aakcht <aakcht@gmail.com>
2024-04-08 06:20:27 +00:00
Andrii Chubatiuk 4d1f5c5bc4
Cleanup after golang upgrade (#1956)
* cleanup after upgrade

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* pr comments

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

* feat: add support for custom service labels (#1952)

* feat: add support for custom service labels

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

* chore: correctly format files

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

* chore: bump chart version to 1.1.30

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

---------

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>
Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>

---------

Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
Signed-off-by: Cian Gallagher <cian@ciangallagher.net>
Co-authored-by: Cian Gallagher <cg1830@gmail.com>
2024-04-08 01:16:27 +00:00
Cian Gallagher 8a31a4638c
feat: add support for custom service labels (#1952)
* feat: add support for custom service labels

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

* chore: correctly format files

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

* chore: bump chart version to 1.1.30

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>

---------

Signed-off-by: Cian Gallagher <cian@ciangallagher.net>
2024-04-07 16:58:26 +00:00
Andrii Chubatiuk c9bfa81663
upgraded golang and dependencies (#1954)
Signed-off-by: Andrew Chubatiuk <andrew.chubatiuk@gmail.com>
2024-04-07 05:48:26 +00:00
shahsiddharth08 43f489dd9a
README for installing operator using kustomize with custom namespace and service name (#1778)
* Installing operator using kustomize and custom namespace and service name

* update quick start guide with suggested changes.
2024-04-06 18:43:25 +00:00
Ataf Fazledin Ahamed 14a39e63a0
BUGFIX: Added cancel method to fix context leak (#1917)
Signed-off-by: fazledyn-or <ataf@openrefactory.com>
2024-04-06 17:34:25 +00:00
Taeyeop Kim 0dd08b220e
remove unmatched quotes from user-guide.md (#1584) 2024-04-06 17:33:25 +00:00
Yang 717a3eccb1
Add PVC permission to Operator role (#1889) 2024-04-06 17:32:25 +00:00
karbyshevds ec1c2e314f
Allow to set webhook job resource limits (#1429,#1300) (#1946)
Signed-off-by: Dmitriy Karbyshev <karbyshevds@gmail.com>
2024-04-02 16:17:22 +00:00
James Liu 547805253b
Merge pull request #1927 from kubeflow/zijianjoy-patch-1
Create OWNERS
2024-03-16 16:10:31 -07:00
James Liu 0a44823002
Create OWNERS
https://github.com/kubeflow/community/issues/684
2024-03-15 10:20:01 -07:00
disaster37 8cb6c8035e
fix: fix issue #1723 about spark-operator not working with volcano on OCP (#1724)
* fix: fix issue #1723 about spark-operator not working with volcano on OCP

Signed-off-by: disaster37 <linuxworkgroup@hotmail.com>

* Update volcano_scheduler.go

---------

Signed-off-by: disaster37 <linuxworkgroup@hotmail.com>
2023-11-09 12:43:12 -08:00
Jacob Salway b49b1b48d8
Add Rokt to who-is-using.md (#1867) 2023-11-09 12:34:20 -08:00
Wilton Rodrigues cd78251daf
Handle invalid API resources in discovery (#1758) 2023-10-27 12:17:38 -07:00
Volodymyr Kot e146848959
Fix docs for Volcano integration (#1719)
Co-authored-by: Volodymyr Kot <volodymyrkot.palantir@gmail.com>
2023-10-26 09:33:58 -07:00
Jose Soto 678d313650
Add qualytics to who is using (#1736)
Co-authored-by: Yinan Li <ynli@google.com>
2023-10-26 09:31:55 -07:00
Xuefeng Chen f21a4e51e7
allowing optional annotation on rbac (#1770) 2023-10-26 09:31:03 -07:00
ordukhanian 86d71c828b
* Added support for `seccompProfile` in Spark application CRD. It is necessary for Kubernetes Pod Security Standards Restricted profile. (#1768)
https://kubernetes.io/docs/concepts/security/pod-security-standards/#restricted

* Fixed pre-commit jobs.
  `build-helm-chart` and `integration-test` were failing with:
  `Run manusa/actions-setup-minikube@v2.4.2
   Error: Unsupported OS, action only works in Ubuntu 18 or 20`
2023-10-26 09:30:27 -07:00
JunaidChaudry 93a9aa88df
Updating webhook docs to also mention eks (#1763) 2023-10-26 09:29:29 -07:00
Alexander Ryabov 223ec8d317
Link to helm upgrade docs fixed (#1783) 2023-10-26 09:26:13 -07:00
Kohei Watanabe 463ed1ec7f
Improve getMasterURL() to add [] to IPv6 if needed (#1825)
Resolves #1344

Spark 3.4 supports IPv6:
- https://github.com/apache/spark/pull/36868

So I want to make the operator support IPv6.

I can confirm that this can submit the spark-job in IPv6-only environment.

Although it is necessary to add the following environment variables to the operator

```yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: spark-on-k8s-spark-operator
spec:
  template:
    spec:
      containers:
      - name: spark-operator
        env:
        - name: _JAVA_OPTIONS
          value: "-Djava.net.preferIPv6Addresses=true"
        - name: KUBERNETES_DISABLE_HOSTNAME_VERIFICATION
          value: "true"

```
2023-10-26 09:25:55 -07:00
Mat Schaffer 1b842a8ab2
Add envFrom to operator deployment (#1785)
* Add envFrom to operator deployment

Useful to when env vars are used for auth when downloading `spark.archives` from S3.

* Fix over-indenting
2023-10-26 09:25:11 -07:00
Mat Schaffer 0ad755c1eb
Expand ingress docs a bit (#1806)
This notes the controller intended to be used with the operator-managed ingress resources.

When setting up my own cluster I also tried https://docs.nginx.com/nginx-ingress-controller/ and https://kubernetes-sigs.github.io/aws-load-balancer-controller/ but the path format generated by the operator won't work with those.
2023-10-26 09:24:56 -07:00