feat: add headless service configuration support by drivebyer · Pull Request #1219 · OT-CONTAINER-KIT/redis-operator

drivebyer · 2025-01-26T03:20:55Z

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.

Description

#1212

Type of change

New feature (non-breaking change which adds functionality)

Checklist

Tests have been added/modified and all tests pass.
Functionality/bugs have been confirmed to be unchanged or fixed.
I have performed a self-review of my own code.
Documentation has been updated or added where necessary.

Additional Context

- 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>

Signed-off-by: drivebyer <wuyangmuc@gmail.com>

codecov · 2025-01-26T03:34:14Z

Codecov Report

Attention: Patch coverage is 6.25000% with 60 lines in your changes missing coverage. Please review.

Project coverage is 29.65%. Comparing base (60ef5a2) to head (3d5a370).
Report is 32 commits behind head on master.

Files with missing lines	Patch %	Lines
api/zz_generated.deepcopy.go	0.00%	20 Missing ⚠️
pkg/k8sutils/redis-cluster.go	0.00%	18 Missing ⚠️
pkg/k8sutils/redis-standalone.go	0.00%	18 Missing ⚠️
api/common_types.go	50.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1219      +/-   ##
==========================================
- Coverage   30.20%   29.65%   -0.56%     
==========================================
  Files          55       57       +2     
  Lines        6294     6657     +363     
==========================================
+ Hits         1901     1974      +73     
- Misses       4200     4497     +297     
+ Partials      193      186       -7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: drivebyer <wuyangmuc@gmail.com>

#1263) * 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>

drivebyer requested review from iamabhishek-dubey and shubham-cmyk as code owners January 26, 2025 03:20

drivebyer added the enhancement New feature or request label Jan 26, 2025

fix

7b62fcc

Signed-off-by: drivebyer <wuyangmuc@gmail.com>

fix

3d5a370

Signed-off-by: drivebyer <wuyangmuc@gmail.com>

drivebyer merged commit 79ea018 into OT-CONTAINER-KIT:master Jan 26, 2025
17 of 19 checks passed

drivebyer deleted the svc-annotation branch January 26, 2025 05:10

arpan57 mentioned this pull request Mar 11, 2025

Option to configure annotations for leader-headless service only #1272

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add headless service configuration support#1219

feat: add headless service configuration support#1219
drivebyer merged 3 commits intoOT-CONTAINER-KIT:masterfrom
drivebyer:svc-annotation

drivebyer commented Jan 26, 2025

Uh oh!

codecov bot commented Jan 26, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

drivebyer commented Jan 26, 2025

Uh oh!

codecov bot commented Jan 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

codecov bot commented Jan 26, 2025 •

edited

Loading