feat: Added support for hostport to allow direct connection to the pod (#1263)

MuhammadQadora · mkl262 · drivebyer · web-flow · commit 555e637dd3de · 2025-03-06T16:38:22.000+08:00
* Added support for hostPort and updated helmcharts * removed changed from helm chart * feat: Add hostPort support to Redis CRDs and update deepcopy methods * feat: sentinel support hostname resolve and annonce (#1247) * feat: Sentinel - support hostname resolve and annonce master Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> --------- Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> Signed-off-by: yangw <wuyangmuc@gmail.com> Co-authored-by: yangw <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Added support for hostPort and updated helmcharts Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump github.com/stretchr/testify from 1.9.0 to 1.10.0 (#1256) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.9.0 to 1.10.0. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.9.0...v1.10.0) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump sigstore/cosign-installer from 3.7.0 to 3.8.1 (#1255) Bumps [sigstore/cosign-installer](https://github.com/sigstore/cosign-installer) from 3.7.0 to 3.8.1. - [Release notes](https://github.com/sigstore/cosign-installer/releases) - [Commits](sigstore/cosign-installer@v3.7.0...v3.8.1) --- updated-dependencies: - dependency-name: sigstore/cosign-installer dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add redis agent with bootstrap configuration generation (#1254) * move main Signed-off-by: drivebyer <wuyangmuc@gmail.com> * feat: add redis agent with bootstrap configuration generation - Introduced new agent package for Redis and Sentinel configuration management - Added Cobra CLI support for agent bootstrap command - Implemented configuration generation for Sentinel with environment-based customization - Created Dockerfile for agent image - Added utility functions for environment variable handling - Prepared foundation for Redis and Sentinel configuration bootstrapping Signed-off-by: Redis Operator Team Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Redis Operator Team Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Implement comprehensive Redis configuration generation for agent bootstrap (#1260) * feat: Implement comprehensive Redis configuration generation for agent bootstrap This commit adds a detailed implementation of Redis configuration generation with support for: - Cluster and standalone modes - TLS configuration - ACL setup - Persistence options - Port and network configurations - Dynamic IP and hostname resolution - External configuration file inclusion The implementation uses environment variables to customize Redis configuration dynamically and handles various deployment scenarios. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * removed changed from helm chart Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Add hostPort support to Redis CRDs and update deepcopy methods Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add dynamic Redis configuration support for Redis Cluster (#1241) * feat: add dynamic Redis configuration support for Redis Cluster - Added `DynamicConfig` field to `RedisConfig` struct in common types - Implemented `GetRedisDynamicConfig()` method in RedisCluster spec - Created `SetRedisClusterDynamicConfig()` utility function to apply dynamic configuration - Updated reconciliation process to set dynamic configuration when cluster is ready - Added e2e test to verify dynamic configuration application This feature allows users to dynamically configure Redis instances at runtime with top-level configuration parameters. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix lint Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: configurable operator maxConcurrentReconciles (#1242) * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> --------- Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * ci: add GitHub workflow for automatic PR rebase (#1243) Add a GitHub Actions workflow to enable automatic rebasing of pull requests via comment commands `/rebase` and `/autosquash` The workflow allows repository collaborators to trigger a rebase of a pull request by commenting `/rebase` or perform an autosquash rebase with `/autosquash`. Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Bump operator version Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * refactor: remove v1beta1 API version and migrate to v1beta2 (#1245) * refactor: remove v1beta1 API version and migrate to v1beta2 This commit removes the v1beta1 API version across the entire project, including: - Deleting v1beta1 API type definitions - Removing v1beta1 CRD configurations - Updating example manifests to use v1beta2 - Cleaning up test suite configurations - Updating documentation references The migration focuses on consolidating the project to use only the v1beta2 API version, simplifying the codebase and reducing maintenance overhead. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: sentinel support hostname resolve and annonce (#1247) * feat: Sentinel - support hostname resolve and annonce master Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> --------- Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> Signed-off-by: yangw <wuyangmuc@gmail.com> Co-authored-by: yangw <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Added support for hostPort and updated helmcharts Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump github.com/stretchr/testify from 1.9.0 to 1.10.0 (#1256) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.9.0 to 1.10.0. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.9.0...v1.10.0) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump sigstore/cosign-installer from 3.7.0 to 3.8.1 (#1255) Bumps [sigstore/cosign-installer](https://github.com/sigstore/cosign-installer) from 3.7.0 to 3.8.1. - [Release notes](https://github.com/sigstore/cosign-installer/releases) - [Commits](sigstore/cosign-installer@v3.7.0...v3.8.1) --- updated-dependencies: - dependency-name: sigstore/cosign-installer dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add redis agent with bootstrap configuration generation (#1254) * move main Signed-off-by: drivebyer <wuyangmuc@gmail.com> * feat: add redis agent with bootstrap configuration generation - Introduced new agent package for Redis and Sentinel configuration management - Added Cobra CLI support for agent bootstrap command - Implemented configuration generation for Sentinel with environment-based customization - Created Dockerfile for agent image - Added utility functions for environment variable handling - Prepared foundation for Redis and Sentinel configuration bootstrapping Signed-off-by: Redis Operator Team Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Redis Operator Team Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Implement comprehensive Redis configuration generation for agent bootstrap (#1260) * feat: Implement comprehensive Redis configuration generation for agent bootstrap This commit adds a detailed implementation of Redis configuration generation with support for: - Cluster and standalone modes - TLS configuration - ACL setup - Persistence options - Port and network configurations - Dynamic IP and hostname resolution - External configuration file inclusion The implementation uses environment variables to customize Redis configuration dynamically and handles various deployment scenarios. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * removed changed from helm chart Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Add hostPort support to Redis CRDs and update deepcopy methods Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Added support for hostPort and updated helmcharts Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * removed changed from helm chart Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Add hostPort support to Redis CRDs and update deepcopy methods Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * fix: replace hardcoded Redis port 6379 with configurable port from cr.Spec.Port (#1261) - Updated RedisClusterStatusHealth function to use cr.Spec.Port instead of hardcoded 6379 port. Co-authored-by: fengyinqiao <fengyinqiao@shizhuang-inc.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * test: enhance testing with data assertion scripts (#1209) * test: enhance Redis cluster testing with data assertion scripts - Added new steps in the e2e test for Redis cluster to apply a data assertion tool, `data-assert`, for generating and validating data in the Redis cluster. - Replaced the previous Redis GET/SET checks with a more comprehensive data generation and validation process using the `data-assert` tool. - Introduced additional assertions to ensure data consistency after scaling operations. This update improves the reliability of e2e tests by ensuring data integrity in Redis clusters. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * path Signed-off-by: drivebyer <wuyangmuc@gmail.com> * password Signed-off-by: drivebyer <wuyangmuc@gmail.com> * password Signed-off-by: drivebyer <wuyangmuc@gmail.com> * password Signed-off-by: drivebyer <wuyangmuc@gmail.com> * ha Signed-off-by: drivebyer <wuyangmuc@gmail.com> * update Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add headless service configuration support (#1219) * feat: add headless service configuration support - Introduced a new `Headless` service configuration in `ServiceConfig` struct - Added method `GetHeadlessServiceAnnotations()` to retrieve headless service annotations - Updated CRD bases to include headless service configuration - Modified service creation logic in Redis cluster and standalone services to support headless service annotations - Added e2e test cases for standalone Redis with headless service configuration This enhancement provides more flexibility in configuring headless services for Redis deployments. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * fix: add missing topologySpreadConstraint on RedisCluster follower (#1218) Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: update redis-operator cert manager configuration. (#1220) * fix: update redis-operator chart configuration and documentation. - Removed Chart.lock file for cert-manager dependency - Updated Chart.yaml to remove explicit cert-manager dependency - Bumped chart version to 0.19.2 - Updated README.md to clarify cert-manager installation instructions - Modified values.yaml to improve cert-manager and webhook configuration documentation - Added configuration validation helper in _helpers.tpl - Introduced NOTES.txt template for helm chart installation guidance Signed-off-by: drivebyer <wuyangmuc@gmail.com> * docs Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore: dockerfile command style consistency (#1222) Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: update certificate serverName to support wildcard certs (#1221) Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add additional service configuration with optional enable flag (#1228) - Introduced `Additional` service configuration in `ServiceConfig` struct - Added `Enabled` field to `Service` struct with default value of true - Implemented `ShouldCreateAdditionalService()` method to control additional service creation - Updated CRD bases to include additional service configuration - Modified service creation logic in Redis cluster and standalone services to respect additional service configuration - Added unit test for `ShouldCreateAdditionalService()` method This enhancement provides more flexibility in configuring additional services for Redis deployments. Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump helm/chart-releaser-action from 1.6.0 to 1.7.0 (#1230) Bumps [helm/chart-releaser-action](https://github.com/helm/chart-releaser-action) from 1.6.0 to 1.7.0. - [Release notes](https://github.com/helm/chart-releaser-action/releases) - [Commits](helm/chart-releaser-action@v1.6.0...v1.7.0) --- updated-dependencies: - dependency-name: helm/chart-releaser-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump helm/chart-testing-action from 2.6.0 to 2.7.0 (#1229) Bumps [helm/chart-testing-action](https://github.com/helm/chart-testing-action) from 2.6.0 to 2.7.0. - [Release notes](https://github.com/helm/chart-testing-action/releases) - [Commits](helm/chart-testing-action@v2.6.0...v2.7.0) --- updated-dependencies: - dependency-name: helm/chart-testing-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump github.com/onsi/ginkgo/v2 from 2.22.1 to 2.22.2 (#1225) Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.22.1 to 2.22.2. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](onsi/ginkgo@v2.22.1...v2.22.2) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * fix: improve pod label patching with error handling and retry mechanism (#1231) fix: improve pod label patching with error handling and retry mechanism - Added JSON pointer escaping for label keys - Implemented retry mechanism for label patching - Enhanced error handling and logging - Added validation for empty labels - Created utility function for JSON pointer escaping in util package Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: enhance Redis HA and node scheduling strategy (#1237) * feat: enhance Redis HA and node scheduling strategy - Updated e2e test to simulate node shutdown instead of pod deletion - Added pod anti-affinity configuration for Redis replication and sentinel nodes - Removed unnecessary StatefulSet readiness check in Redis replication controller - Improved node scheduling to distribute Redis pods across different worker nodes Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add securityContext config in chart for redis-exporter (#1238) feat: add securityContext config for redis-exporter Add ability to configure securityContext for redis-exporter in redis helm charts: - redis - redis-cluster - redis-replication - redis-sentinel Supported in: - v1beta1 & v1beta2: Redis, RedisCluster, RedisReplication Operator CRDs - v1beta2 only: RedisSentinel Operator CRD This aligns with existing redis operator CRD functionality. Signed-off-by: Hexoplon <github@zackeus.no> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * fix: make recreate statefulset only trigger when value true (#1240) Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add dynamic Redis configuration support for Redis Cluster (#1241) * feat: add dynamic Redis configuration support for Redis Cluster - Added `DynamicConfig` field to `RedisConfig` struct in common types - Implemented `GetRedisDynamicConfig()` method in RedisCluster spec - Created `SetRedisClusterDynamicConfig()` utility function to apply dynamic configuration - Updated reconciliation process to set dynamic configuration when cluster is ready - Added e2e test to verify dynamic configuration application This feature allows users to dynamically configure Redis instances at runtime with top-level configuration parameters. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix lint Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: configurable operator maxConcurrentReconciles (#1242) * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> * feat: configurable operator maxConcurrentReconciles Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> --------- Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * ci: add GitHub workflow for automatic PR rebase (#1243) Add a GitHub Actions workflow to enable automatic rebasing of pull requests via comment commands `/rebase` and `/autosquash` The workflow allows repository collaborators to trigger a rebase of a pull request by commenting `/rebase` or perform an autosquash rebase with `/autosquash`. Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Bump operator version Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * refactor: remove v1beta1 API version and migrate to v1beta2 (#1245) * refactor: remove v1beta1 API version and migrate to v1beta2 This commit removes the v1beta1 API version across the entire project, including: - Deleting v1beta1 API type definitions - Removing v1beta1 CRD configurations - Updating example manifests to use v1beta2 - Cleaning up test suite configurations - Updating documentation references The migration focuses on consolidating the project to use only the v1beta2 API version, simplifying the codebase and reducing maintenance overhead. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: sentinel support hostname resolve and annonce (#1247) * feat: Sentinel - support hostname resolve and annonce master Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> * fix Signed-off-by: yangw <wuyangmuc@gmail.com> --------- Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> Signed-off-by: yangw <wuyangmuc@gmail.com> Co-authored-by: yangw <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Added support for hostPort and updated helmcharts Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump github.com/stretchr/testify from 1.9.0 to 1.10.0 (#1256) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.9.0 to 1.10.0. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.9.0...v1.10.0) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump sigstore/cosign-installer from 3.7.0 to 3.8.1 (#1255) Bumps [sigstore/cosign-installer](https://github.com/sigstore/cosign-installer) from 3.7.0 to 3.8.1. - [Release notes](https://github.com/sigstore/cosign-installer/releases) - [Commits](sigstore/cosign-installer@v3.7.0...v3.8.1) --- updated-dependencies: - dependency-name: sigstore/cosign-installer dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: add redis agent with bootstrap configuration generation (#1254) * move main Signed-off-by: drivebyer <wuyangmuc@gmail.com> * feat: add redis agent with bootstrap configuration generation - Introduced new agent package for Redis and Sentinel configuration management - Added Cobra CLI support for agent bootstrap command - Implemented configuration generation for Sentinel with environment-based customization - Created Dockerfile for agent image - Added utility functions for environment variable handling - Prepared foundation for Redis and Sentinel configuration bootstrapping Signed-off-by: Redis Operator Team Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Redis Operator Team Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Implement comprehensive Redis configuration generation for agent bootstrap (#1260) * feat: Implement comprehensive Redis configuration generation for agent bootstrap This commit adds a detailed implementation of Redis configuration generation with support for: - Cluster and standalone modes - TLS configuration - ACL setup - Persistence options - Port and network configurations - Dynamic IP and hostname resolution - External configuration file inclusion The implementation uses environment variables to customize Redis configuration dynamically and handles various deployment scenarios. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * removed changed from helm chart Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Add hostPort support to Redis CRDs and update deepcopy methods Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * Added support for hostPort and updated helmcharts Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * chore(deps): bump github.com/stretchr/testify from 1.9.0 to 1.10.0 (#1256) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.9.0 to 1.10.0. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.9.0...v1.10.0) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat: add redis agent with bootstrap configuration generation (#1254) * move main Signed-off-by: drivebyer <wuyangmuc@gmail.com> * feat: add redis agent with bootstrap configuration generation - Introduced new agent package for Redis and Sentinel configuration management - Added Cobra CLI support for agent bootstrap command - Implemented configuration generation for Sentinel with environment-based customization - Created Dockerfile for agent image - Added utility functions for environment variable handling - Prepared foundation for Redis and Sentinel configuration bootstrapping Signed-off-by: Redis Operator Team Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Redis Operator Team * feat: Implement comprehensive Redis configuration generation for agent bootstrap (#1260) * feat: Implement comprehensive Redis configuration generation for agent bootstrap This commit adds a detailed implementation of Redis configuration generation with support for: - Cluster and standalone modes - TLS configuration - ACL setup - Persistence options - Port and network configurations - Dynamic IP and hostname resolution - External configuration file inclusion The implementation uses environment variables to customize Redis configuration dynamically and handles various deployment scenarios. Signed-off-by: drivebyer <wuyangmuc@gmail.com> * fix Signed-off-by: drivebyer <wuyangmuc@gmail.com> --------- Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * removed changed from helm chart Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * feat: Add hostPort support to Redis CRDs and update deepcopy methods Signed-off-by: Muhammad Qadora <memomq70@gmail.com> * fix: replace hardcoded Redis port 6379 with configurable port from cr.Spec.Port (#1261) - Updated RedisClusterStatusHealth function to use cr.Spec.Port instead of hardcoded 6379 port. Co-authored-by: fengyinqiao <fengyinqiao@shizhuang-inc.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> --------- Signed-off-by: Michael Kanchuker <michaelkanchuker@gmail.com> Signed-off-by: yangw <wuyangmuc@gmail.com> Signed-off-by: Muhammad Qadora <memomq70@gmail.com> Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: drivebyer <wuyangmuc@gmail.com> Signed-off-by: Redis Operator Team Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com> Signed-off-by: Hexoplon <github@zackeus.no> Co-authored-by: Michael Kanchuker <michaelkanchuker@gmail.com> Co-authored-by: yangw <wuyangmuc@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: yangw <yang.wu@daocloud.io> Co-authored-by: Husni Alhamdani <dhanielluis@gmail.com> Co-authored-by: Leslie__Joe <58540165+fengyinqiao@users.noreply.github.com> Co-authored-by: fengyinqiao <fengyinqiao@shizhuang-inc.com> Co-authored-by: Zackeus Bengtsson <32719220+Hexoplon@users.noreply.github.com>
diff --git a/api/v1beta2/redis_types.go b/api/v1beta2/redis_types.go
@@ -45,6 +45,7 @@ type RedisSpec struct {
 	ServiceAccountName            *string                    `json:"serviceAccountName,omitempty"`
 	TerminationGracePeriodSeconds *int64                     `json:"terminationGracePeriodSeconds,omitempty" protobuf:"varint,4,opt,name=terminationGracePeriodSeconds"`
 	EnvVars                       *[]corev1.EnvVar           `json:"env,omitempty"`
+	HostPort                      *int                       `json:"hostPort,omitempty"`
 }
 
 // RedisStatus defines the observed state of Redis
diff --git a/api/v1beta2/rediscluster_types.go b/api/v1beta2/rediscluster_types.go
@@ -47,6 +47,7 @@ type RedisClusterSpec struct {
 	ServiceAccountName *string                      `json:"serviceAccountName,omitempty"`
 	PersistenceEnabled *bool                        `json:"persistenceEnabled,omitempty"`
 	EnvVars            *[]corev1.EnvVar             `json:"env,omitempty"`
+	HostPort           *int                         `json:"hostPort,omitempty"`
 }
 
 func (cr *RedisClusterSpec) GetReplicaCounts(t string) int32 {
diff --git a/api/v1beta2/redisreplication_types.go b/api/v1beta2/redisreplication_types.go
@@ -29,6 +29,7 @@ type RedisReplicationSpec struct {
 	TerminationGracePeriodSeconds *int64                            `json:"terminationGracePeriodSeconds,omitempty" protobuf:"varint,4,opt,name=terminationGracePeriodSeconds"`
 	EnvVars                       *[]corev1.EnvVar                  `json:"env,omitempty"`
 	TopologySpreadConstrains      []corev1.TopologySpreadConstraint `json:"topologySpreadConstraints,omitempty"`
+	HostPort                      *int                              `json:"hostPort,omitempty"`
 }
 
 func (cr *RedisReplicationSpec) GetReplicationCounts(t string) int32 {
diff --git a/api/v1beta2/redissentinel_types.go b/api/v1beta2/redissentinel_types.go
@@ -30,6 +30,7 @@ type RedisSentinelSpec struct {
 	EnvVars                       *[]corev1.EnvVar                  `json:"env,omitempty"`
 	VolumeMount                   *common.AdditionalVolume          `json:"volumeMount,omitempty"`
 	TopologySpreadConstrains      []corev1.TopologySpreadConstraint `json:"topologySpreadConstraints,omitempty"`
+	HostPort                      *int                              `json:"hostPort,omitempty"`
 }
 
 func (cr *RedisSentinelSpec) GetSentinelCounts(t string) int32 {
diff --git a/api/v1beta2/zz_generated.deepcopy.go b/api/v1beta2/zz_generated.deepcopy.go
diff --git a/charts/redis-cluster/values.yaml b/charts/redis-cluster/values.yaml
@@ -190,4 +190,4 @@ env: []
   # - name: VAR_NAME
   #   value: "value1"
 
-serviceAccountName: ""
+serviceAccountName: ""
diff --git a/charts/redis-replication/values.yaml b/charts/redis-replication/values.yaml
@@ -155,4 +155,4 @@ acl:
 
 env: []
   # - name: VAR_NAME
-  #   value: "value1"
+  #   value: "value1"
diff --git a/charts/redis-sentinel/values.yaml b/charts/redis-sentinel/values.yaml
@@ -174,4 +174,4 @@ readinessProbe:
 
 env: []
   # - name: VAR_NAME
-  #   value: "value1"
+  #   value: "value1"
diff --git a/charts/redis/values.yaml b/charts/redis/values.yaml
@@ -151,4 +151,4 @@ acl:
 
 env: []
   # - name: VAR_NAME
-  #   value: "value1"
+  #   value: "value1"
diff --git a/config/crd/bases/redis.redis.opstreelabs.in_redis.yaml b/config/crd/bases/redis.redis.opstreelabs.in_redis.yaml
@@ -1179,6 +1179,8 @@ spec:
                   - name
                   type: object
                 type: array
+              hostPort:
+                type: integer
               initContainer:
                 description: InitContainer for each Redis pods
                 properties:
diff --git a/config/crd/bases/redis.redis.opstreelabs.in_redisclusters.yaml b/config/crd/bases/redis.redis.opstreelabs.in_redisclusters.yaml
@@ -329,6 +329,8 @@ spec:
                 type: array
               hostNetwork:
                 type: boolean
+              hostPort:
+                type: integer
               initContainer:
                 description: InitContainer for each Redis pods
                 properties:
diff --git a/config/crd/bases/redis.redis.opstreelabs.in_redisreplications.yaml b/config/crd/bases/redis.redis.opstreelabs.in_redisreplications.yaml
@@ -1188,6 +1188,8 @@ spec:
                   - name
                   type: object
                 type: array
+              hostPort:
+                type: integer
               initContainer:
                 description: InitContainer for each Redis pods
                 properties:
diff --git a/config/crd/bases/redis.redis.opstreelabs.in_redissentinels.yaml b/config/crd/bases/redis.redis.opstreelabs.in_redissentinels.yaml
@@ -1113,6 +1113,8 @@ spec:
                   - name
                   type: object
                 type: array
+              hostPort:
+                type: integer
               initContainer:
                 description: InitContainer for each Redis pods
                 properties:
diff --git a/pkg/k8sutils/redis-cluster.go b/pkg/k8sutils/redis-cluster.go
@@ -119,6 +119,7 @@ func generateRedisClusterContainerParams(ctx context.Context, cl kubernetes.Inte
 		Resources:       resources,
 		SecurityContext: securityContext,
 		Port:            cr.Spec.Port,
+		HostPort:        cr.Spec.HostPort,
 	}
 	if cr.Spec.EnvVars != nil {
 		containerProp.EnvVars = cr.Spec.EnvVars
diff --git a/pkg/k8sutils/redis-replication.go b/pkg/k8sutils/redis-replication.go
@@ -138,6 +138,7 @@ func generateRedisReplicationContainerParams(cr *redisv1beta2.RedisReplication)
 		Resources:       cr.Spec.KubernetesConfig.Resources,
 		SecurityContext: cr.Spec.SecurityContext,
 		Port:            ptr.To(redisPort),
+		HostPort:        cr.Spec.HostPort,
 	}
 	if cr.Spec.EnvVars != nil {
 		containerProp.EnvVars = cr.Spec.EnvVars
diff --git a/pkg/k8sutils/redis-sentinel.go b/pkg/k8sutils/redis-sentinel.go
@@ -160,6 +160,7 @@ func generateRedisSentinelContainerParams(ctx context.Context, client kubernetes
 		Resources:             cr.Spec.KubernetesConfig.Resources,
 		SecurityContext:       cr.Spec.SecurityContext,
 		Port:                  ptr.To(sentinelPort),
+		HostPort:              cr.Spec.HostPort,
 		AdditionalEnvVariable: getSentinelEnvVariable(ctx, client, cr, dcl),
 	}
 	if cr.Spec.EnvVars != nil {
diff --git a/pkg/k8sutils/redis-standalone.go b/pkg/k8sutils/redis-standalone.go
@@ -146,6 +146,7 @@ func generateRedisStandaloneContainerParams(cr *redisv1beta2.Redis) containerPar
 		Resources:       cr.Spec.KubernetesConfig.Resources,
 		SecurityContext: cr.Spec.SecurityContext,
 		Port:            ptr.To(redisPort),
+		HostPort:        cr.Spec.HostPort,
 	}
 
 	if cr.Spec.EnvVars != nil {
diff --git a/pkg/k8sutils/statefulset.go b/pkg/k8sutils/statefulset.go
@@ -144,6 +144,7 @@ type containerParameters struct {
 	AdditionalMountPath          []corev1.VolumeMount
 	EnvVars                      *[]corev1.EnvVar
 	Port                         *int
+	HostPort                     *int
 }
 
 type initContainerParameters struct {
@@ -440,6 +441,15 @@ func generateContainerDef(name string, containerParams containerParameters, clus
 		},
 	}
 
+	if containerParams.HostPort != nil {
+		containerDefinition[0].Ports = []corev1.ContainerPort{
+			{
+				HostPort:      int32(*containerParams.HostPort),
+				ContainerPort: int32(*containerParams.Port),
+			},
+		}
+	}
+
 	if containerParams.Resources != nil {
 		containerDefinition[0].Resources = *containerParams.Resources
 	}

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,7 @@ type RedisSpec struct {`
`45`	`45`	ServiceAccountName *string `json:"serviceAccountName,omitempty"`
`46`	`46`	TerminationGracePeriodSeconds *int64 `json:"terminationGracePeriodSeconds,omitempty" protobuf:"varint,4,opt,name=terminationGracePeriodSeconds"`
`47`	`47`	EnvVars *[]corev1.EnvVar `json:"env,omitempty"`
	`48`	+ HostPort *int `json:"hostPort,omitempty"`
`48`	`49`	`}`
`49`	`50`
`50`	`51`	`// RedisStatus defines the observed state of Redis`
Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ type RedisClusterSpec struct {`
`47`	`47`	ServiceAccountName *string `json:"serviceAccountName,omitempty"`
`48`	`48`	PersistenceEnabled *bool `json:"persistenceEnabled,omitempty"`
`49`	`49`	EnvVars *[]corev1.EnvVar `json:"env,omitempty"`
	`50`	+ HostPort *int `json:"hostPort,omitempty"`
`50`	`51`	`}`
`51`	`52`
`52`	`53`	`func (cr *RedisClusterSpec) GetReplicaCounts(t string) int32 {`
Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,7 @@ type RedisReplicationSpec struct {`
`29`	`29`	TerminationGracePeriodSeconds *int64 `json:"terminationGracePeriodSeconds,omitempty" protobuf:"varint,4,opt,name=terminationGracePeriodSeconds"`
`30`	`30`	EnvVars *[]corev1.EnvVar `json:"env,omitempty"`
`31`	`31`	TopologySpreadConstrains []corev1.TopologySpreadConstraint `json:"topologySpreadConstraints,omitempty"`
	`32`	+ HostPort *int `json:"hostPort,omitempty"`
`32`	`33`	`}`
`33`	`34`
`34`	`35`	`func (cr *RedisReplicationSpec) GetReplicationCounts(t string) int32 {`
Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ type RedisSentinelSpec struct {`
`30`	`30`	EnvVars *[]corev1.EnvVar `json:"env,omitempty"`
`31`	`31`	VolumeMount *common.AdditionalVolume `json:"volumeMount,omitempty"`
`32`	`32`	TopologySpreadConstrains []corev1.TopologySpreadConstraint `json:"topologySpreadConstraints,omitempty"`
	`33`	+ HostPort *int `json:"hostPort,omitempty"`
`33`	`34`	`}`
`34`	`35`
`35`	`36`	`func (cr *RedisSentinelSpec) GetSentinelCounts(t string) int32 {`