Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dns policy for sandbox buildkit instance to ClusterFirstWithHo… #4678

Merged
merged 1 commit into from
Jan 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions charts/flyte-sandbox/templates/buildkit/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ spec:
metadata:
labels: {{- include "flyte-sandbox.buildkitSelectorLabels" . | nindent 8 }}
spec:
dnsPolicy: ClusterFirstWithHostNet
hostNetwork: true
containers:
- name: buildkit
Expand Down
3 changes: 2 additions & 1 deletion docker/sandbox-bundled/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,9 @@ flyte:
.PHONY: manifests
manifests:
mkdir -p manifests
helm dependency update ../../charts/flyte-sandbox
helm dependency update ../../charts/flyteagent
helm dependency update ../../charts/flyte-binary
helm dependency update ../../charts/flyte-sandbox
kustomize build \
--enable-helm \
--load-restrictor=LoadRestrictionsNone \
Expand Down
5 changes: 3 additions & 2 deletions docker/sandbox-bundled/manifests/complete-agent.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -816,7 +816,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: SlF2M0tXNXdGTUpHZzdvNg==
haSharedSecret: RkZLd2xsVEZSNXNHMExjeg==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1388,6 +1388,7 @@ spec:
periodSeconds: 30
securityContext:
privileged: true
dnsPolicy: ClusterFirstWithHostNet
hostNetwork: true
---
apiVersion: apps/v1
Expand All @@ -1411,7 +1412,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: b76914839daf95717da7a8ef31769c3b97f6c3c46a09776e5d0f8cc3fe9b1e9a
checksum/secret: 54b8e8ed67f9cf2f4b8c83b917ba11b0ed9f81f453df70376c332e202522643e
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
5 changes: 3 additions & 2 deletions docker/sandbox-bundled/manifests/complete.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -796,7 +796,7 @@ type: Opaque
---
apiVersion: v1
data:
haSharedSecret: QU5jN3V6ZjA4Y0tFbzZ0dw==
haSharedSecret: SENrVEZGc09ob3RKdFJRSA==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -1336,6 +1336,7 @@ spec:
periodSeconds: 30
securityContext:
privileged: true
dnsPolicy: ClusterFirstWithHostNet
hostNetwork: true
---
apiVersion: apps/v1
Expand All @@ -1359,7 +1360,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 436cdf2d6630635a2652233c83f71c617facb2fda8edabb639ab7dfa0f15e46f
checksum/secret: 7c61351bdfcc86e008feb9f8d023e02d3aa7e570e1759790f8d7c3f6a4fd9c86
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
5 changes: 3 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: MGNlbmFPenJydE1hNlB0Sw==
haSharedSecret: RjVjd3lFdXVJRllyTzlTNA==
proxyPassword: ""
proxyUsername: ""
kind: Secret
Expand Down Expand Up @@ -910,6 +910,7 @@ spec:
periodSeconds: 30
securityContext:
privileged: true
dnsPolicy: ClusterFirstWithHostNet
hostNetwork: true
---
apiVersion: apps/v1
Expand All @@ -933,7 +934,7 @@ spec:
metadata:
annotations:
checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
checksum/secret: 05cc4dc3bf13796e3a35e79da36baf286c81f80b207628f1f64197c80a04130c
checksum/secret: 048f8e7e067f646dce51250b72fe31919755c8fa62f7f4f4ccc054e20a719d9e
labels:
app: docker-registry
release: flyte-sandbox
Expand Down
Loading