Skip to content

chore: deprecate old annotations#301

Merged
Dav-14 merged 1 commit intomainfrom
chore/rename_job_annotations
Feb 9, 2026
Merged

chore: deprecate old annotations#301
Dav-14 merged 1 commit intomainfrom
chore/rename_job_annotations

Conversation

@Dav-14
Copy link
Contributor

@Dav-14 Dav-14 commented Feb 9, 2026

No description provided.

@Dav-14 Dav-14 requested a review from a team as a code owner February 9, 2026 10:09
@github-actions github-actions bot added the docs label Feb 9, 2026
Base automatically changed from chore/EN-723-clean-demo-test to main February 9, 2026 10:10
flemzord
flemzord previously approved these changes Feb 9, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates version badges across 7 Helm charts (Agent, Cloudprem, Console-V3, Core, Membership, Portal, and Stargate) and refactors Helm hook annotations in the Core chart by extracting them into a shared template definition.

Changes

Cohort / File(s) Summary
Version Badge Updates
README.md, charts/agent/README.md, charts/cloudprem/README.md, charts/console-v3/README.md, charts/core/README.md, charts/membership/README.md, charts/portal/README.md, charts/stargate/README.md
Updates version badges across 8 README files: Agent (2.11.1 → 2.12.0), Cloudprem (4.0.0-beta.20 → 4.0.0-beta.21), Console-V3 (3.0.0-beta.14 → 3.0.0-beta.15), Core (1.4.0 → 1.5.0), Membership (3.0.0-beta.7 → 3.0.0-beta.8), Portal (3.0.0-beta.15 → 3.0.0-beta.16), Stargate (0.9.1 → 0.10.0).
Core Template Refactoring
charts/core/templates/_job.tpl, charts/core/templates/_postgres.tpl
Introduces new core.job.annotations template for pre-upgrade Helm hook metadata; refactors core.postgres.job.annotations and core.postgres.job.sa.annotations to delegate to this shared template instead of embedding hook logic inline.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • flemzord
  • reslene

Poem

🐰 Charts bumped with versions bright,
Templates cleaned and refactored right,
Hooks now shared, no duplication,
One template for the whole migration! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/rename_job_annotations

Comment @coderabbitai help to get the list of available commands and usage tips.

@Dav-14 Dav-14 force-pushed the chore/rename_job_annotations branch from 99dd9ac to 31f5835 Compare February 9, 2026 10:13
Copy link
Contributor Author

@Dav-14 Dav-14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EN-726

@Dav-14 Dav-14 requested a review from flemzord February 9, 2026 10:14
@Dav-14 Dav-14 merged commit 2ef3017 into main Feb 9, 2026
4 checks passed
@Dav-14 Dav-14 deleted the chore/rename_job_annotations branch February 9, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants