-
Notifications
You must be signed in to change notification settings - Fork 27
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
Connect: add support for RHEL distributions to startup script #502
base: main
Are you sure you want to change the base?
Conversation
Looks good! This will need a chart version bump and then I think it's good to merge |
@dbkegley Should be good to go. Somewhat confusing that CI for forked PRs always fails due to missing write permissions on the fork by the CI bot. |
@jforest How should we address contributions from forks for this repo given that CI doesn't run? I can't override these merges anymore since the permissions have changed. |
@dbkegley Can we get this merged? Would like to use the upstream chart again with some RHEL-based images :) |
Sorry @dbkegley I missed this while out on vacation. For now, I think you'll have to recreate this in order for tests to run, we do not currently have a way for forks to run the tests properly. because they don't have access to our secrets |
Same as #453