Skip to content

fix: add podConfigVersion and save a calculated struct as annotation on a pod#2298

Open
rugggger wants to merge 1 commit intomainfrom
02-26-fix_add_podconfigversion_and_save_a_calculated_struct_as_annotation_on_a_pod
Open

fix: add podConfigVersion and save a calculated struct as annotation on a pod#2298
rugggger wants to merge 1 commit intomainfrom
02-26-fix_add_podconfigversion_and_save_a_calculated_struct_as_annotation_on_a_pod

Conversation

@rugggger
Copy link
Contributor

@rugggger rugggger commented Feb 26, 2026

TL;DR

Added pod configuration versioning to track when pods need to be rotated due to configuration changes.

What changed?

  • Added POD_CONFIG_VERSION environment variable to the manager deployment template
  • Introduced podConfigVersion: 1 in the Helm chart values
  • Created a new configuration field PodConfigVersion in the environment configuration
  • Added WekaRuntimeVersion constant set to 1
  • Implemented PodConfigHash() function that generates a SHA-256 hash from both pod config version and runtime version
  • Added podConfigVersionAnnotation annotation to pods using the generated hash

How to test?

  1. Deploy the operator with the new configuration
  2. Verify that pods have the weka.io/pod-config-version annotation with a hash value
  3. Change the podConfigVersion value in the Helm chart and redeploy
  4. Confirm that the annotation hash changes, triggering pod rotation

Why make this change?

This enables the operator to automatically rotate pods when configuration changes occur, ensuring that pods are running with the latest configuration without requiring manual intervention.

Copy link
Contributor Author

rugggger commented Feb 26, 2026


How to use the Graphite Merge Queue

Add the label main-merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@rugggger rugggger marked this pull request as ready for review February 26, 2026 10:25
@rugggger rugggger requested a review from a team as a code owner February 26, 2026 10:25
@graphite-app
Copy link

graphite-app bot commented Feb 26, 2026

Graphite Automations

"Add anton/matt/sergey/kristina as reviwers on operator PRs" took an action on this PR • (02/26/26)

3 reviewers were added to this PR based on Anton Bykov's automation.

@rugggger rugggger force-pushed the 02-26-fix_add_podconfigversion_and_save_a_calculated_struct_as_annotation_on_a_pod branch from 5bf5f3d to cf93221 Compare March 4, 2026 09:32
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.

2 participants