-
-
Notifications
You must be signed in to change notification settings - Fork 432
Add recursive user manifests #1314
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
Conversation
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.
@markusheinemann Thanks for this. Looking good. Please if you could tweak the docs for this, especially in the examples section in the readme and other relevant places (if this is mentioned there, like kube.tf), it would be great.
10082ec
to
96b29cc
Compare
I have updated the related docs and updated the examples as well. I am happy about your feedback. |
Actual the provided solution will not work due to hashicorp/terraform#16330. I'm currently looking for a workaround |
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.
Awesome @markusheinemann! Please let me know when it's working.
@markusheinemann What about now? |
lgtm. I think the PR can be merged in my eyes. Can you please rerun the workflow? PS. Sorry for closing the PR due to an oversight |
Thanks again for this @markusheinemann, awesome! |
see #1313
How to Test
extra-manifests
:kustomization.yaml.tpl
subfolder/kustomization.yaml.tpl
subfolder/namespace.yaml.tpl
Create a cluster using
terrafom apply
Check if the resource has been applied:
kubectl get ns
--> It work's on my machine 🤣