argo-helm icon indicating copy to clipboard operation
argo-helm copied to clipboard

feat(argo-cd): add support for passing redis - sentinel credentials

Open shlomitubul opened this issue 4 months ago • 6 comments

This PR enables authentication for external Redis with Sentinel by introducing REDIS_SENTINEL_USERNAME and REDIS_SENTINEL_PASSWORD as optional environment variables. These credentials can now be passed via secrets to the controller, repo-server, and server templates, mirroring the approach used for standard external Redis connections.

NOTE: The actual support for username/password for Sentinel in argo code added in this PR: https://github.com/argoproj/argo-cd/pull/17168

Checklist:

  • [x] I have bumped the chart version according to versioning
  • [x] I have updated the documentation according to documentation
  • [x] I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • [x] Any new values are backwards compatible and/or have sensible default.
  • [x] I have signed off all my commits as required by DCO.
  • [x] My build is green (troubleshooting builds).

shlomitubul avatar Feb 10 '24 17:02 shlomitubul

I have updated the chart changelog with all the changes that come with this pull request according to changelog.

Please handle this.

yu-croco avatar Feb 11 '24 02:02 yu-croco

@yu-croco @pdrastil is there anything else that needs to be changed/fixed?

shlomitubul avatar Feb 13 '24 09:02 shlomitubul

@shlomitubul PR looks good, we are just waiting on upstream to cherry pick this into release branch

pdrastil avatar Mar 01 '24 15:03 pdrastil

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar May 01 '24 02:05 github-actions[bot]

Any updates on when this might be merged for release? I'm working on a deployment where I'd like to use Sentinel but it looks like this is a blocker to that.

corradomatt avatar May 03 '24 19:05 corradomatt

Hi @corradomatt . As discussed https://github.com/argoproj/argo-helm/pull/2492#issuecomment-1973361762 , we are waiting for upstream that https://github.com/argoproj/argo-cd/pull/17168 is released as specific version. 🙋 We will merge this PR as soon as upstream is handled.

yu-croco avatar May 04 '24 01:05 yu-croco