Anonymous - 2025-09-10

Originally posted by: coderabbitai[bot]

Walkthrough

Introduces alerta.extraEnvs to allow injecting additional environment variables into the Alerta container via Helm values, updates schema, defaults, and documentation, and wires rendering in the Deployment template. Bumps monitoring chart version to 1.14.0 and updates versions_map to pin 1.13.0 and add 1.14.0.

Changes

Cohort / File(s) Summary
Monitoring versioning
packages/extra/monitoring/Chart.yaml, packages/extra/versions_map		 Bump chart version 1.13.0 → 1.14.0; pin 1.13.0 to commit and add 1.14.0 HEAD entry in versions_map.
Alerta extraEnvs support
packages/extra/monitoring/README.md, packages/extra/monitoring/templates/alerta/alerta.yaml, packages/extra/monitoring/values.schema.json, packages/extra/monitoring/values.yaml		 Add alerta.extraEnvs (array) to values and schema; document new field; template renders extra env vars into Alerta container’s env list; defaults to empty list.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor U as User
  participant V as values.yaml
  participant H as Helm (Rendering)
  participant T as alerta.yaml (Template)
  participant K as Kubernetes API
  participant P as Alerta Pod

  U->>V: Set alerta.extraEnvs (optional)
  V->>H: Provide values
  H->>T: Render template with .Values.alerta.extraEnvs
  T-->>H: Emit Deployment with env + extraEnvs
  H->>K: Apply Deployment
  K->>P: Create Pod with merged env
  note right of P: If extraEnvs is empty, no additional env entries are added.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • [[#803]](https://github.com/cozystack/cozystack/issues/803): Similar monitoring chart version bump and versions_map updates aligning version pinning and HEAD mapping.

Suggested labels

ok-to-test

Suggested reviewers

  • lllamnyp
  • kvaps

Pre-merge checks (2 passed, 1 warning)

❌ Failed checks (1 warning) | Check name | Status | Explanation | Resolution | | :---------: | :--------- | :--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | Title Check | ⚠️ Warning | The current title “feature add possible configure alerta” is vague, grammatically incorrect, and does not clearly summarize the primary change of introducing an extraEnvs configuration parameter for injecting environment variables into the Alerta monitoring chart; it fails to convey the main feature enhancement to reviewers scanning PR history. | Please update the title to a concise, descriptive sentence such as “Add extraEnvs configuration to Alerta monitoring chart” to clearly reflect the main purpose of the changeset. |
✅ Passed checks (2 passed) | Check name | Status | Explanation | | :----------------: | :------- | :------------------------------------------------------------------- | | Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. | | Docstring Coverage | ✅ Passed | No functions found in the changes. Docstring coverage check skipped. |

Poem

A bunny taps the helm with glee,
extra envs hop in, so free.
Schema trimmed, values neat,
templates line up, quick and sweet.
Version bumps—thump-thump—let’s go!
Alerta pods now better show. 🐇✨

[!TIP]


👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

yaml reviews: pre_merge_checks: custom_checks: - name: "Undocumented Breaking Changes" mode: "warning" instructions: | Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

✨ Finishing Touches
🧪 Generate unit tests - [ ] Create PR with unit tests - [ ] Post copyable unit tests in a comment - [ ] Commit unit tests in branch `feature-add-possible-configure-alerta`

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.

❤️ Share - [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)

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