Skip to content

Commit

Permalink
Helm - Add kubernetes dashboard and update deployments
Browse files Browse the repository at this point in the history
Based on the following kustomize changes:
- a14fd47 Update deployments with latest k8s library deps (#826)
- f149af8 Add kubernetes dashboard to sandbox overlay (#789)

Signed-off-by: Sören Brunk <[email protected]>
Signed-off-by: Haytham Abuelfutuh <[email protected]>
  • Loading branch information
sbrunk authored and EngHabu committed May 28, 2021
1 parent 08a4581 commit ddbdaeb
Show file tree
Hide file tree
Showing 7 changed files with 27 additions and 18 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,4 @@ docs/flytekit/flytekit.interfaces.html
docs/searchindex.js
docs/
__pycache__/
/helm/charts/
5 changes: 3 additions & 2 deletions helm/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,6 @@ dependencies:
version: 1.0.6
repository: https://googlecloudplatform.github.io/spark-on-k8s-operator
condition: spark.enabled


- name: kubernetes-dashboard
version: 4.0.2
repository: https://kubernetes.github.io/dashboard/
1 change: 0 additions & 1 deletion helm/templates/admin/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ spec:
containers:
- command:
- flyteadmin
- --logtostderr
- --config
- {{ .Values.flyteadmin.configPath }}
- clusterresource
Expand Down
4 changes: 0 additions & 4 deletions helm/templates/admin/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ spec:
{{- end }}
- command:
- flyteadmin
- --logtostderr
- --config
- {{ .Values.flyteadmin.configPath }}
- migrate
Expand All @@ -41,7 +40,6 @@ spec:
name: config-volume
- command:
- flyteadmin
- --logtostderr
- --config
- {{ .Values.flyteadmin.configPath }}
- migrate
Expand All @@ -57,7 +55,6 @@ spec:
name: config-volume
- command:
- flyteadmin
- --logtostderr
- --config
- {{ .Values.flyteadmin.configPath }}
- clusterresource
Expand All @@ -73,7 +70,6 @@ spec:
containers:
- command:
- flyteadmin
- --logtostderr
- --config
- {{ .Values.flyteadmin.configPath }}
{{- with .Values.flyteadmin.extraArgs }}
Expand Down
2 changes: 0 additions & 2 deletions helm/templates/datacatalog/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ spec:
initContainers:
- command:
- datacatalog
- --logtostderr
- --config
- {{ .Values.datacatalog.configPath }}
- migrate
Expand All @@ -34,7 +33,6 @@ spec:
containers:
- command:
- datacatalog
- --logtostderr
- --config
- {{ .Values.datacatalog.configPath }}
{{- with .Values.datacatalog.extraArgs }}
Expand Down
15 changes: 14 additions & 1 deletion helm/values-sandbox.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,19 @@ sparkoperator:
pytorchoperator:
enabled: false

#
# KUBERNETES DASHBOARD
#
kubernetes-dashboard:
extraArgs:
- --enable-skip-login
- --enable-insecure-login
- --disable-settings-authorizer
protocolHttp: true
service:
type: NodePort
externalPort: 30082

#
# COMMON
#
Expand Down Expand Up @@ -138,7 +151,7 @@ configmap:
plugins:
logs:
kubernetes-enabled: true
kubernetes-url: http://localhost:30082
kubernetes-template-uri: "http://localhost:30082/#/log/{{ .namespace }}/{{ .podName }}/pod?namespace={{ .namespace }}"

logger:
logger:
Expand Down
17 changes: 9 additions & 8 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ flyteadmin:
replicaCount: 1
image:
# -- Docker image for Flyteadmin deployment
repository: ghcr.io/lyft/flyteadmin
tag: v0.3.29
repository: ghcr.io/flyteorg/flyteadmin
tag: v0.3.38
pullPolicy: IfNotPresent
# -- Default resources requests and limits for Flyteadmin deployment
resources:
Expand Down Expand Up @@ -48,8 +48,8 @@ datacatalog:
replicaCount: 1
image:
# -- Docker image for Datacatalog deployment
repository: ghcr.io/lyft/datacatalog
tag: v0.2.130
repository: ghcr.io/flyteorg/datacatalog
tag: v0.3.0
pullPolicy: IfNotPresent
# -- Default resources requests and limits for Datacatalog deployment
resources:
Expand Down Expand Up @@ -89,7 +89,7 @@ flytepropeller:
image:
# -- Docker image for Flytepropeller deployment
repository: ghcr.io/flyteorg/flytepropeller
tag: v0.5.13
tag: v0.7.1
pullPolicy: IfNotPresent
# -- Default resources requests and limits for Flytepropeller deployment
resources:
Expand Down Expand Up @@ -123,8 +123,8 @@ flyteconsole:
replicaCount: 1
image:
# -- Docker image for Flyteconsole deployment
repository: ghcr.io/lyft/flyteconsole
tag: v0.19.1
repository: ghcr.io/flyteorg/flyteconsole
tag: v0.19.6
pullPolicy: IfNotPresent
# -- Default resources requests and limits for Flyteconsole deployment
resources:
Expand Down Expand Up @@ -361,7 +361,8 @@ common:
ingress:
# --- enable or disable creating Ingress for Flyte. Relevant to disable when using e.g. Istio as ingress controller.
enabled: true
annotations: {}
annotations:
nginx.ingress.kubernetes.io/ssl-redirect: "false"
flyteNamespaceTemplate:
# --- enable or disable creating Flyte namespace in template. Enable when using helm as template-engine only. Disable when using `helm install ...`.
enabled: false
Expand Down

0 comments on commit ddbdaeb

Please sign in to comment.