Skip to content

Commit

Permalink
Update agent endpoint in the single binary (#3779)
Browse files Browse the repository at this point in the history
* Update agnet endpoint in the single binary

Signed-off-by: Kevin Su <[email protected]>

* Update sandbox

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* nitt

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

---------

Signed-off-by: Kevin Su <[email protected]>
  • Loading branch information
pingsutw authored Jun 13, 2023
1 parent 47fea41 commit 1ae254f
Show file tree
Hide file tree
Showing 7 changed files with 14 additions and 8 deletions.
2 changes: 1 addition & 1 deletion charts/flyte-binary/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ Chart for basic single Flyte executable deployment
| clusterResourceTemplates.labels | object | `{}` | |
| commonAnnotations | object | `{}` | |
| commonLabels | object | `{}` | |
| configuration.agentService.defaultGrpcEndpoint | string | `"flyte-sandbox-http.flyte.svc.cluster.local:8000"` | |
| configuration.agentService.defaultGrpcEndpoint | string | `"flyte-binary-http.flyte.svc.cluster.local:8000"` | |
| configuration.agentService.supportedTaskTypes[0] | string | `"bigquery_query_job_task"` | |
| configuration.annotations | object | `{}` | |
| configuration.auth.authorizedUris | list | `[]` | |
Expand Down
2 changes: 1 addition & 1 deletion charts/flyte-binary/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ configuration:
tag: v0.0.30 # FLYTECOPILOT_TAG
# Flyteagent Configuration
agentService:
defaultGrpcEndpoint: flyte-sandbox-http.flyte.svc.cluster.local:8000
defaultGrpcEndpoint: flyte-binary-http.flyte.svc.cluster.local:8000
supportedTaskTypes:
- bigquery_query_job_task
# externalConfigMap Specify an existing, external ConfigMap to use as configuration for Flyte
Expand Down
2 changes: 2 additions & 0 deletions charts/flyte-sandbox/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ A Helm chart for the Flyte local sandbox
| docker-registry.service.nodePort | int | `30000` | |
| docker-registry.service.type | string | `"NodePort"` | |
| flyte-binary.clusterResourceTemplates.inlineConfigMap | string | `"{{ include \"flyte-sandbox.clusterResourceTemplates.inlineConfigMap\" . }}"` | |
| flyte-binary.configuration.agentService.defaultGrpcEndpoint | string | `"flyte-sandbox-http.flyte.svc.cluster.local:8000"` | |
| flyte-binary.configuration.agentService.supportedTaskTypes[0] | string | `"bigquery_query_job_task"` | |
| flyte-binary.configuration.database.host | string | `"{{ printf \"%s-postgresql\" .Release.Name | trunc 63 | trimSuffix \"-\" }}"` | |
| flyte-binary.configuration.database.password | string | `"postgres"` | |
| flyte-binary.configuration.inline.plugins.k8s.default-env-vars[0].FLYTE_AWS_ENDPOINT | string | `"http://{{ printf \"%s-minio\" .Release.Name | trunc 63 | trimSuffix \"-\" }}.{{ .Release.Namespace }}:9000"` | |
Expand Down
4 changes: 4 additions & 0 deletions charts/flyte-sandbox/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ flyte-binary:
database:
host: '{{ printf "%s-postgresql" .Release.Name | trunc 63 | trimSuffix "-" }}'
password: postgres
agentService:
defaultGrpcEndpoint: flyte-sandbox-http.flyte.svc.cluster.local:8000
supportedTaskTypes:
- bigquery_query_job_task
storage:
metadataContainer: my-s3-bucket
userDataContainer: my-s3-bucket
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ data:
cloudwatch-enabled: false
stackdriver-enabled: false
agent-service:
defaultGrpcEndpoint: flyte-sandbox-http.flyte.svc.cluster.local:8000
defaultGrpcEndpoint: flyte-binary-http.flyte.svc.cluster.local:8000
supportedTaskTypes:
- bigquery_query_job_task
002-database.yaml: |
Expand Down Expand Up @@ -350,7 +350,7 @@ spec:
app.kubernetes.io/name: flyte-binary
app.kubernetes.io/instance: flyte
annotations:
checksum/configuration: d0eb5481982cd41bc10637aa4ef0f2356e4c5a62a0a32e1a9f361ab915460b15
checksum/configuration: ad26141a590bda863600191912e25b9ff98789a3c07a7932d13ccf90218ed1b5
checksum/cluster-resource-templates: 7dfa59f3d447e9c099b8f8ffad3af466fecbc9cf9f8c97295d9634254a55d4ae
spec:
serviceAccountName: flyte-flyte-binary
Expand Down
4 changes: 2 additions & 2 deletions docker/sandbox-bundled/manifests/complete.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -784,7 +784,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: Wm9NZkpoUExVeHRIZVFXdQ==
haSharedSecret: ZHdWUkRjZ0c3eG96Q2pmdw==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1299,7 +1299,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 1877f7ae4570a2829bc6ddeccc1fe2e2908839e31778233f2c94cc9ed5945f5b
checksum/secret: 38bb3051ec8a9a1187a0b4bce306dd83e331e5ba23a719e905996a1db214d770
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: YzNLTWRRM2JrYnU5eElvUg==
haSharedSecret: RVlCdFpwQTI4SlVZYllzSg==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -875,7 +875,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 0276ad91431541e35a00efa6a9deb2414112a1328c2a842983d8a682ab005c67
checksum/secret: 06215b00158e7a4db172bc01358fdc3846e476f94d0e653acc9aefedd4a765fc
labels:
app: docker-registry
release: flyte-sandbox
Expand Down

0 comments on commit 1ae254f

Please sign in to comment.