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

Add new resource 'kubectl_kustomize_documents' #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matyat
Copy link

@matyat matyat commented Jun 4, 2021

This will allow the use of https://kustomize.io/ to generate a series of
k8s YAML docs, then apply them to a cluster in much the same way as
'kubectl_kustomize_documents'

@matyat
Copy link
Author

matyat commented Jun 4, 2021

As per #78, it's a bit rough and ready. Test cases probably needs expanding but it's a start.

go.mod Outdated
sigs.k8s.io/yaml v1.2.0
)

//replace github.com/Azure/go-autorest v10.15.4+incompatible => github.com/Azure/go-autorest v13.0.0+incompatible
replace github.com/tencentcloud/tencentcloud-sdk-go v3.0.82+incompatible => github.com/tencentcloud/tencentcloud-sdk-go v1.0.169
Copy link
Author

@matyat matyat Jun 4, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was required for me to be able to build at all, seems like as far as I can tell this project has deleted all its tags at some point in the past.

@schollii
Copy link

Looks good, any chance for a review and merge to master?

@gavinbunney
Copy link
Owner

@Logibox Thanks! Can you rebase the PR and fix up the merge conflicts? Those issues with go.mod should be fixed

@NicklasWallgren
Copy link

Great addition! @matyat Have you had time to rebase the PR?

This will allow the use of https://kustomize.io/ to generate a series of
k8s YAML docs, then apply them to a cluster in much the same way as
'kubectl_file_documents'
@matyat
Copy link
Author

matyat commented Nov 17, 2021

Hey guys, sorry for the very long delay on this - rebased now. Let me know if any else need adapting, as I realise it's been a while since I opened this PR and the codebase may have moved on a bit since then.

@burningalchemist
Copy link

Hey folks! Do you have any blockers to review/merge this PR?

@ritsuki1227
Copy link

@gavinbunney Hi, thank you for this useful module! I'd appreciate it if you review this feature.

@bbigras
Copy link

bbigras commented Aug 10, 2022

Any progress on this?

@burningalchemist
Copy link

Keep summoning @gavinbunney to the thread. 😃

@kholisrag
Copy link

any blocker on this one?, we would like this PR to be release soon. Thank You!

@iuliancristea
Copy link

@gavinbunney this would be immensely useful if merged

@fdzuJ
Copy link

fdzuJ commented Mar 22, 2023

please @gavinbunney

@falmar
Copy link

falmar commented Mar 10, 2024

hello @gavinbunney hope this find you well, any plans for this to merged soon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.