Skip to content

feat: add imagePullSecrets support for operator and managed resources#395

Open
Richie1710 wants to merge 2 commits intohyperspike:mainfrom
Richie1710:feature/image-pull-secrets
Open

feat: add imagePullSecrets support for operator and managed resources#395
Richie1710 wants to merge 2 commits intohyperspike:mainfrom
Richie1710:feature/image-pull-secrets

Conversation

@Richie1710
Copy link
Copy Markdown

Summary

Adds support for imagePullSecrets to enable pulling container images from private registries. This feature works on two levels:

  1. Operator Deployment: Configure imagePullSecrets in the Helm chart values to pull the operator image itself from a private registry
  2. Managed Resources: Configure imagePullSecrets in the Valkey CRD spec to pull Valkey server and proxy images from private registries

Changes

CRD Changes

  • Added imagePullSecrets []corev1.LocalObjectReference field to ValkeySpec API type
  • Field is optional and propagates to StatefulSet and Proxy Deployment pod specs
  • Includes update detection logic to trigger StatefulSet updates when imagePullSecrets change

Helm Chart Changes

  • Added post-processing step in make helm-gen that injects conditional imagePullSecrets template into the operator deployment
  • Users can configure controllerManager.imagePullSecrets in Helm values

Allow users to specify imagePullSecrets in the Valkey CRD spec to
enable pulling container images from private registries. The secrets
are propagated to both the StatefulSet and the external access proxy
Deployment pod specs.
Add post-processing step to helm-gen that injects a conditional
imagePullSecrets block into the operator deployment template.
Users can set controllerManager.imagePullSecrets in their Helm
values to pull the operator image from private registries.
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.

1 participant