https://flyte.org logo
#flyte-deployment
Title
# flyte-deployment
a

Alex Papanicolaou

03/05/2023, 5:51 AM
Hey all, I put together this repo with a bunch of tooling to get the multi-cluster deployment operational on EKS. https://github.com/alexifm/flyte-eks-deployment
I actually haven’t tested it much so it’s probably riddled with bugs. It’s a cleanup and refactoring of what we’ve had success with so I expect flaws to be minor and not major issues with the logic.
k

Kevin Su

03/05/2023, 10:46 PM
This is awesome. Thanks for sharing
a

Alex Papanicolaou

03/06/2023, 2:31 AM
no prob. hopefully it helps people wade through it a little easier.
s

Samhita Alla

03/06/2023, 3:58 AM
Wow! Thanks for putting together a guide, Alex! Really appreciate it. @Niels Bantilan, shall we add a link to this in our deployment guide somewhere?
a

Alex Papanicolaou

03/06/2023, 4:01 AM
I’m commenting on this PR now: https://github.com/flyteorg/flyte/pull/3363 I think there’s a lot of disparate sources that have shades of correctness or were correct at some point in time. For example, I found the below lines to be incorrect (searched the
values.yaml
and the referenced part didn’t exist). Also, if you see the
values-auth.yaml
file I constructed, this part of the PR appears inadequate for getting Auth working. https://github.com/flyteorg/flyte/pull/3363/files#diff-ee77fbe074a2e541ba69e70ac154aa8fcffcdc3699cb4f0abc8379e4ee2a3b9bR191-R195
s

Samhita Alla

03/06/2023, 4:04 AM
Oh okay. Please leave a comment on the PR. We'll make sure to incorporate your suggestion!
a

Alex Papanicolaou

03/06/2023, 4:07 AM
Also, if you see in my repo, I will need to submit a PR to the main flyte repo because the helm chart was insufficient (maybe it’s been updated, haven’t checked in the last week or two) https://github.com/alexifm/flyte-eks-deployment/commit/69e327734acf0bca67cd89dc62e7c26e8ca7a9c9
s

Samhita Alla

03/06/2023, 4:13 AM
The extra lines of config you added to your repo aren't present in the flyte helm chart. @Yee, is this something we need to add to the flyte helm chart? @Alex Papanicolaou, please feel free to create a PR meanwhile.
a

Alex Papanicolaou

03/06/2023, 4:21 AM
Right. That commit is a quick hack I put in because I was finding the the
additionalVolumes
and
additionalVolumeMounts
described in the
values-override.yaml
file in this section weren’t working out of the box. I was getting errors on some of the containers/init-containers for
flyteadmin
and `clusterresourcesync`and it was due to the secrets not being mounted in enough places.
d

David Espejo (he/him)

03/07/2023, 12:13 AM
this is great @Alex Papanicolaou thank you. I will use it and report back any findings
11 Views