Skip to content

Commit

Permalink
Enable echo plugin by default (#5679)
Browse files Browse the repository at this point in the history
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: pmahindrakar-oss <[email protected]>
  • Loading branch information
pingsutw authored and pmahindrakar-oss committed Sep 9, 2024
1 parent 5adcda3 commit 29044ff
Show file tree
Hide file tree
Showing 24 changed files with 52 additions and 32 deletions.
6 changes: 3 additions & 3 deletions charts/flyte-binary/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,9 @@ Chart for basic single Flyte executable deployment
| deployment.waitForDB.image.pullPolicy | string | `"IfNotPresent"` | |
| deployment.waitForDB.image.repository | string | `"postgres"` | |
| deployment.waitForDB.image.tag | string | `"15-alpine"` | |
| enabled_plugins.tasks | object | `{"task-plugins":{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service"]}}` | Tasks specific configuration [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#GetConfig) |
| enabled_plugins.tasks.task-plugins | object | `{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service"]}` | Plugins configuration, [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#TaskPluginConfig) |
| enabled_plugins.tasks.task-plugins.enabled-plugins | list | `["container","sidecar","k8s-array","agent-service"]` | [Enabled Plugins](https://pkg.go.dev/github.com/lyft/flyteplugins/go/tasks/config#Config). Enable sagemaker*, athena if you install the backend plugins |
| enabled_plugins.tasks | object | `{"task-plugins":{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service","echo"]}}` | Tasks specific configuration [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#GetConfig) |
| enabled_plugins.tasks.task-plugins | object | `{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service","echo"]}` | Plugins configuration, [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#TaskPluginConfig) |
| enabled_plugins.tasks.task-plugins.enabled-plugins | list | `["container","sidecar","k8s-array","agent-service","echo"]` | [Enabled Plugins](https://pkg.go.dev/github.com/lyft/flyteplugins/go/tasks/config#Config). Enable sagemaker*, athena if you install the backend plugins |
| flyte-core-components.admin.disableClusterResourceManager | bool | `false` | |
| flyte-core-components.admin.disableScheduler | bool | `false` | |
| flyte-core-components.admin.disabled | bool | `false` | |
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-binary/eks-production.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ configuration:
- sidecar
- K8S-ARRAY #used for MapTasks
- agent-service
- echo
default-for-task-types:
- container: container
- container_array: K8S-ARRAY
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-binary/eks-starter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ configuration:
- sidecar
- K8S-ARRAY #used for MapTasks
- agent-service
- echo
default-for-task-types:
- container: container
- container_array: K8S-ARRAY
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-binary/gke-starter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,7 @@ configuration:
- container
- sidecar
- K8S-ARRAY #used for MapTasks
- echo
default-for-task-types:
- container: container
- container_array: K8S-ARRAY
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-binary/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -399,6 +399,7 @@ enabled_plugins:
- sidecar
- k8s-array
- agent-service
- echo
default-for-task-types:
container: container
sidecar: sidecar
Expand Down
6 changes: 3 additions & 3 deletions charts/flyte-core/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,9 @@ helm install gateway bitnami/contour -n flyte
| configmap.core.propeller | object | `{"downstream-eval-duration":"30s","enable-admin-launcher":true,"leader-election":{"enabled":true,"lease-duration":"15s","lock-config-map":{"name":"propeller-leader","namespace":"flyte"},"renew-deadline":"10s","retry-period":"2s"},"limit-namespace":"all","max-workflow-retries":30,"metadata-prefix":"metadata/propeller","metrics-prefix":"flyte","prof-port":10254,"queue":{"batch-size":-1,"batching-interval":"2s","queue":{"base-delay":"5s","capacity":1000,"max-delay":"120s","rate":100,"type":"maxof"},"sub-queue":{"capacity":100,"rate":10,"type":"bucket"},"type":"batch"},"rawoutput-prefix":"s3://my-s3-bucket/","workers":4,"workflow-reeval-duration":"30s"}` | follows the structure specified [here](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/config). |
| configmap.datacatalogServer | object | `{"application":{"grpcPort":8089,"grpcServerReflection":true,"httpPort":8080},"datacatalog":{"heartbeat-grace-period-multiplier":3,"max-reservation-heartbeat":"30s","metrics-scope":"datacatalog","profiler-port":10254,"storage-prefix":"metadata/datacatalog"}}` | Datacatalog server config |
| configmap.domain | object | `{"domains":[{"id":"development","name":"development"},{"id":"staging","name":"staging"},{"id":"production","name":"production"}]}` | Domains configuration for Flyte projects. This enables the specified number of domains across all projects in Flyte. |
| configmap.enabled_plugins.tasks | object | `{"task-plugins":{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service"]}}` | Tasks specific configuration [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#GetConfig) |
| configmap.enabled_plugins.tasks.task-plugins | object | `{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service"]}` | Plugins configuration, [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#TaskPluginConfig) |
| configmap.enabled_plugins.tasks.task-plugins.enabled-plugins | list | `["container","sidecar","k8s-array","agent-service"]` | [Enabled Plugins](https://pkg.go.dev/github.com/lyft/flyteplugins/go/tasks/config#Config). Enable sagemaker*, athena if you install the backend plugins |
| configmap.enabled_plugins.tasks | object | `{"task-plugins":{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service","echo"]}}` | Tasks specific configuration [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#GetConfig) |
| configmap.enabled_plugins.tasks.task-plugins | object | `{"default-for-task-types":{"container":"container","container_array":"k8s-array","sidecar":"sidecar"},"enabled-plugins":["container","sidecar","k8s-array","agent-service","echo"]}` | Plugins configuration, [structure](https://pkg.go.dev/github.com/flyteorg/flytepropeller/pkg/controller/nodes/task/config#TaskPluginConfig) |
| configmap.enabled_plugins.tasks.task-plugins.enabled-plugins | list | `["container","sidecar","k8s-array","agent-service","echo"]` | [Enabled Plugins](https://pkg.go.dev/github.com/lyft/flyteplugins/go/tasks/config#Config). Enable sagemaker*, athena if you install the backend plugins |
| configmap.k8s | object | `{"plugins":{"k8s":{"default-cpus":"100m","default-env-vars":[],"default-memory":"100Mi"}}}` | Kubernetes specific Flyte configuration |
| configmap.k8s.plugins.k8s | object | `{"default-cpus":"100m","default-env-vars":[],"default-memory":"100Mi"}` | Configuration section for all K8s specific plugins [Configuration structure](https://pkg.go.dev/github.com/lyft/flyteplugins/go/tasks/pluginmachinery/flytek8s/config) |
| configmap.remoteData.remoteData.region | string | `"us-east-1"` | |
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-core/values-eks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -282,6 +282,7 @@ configmap:
- sidecar
- k8s-array
- agent-service
- echo
# - sagemaker_hyperparameter_tuning
# - sagemaker_custom_training
# - sagemaker_training
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-core/values-gcp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,7 @@ configmap:
- sidecar
- k8s-array
- agent-service
- echo
default-for-task-types:
container: container
sidecar: sidecar
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -674,6 +674,7 @@ configmap:
- sidecar
- k8s-array
- agent-service
- echo
default-for-task-types:
container: container
sidecar: sidecar
Expand Down
1 change: 1 addition & 0 deletions charts/flyte-core/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -863,6 +863,7 @@ configmap:
- sidecar
- k8s-array
- agent-service
- echo
default-for-task-types:
container: container
sidecar: sidecar
Expand Down
10 changes: 5 additions & 5 deletions charts/flyte/README.md

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions charts/flyte/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -528,6 +528,7 @@ flyte:
- sidecar
- k8s-array
- agent-service
- echo
default-for-task-types:
container: container
sidecar: sidecar
Expand Down
5 changes: 3 additions & 2 deletions deployment/eks/flyte_aws_scheduler_helm_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -491,6 +491,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -1277,7 +1278,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -1361,7 +1362,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
5 changes: 3 additions & 2 deletions deployment/eks/flyte_helm_dataplane_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -428,7 +429,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -512,7 +513,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
5 changes: 3 additions & 2 deletions deployment/eks/flyte_helm_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -522,6 +522,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -1407,7 +1408,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -1491,7 +1492,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "cf07450d68fd5fcd2055e08e48664a18edabf2e13904b93096f2f6aa7ea7f5c"
configChecksum: "33bc4dd986fdb015ce49d998deedf122e119579ec09db311e67276230678a70"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
5 changes: 3 additions & 2 deletions deployment/gcp/flyte_helm_dataplane_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -436,7 +437,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "3ceda510332c8f7f6cd2cdd75234ef366e2b31dfe0632a2a93cc25bc326cf28"
configChecksum: "64a3f5e546eddd8126d03c005460b964f428a0da31c0bfac5f70c63fbf3d635"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -519,7 +520,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "3ceda510332c8f7f6cd2cdd75234ef366e2b31dfe0632a2a93cc25bc326cf28"
configChecksum: "64a3f5e546eddd8126d03c005460b964f428a0da31c0bfac5f70c63fbf3d635"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
5 changes: 3 additions & 2 deletions deployment/gcp/flyte_helm_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -535,6 +535,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -1430,7 +1431,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "3ceda510332c8f7f6cd2cdd75234ef366e2b31dfe0632a2a93cc25bc326cf28"
configChecksum: "64a3f5e546eddd8126d03c005460b964f428a0da31c0bfac5f70c63fbf3d635"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -1513,7 +1514,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "3ceda510332c8f7f6cd2cdd75234ef366e2b31dfe0632a2a93cc25bc326cf28"
configChecksum: "64a3f5e546eddd8126d03c005460b964f428a0da31c0bfac5f70c63fbf3d635"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
plugins:
logs:
kubernetes-enabled: false
Expand Down Expand Up @@ -359,7 +360,7 @@ spec:
app.kubernetes.io/instance: flyte
app.kubernetes.io/component: flyte-binary
annotations:
checksum/configuration: f0ece2b70412090c94dd394c3b7c213edac6e7e6d53b0696ce1b1e748176bb45
checksum/configuration: 1dd761465a57869e165697d10e35a81beeb988253055d6a74ed80aa5e3e5e106
checksum/configuration-secret: d5d93f4e67780b21593dc3799f0f6682aab0765e708e4020939975d14d44f929
checksum/cluster-resource-templates: 7dfa59f3d447e9c099b8f8ffad3af466fecbc9cf9f8c97295d9634254a55d4ae
spec:
Expand Down
5 changes: 3 additions & 2 deletions deployment/sandbox/flyte_helm_generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -643,6 +643,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
k8s.yaml: |
plugins:
k8s:
Expand Down Expand Up @@ -7181,7 +7182,7 @@ spec:
template:
metadata:
annotations:
configChecksum: "b535173fc46161f52db8a753f6b82289cdf866def58cd388585d9a78c19363f"
configChecksum: "84d449758d51dc641aff55ae07f4376a860b5038e8407cb9d2444c4f895d953"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
labels:
Expand Down Expand Up @@ -7257,7 +7258,7 @@ spec:
app.kubernetes.io/name: flyte-pod-webhook
app.kubernetes.io/version: v1.13.1
annotations:
configChecksum: "b535173fc46161f52db8a753f6b82289cdf866def58cd388585d9a78c19363f"
configChecksum: "84d449758d51dc641aff55ae07f4376a860b5038e8407cb9d2444c4f895d953"
prometheus.io/path: "/metrics"
prometheus.io/port: "10254"
spec:
Expand Down
7 changes: 4 additions & 3 deletions docker/sandbox-bundled/manifests/complete-agent.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -461,6 +461,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
plugins:
logs:
kubernetes-enabled: true
Expand Down Expand Up @@ -817,7 +818,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: czRSTHlPalFoaXh2eG8ybQ==
haSharedSecret: cWlOc1c1bnl5ZGI3YTlzSw==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1248,7 +1249,7 @@ spec:
metadata:
annotations:
checksum/cluster-resource-templates: 6fd9b172465e3089fcc59f738b92b8dc4d8939360c19de8ee65f68b0e7422035
checksum/configuration: c947a76cd5e9d59db63c98247b8944d64351090c65c4741d0ac30b6d17bde944
checksum/configuration: 6e8a4cc6177037f26cee65d09c37c010437ea3f0989a2a2dfef380fed9f468c2
checksum/configuration-secret: 09216ffaa3d29e14f88b1f30af580d02a2a5e014de4d750b7f275cc07ed4e914
labels:
app.kubernetes.io/component: flyte-binary
Expand Down Expand Up @@ -1414,7 +1415,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 26c674111980a6c187443e32ee29419ae3678da338eabcc88055efe5b975736a
checksum/secret: 7f8247a0b84f43018fdf11a598132b8a67ed9fde6573ffce801b725a6f955012
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
7 changes: 4 additions & 3 deletions docker/sandbox-bundled/manifests/complete.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,7 @@ data:
- sidecar
- k8s-array
- agent-service
- echo
plugins:
logs:
kubernetes-enabled: true
Expand Down Expand Up @@ -797,7 +798,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: bVZ2dDVDNDBYYUZUWDhaOQ==
haSharedSecret: UUxqaW5SeGlBbFNlQzVoag==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1195,7 +1196,7 @@ spec:
metadata:
annotations:
checksum/cluster-resource-templates: 6fd9b172465e3089fcc59f738b92b8dc4d8939360c19de8ee65f68b0e7422035
checksum/configuration: dcece0d233748108b134ff58a6c1f2c5eafb960e009c8a55b90734ffc884a435
checksum/configuration: 967349c227efb6765bb7509d14dc7b0d62b07904a337dd70c8682d52d870590a
checksum/configuration-secret: 09216ffaa3d29e14f88b1f30af580d02a2a5e014de4d750b7f275cc07ed4e914
labels:
app.kubernetes.io/component: flyte-binary
Expand Down Expand Up @@ -1361,7 +1362,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: bd5c554c59981c4aeac7afc8c548d72de08d84f5a0c3b874d0122c423370f51a
checksum/secret: bea0c8f293b54e309a353e0e8563e709ad817d372d2b1dce1114188693aa3f12
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
4 changes: 2 additions & 2 deletions docker/sandbox-bundled/manifests/dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,7 @@ metadata:
---
apiVersion: v1
data:
haSharedSecret: Tmp2T0hScHN2NFJtWUxKVQ==
haSharedSecret: ZmdJNWs5RUg4cWNVTVBzRw==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -934,7 +934,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 599e28a6ad1e5b63c91487bf3fea5d5b413b9ea50a0262e0ac6b00c65db92dc1
checksum/secret: a896f2c43dff6c05c154b51e4c9ec21c9e2f03ecaf4c1fed045d84523219cf63
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
1 change: 1 addition & 0 deletions flyte-single-binary-local.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ tasks:
- container
- sidecar
- K8S-ARRAY
- echo
default-for-task-types:
- container: container
- container_array: K8S-ARRAY
Expand Down
1 change: 1 addition & 0 deletions flytepropeller/propeller-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ tasks:
- container
- sidecar
- K8S-ARRAY
- echo
# Uncomment to enable sagemaker plugin
# - sagemaker_training
# - sagemaker_hyperparameter_tuning
Expand Down

0 comments on commit 29044ff

Please sign in to comment.