[Backport release-1.2] fix(mariadb): remove duplicate template key in backup CronJob#2652
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a deployment issue in the MariaDB chart for the release-1.2 branch. By eliminating a redundant key in the CronJob template, the configuration now adheres to valid YAML syntax, ensuring that deployments with backups enabled can proceed without encountering unmarshal errors. Highlights
New Features🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
Minimal backport from #2279 to release-1.2 — only the dup-key removal, without the breaking service-naming / replication changes. Signed-off-by: Andrei Kvapil <kvapss@gmail.com> Co-Authored-By: Claude <noreply@anthropic.com>
0738390 to
af91ad5
Compare
There was a problem hiding this comment.
Code Review
This pull request cleans up the MariaDB backup CronJob template by removing a redundant template specification block. The review feedback indicates that the pull request body is missing the mandatory release-note block required by the project's style guide.
| @@ -13,9 +13,6 @@ spec: | |||
| jobTemplate: | |||
There was a problem hiding this comment.
The pull request body is missing the required release-note block. According to the repository style guide, every PR must include a release note block in the following format:
```release-note
type(scope): human-readable changelog entry
<details>
<summary>References</summary>
1. PR body must contain a release note block. Flag any PR whose body has no release-note block. <sup>([link](https://github.com/cozystack/cozystack/blob/main/.gemini/styleguide.md))</sup>
</details>
## Summary Adds `docs/changelogs/v1.2.4.md` so the `Update Release Notes` workflow can publish the existing `v1.2.4` draft release. `v1.2.4` is a hotfix patch backporting only the duplicate-`template:` key removal from #2279 — the breaking service-naming changes in that PR were intentionally omitted as not appropriate for a patch (see #2652 for the minimal-backport details). The `Generate Changelog` step on the tag-push workflow failed twice in a row (yesterday's cron run and today's manual dispatch) due to `402 You have no quota` on the Copilot API, so this changelog is hand-written following `docs/changelogs/patch-template.md` and matches the v1.2.3 style. ## Test plan - [x] file follows `patch-template.md` structure - [x] author attribution verified against `gh pr view` for #2279 and #2652 - [ ] CI passes - [ ] after merge, `Update Release Notes` picks up the file and publishes draft release `v1.2.4`
Hand-written patch changelog for v1.2.4 (Copilot quota exhausted on the automatic Generate Changelog step). Single fix: minimal backport of the MariaDB backup CronJob duplicate-key removal from cozystack#2279 via cozystack#2652. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
Summary
Minimal backport from #2279 to
release-1.2— only the duplicate-template:key removal inpackages/apps/mariadb/templates/backup-cronjob.yaml, without the breaking service-naming / forced-replication changes from #2279 (those are not appropriate for a patch release).Fixes the
yaml: unmarshal errors: line 17: mapping key "template" already defined at line 14error when deploying MariaDB withbackup.enabled: trueon v1.2.x.Related: #2293
Test plan
helm templateof the mariadb chart withbackup.enabled: truerenders without YAML duplicate-key errors