Hey all, I put together this repo with a bunch of ...
# flyte-deployment
h
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
🔥 4
flyte 4
🙇 4
🙇🏽 1
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.
g
This is awesome. Thanks for sharing
h
no prob. hopefully it helps people wade through it a little easier.
t
Wow! Thanks for putting together a guide, Alex! Really appreciate it. @broad-monitor-993, shall we add a link to this in our deployment guide somewhere?
h
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
t
Oh okay. Please leave a comment on the PR. We'll make sure to incorporate your suggestion!
h
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
t
The extra lines of config you added to your repo aren't present in the flyte helm chart. @thankful-minister-83577, is this something we need to add to the flyte helm chart? @hallowed-doctor-67759, please feel free to create a PR meanwhile.
h
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.
a
this is great @hallowed-doctor-67759 thank you. I will use it and report back any findings
160 Views