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
fix: smooth out v2 dev process #1608
Conversation
This is not a "finished product", I am simply doing the bare minimum to unblock people and make life easier while I tidy some things up. Here I am just adding some more manifests to the Tiltfile; I have stashed these in `tools/dev-manifests` for now, but this may change. Either way people will not notice as they are just running `make cluster-dev`. The manifests under `manifests/wego-dev` are technically out of use as we move onto a Helm Chart install flow, but I am leaving them there for now as I tinker, plus I know other people are working on the Helm bit so I am staying out of their way. I have also left the SuperUser and the "stripping out the service account" bit for now, as they do not need to be in place to unblock people. They are next on my list.
51b6480
to
28cddb8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The allow everything because why not, #yolo
seems a bit too much, but I'm happy with that, we will update those over time.
verbs: [ "list", "get" ] | ||
- apiGroups: ["kustomize.toolkit.fluxcd.io"] | ||
resources: [ "kustomizations" ] | ||
verbs: [ "*" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't *
too broad? We have only read operations on our short term roadmap and removed everything that would write.
verbs: [ "*" ] | ||
- apiGroups: ["helm.toolkit.fluxcd.io"] | ||
resources: [ "helmreleases" ] | ||
verbs: [ "*" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
verbs: [ "*" ] | ||
- apiGroups: ["source.toolkit.fluxcd.io"] | ||
resources: [ "buckets", "helmcharts", "gitrepositories", "helmrepositories" ] | ||
verbs: [ "*" ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
All the |
This is not a "finished product", I am simply doing the bare minimum to
unblock people and make life easier while I tidy some things up.
Here I am just adding some more manifests to the
Tiltfile
; I havestashed these in
tools/dev-manifests
for now, but this may change.Either way people will not notice as they are just running
make cluster-dev
.The manifests under
manifests/wego-dev
are technically out of use aswe move onto a Helm Chart install flow, but I am leaving them there for
now as I tinker, plus I know other people are working on the Helm bit so
I am staying out of their way.
I have also left the SuperUser and the "stripping out the service
account" bit for now, as they do not need to be in place to unblock
people. They are next on my list.
Closes #1594