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

feat(cloudflare-tunnel-remote): add external secrets support #68

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

Conversation

hedgieinsocks
Copy link

PR #38 has introduced external secrets support for the main cloudflare-tunnel chart.

As per the listed below links, the community does need this feature to be ported to cloudflare-tunnel-remote chart as well.

#36
#60
#63

This PR is a desperate attempt to push this forward by making minimal changes to ease the review by the maintainers.

@hedgieinsocks
Copy link
Author

Hello, tagged you obezuk as the approver for the similar pr #38 in hope to get this one noticed.

@Sea-you
Copy link

Sea-you commented Jun 19, 2024

Dear ppl of @cloudflare, would it be possible to review and merge this PR?

@mergwyn
Copy link

mergwyn commented Sep 21, 2024

is there anything the community can do to enable this PR to be merged?

@jackylamhk
Copy link

I was about to open a PR for the same thing, leaving this comment for visibility.

@dudo
Copy link

dudo commented Jan 18, 2025

@obezuk Do you think you could take a look at this? I'd rather not have to fork this. There aren't that many issues / PRs to maintain here...

@dudo
Copy link

dudo commented Jan 19, 2025

I created my own chart... took less than an hour after starting with helm create.

https://github.com/dudo-home-lab/helm-charts/tree/main/charts/cloudflare-tunnel-remote

Not ideal, as I don't love this type fracturing, but this is a very lightweight wrapper around cloudflared ultimately. If @cloudflare isn't going to maintain these charts, I'd rather see them archived, honestly.

@dustinrue
Copy link

Unfortunate that such a simple PR was ignored. Thank you @dudo for the fork.

@kornelkalnai
Copy link

@obezuk, @jcsf, @uhthomas, @bwesterb Can someone review and merge this PR, please?

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.

8 participants