Skip to content

[stable/redis-ha] Add tls ports to netpol if defined#313

Merged
DandyDeveloper merged 2 commits intoDandyDeveloper:masterfrom
OpenGuidou:netpol
Sep 15, 2025
Merged

[stable/redis-ha] Add tls ports to netpol if defined#313
DandyDeveloper merged 2 commits intoDandyDeveloper:masterfrom
OpenGuidou:netpol

Conversation

@OpenGuidou
Copy link
Contributor

@OpenGuidou OpenGuidou commented Jan 2, 2025

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

@OpenGuidou OpenGuidou changed the title feat(redis-ha): Add tls ports to netpol if defined [chart/redis-ha] Add tls ports to netpol if defined Jan 2, 2025
@OpenGuidou OpenGuidou changed the title [chart/redis-ha] Add tls ports to netpol if defined [stable/redis-ha] Add tls ports to netpol if defined Jan 2, 2025
Signed-off-by: OpenGuidou <guillaume.doussin@gmail.com>
@OpenGuidou
Copy link
Contributor Author

@DandyDeveloper Would you mind reviewing this one as well ?
The CI seems to fail for another reason

@OpenGuidou
Copy link
Contributor Author

Hi @DandyDeveloper ,
Any chance you could check this one soon ?

@DandyDeveloper
Copy link
Owner

DandyDeveloper commented Jul 8, 2025

@OpenGuidou On it now. Sorry.

Edit: I'm gonna take my time with this one because of the way it overrides all the ports globally. I'll revise this after looking at some other PRs.

@pierluigilenoci
Copy link
Contributor

@OpenGuidou, could you please rebase?
@DandyDeveloper, could you please take a look?

@DandyDeveloper DandyDeveloper merged commit d4f5cee into DandyDeveloper:master Sep 15, 2025
2 checks passed
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.

[chart/redis-ha][REQUEST] Include redis and sentinel tls ports in networkpolicies

3 participants