[LFXV2-618] add lfx-v2-mailing-list-service to platform chart

[prabodhcs]

Issue - https://linuxfoundation.atlassian.net/browse/LFXV2-618
This pull request updates the Helm chart for the LFX Platform to add support for the new mailing list service. The main changes include bumping the chart version, adding the mailing list service as a dependency, and providing default configuration values for it.

Helm chart version update:

• Increased the chart version in Chart.yaml from 0.3.2 to 0.3.3 to reflect the new changes.

Dependency management:

• Added lfx-v2-mailing-list-service as a new dependency in Chart.yaml, including its repository, version constraint, and enablement condition.

Default configuration:

• Introduced default values for lfx-v2-mailing-list-service in values.yaml, enabling the service and setting its domain.

[coderabbitai]

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

Bumps lfx-platform chart version to 0.3.4 and adds a new Helm dependency entry for lfx-v2-mailing-list-service. Adds a corresponding top-level lfx-v2-mailing-list-service values block with enabled: true and lfx.domain: k8s.orb.local.

Changes

Cohort / File(s)	Summary
Helm chart metadata charts/lfx-platform/Chart.yaml	Bumps version 0.3.3 → 0.3.4. Inserts dependency lfx-v2-mailing-list-service (repository: oci://ghcr.io/linuxfoundation/lfx-v2-mailing-list-service/chart, version: ~0.1.0, condition: lfx-v2-mailing-list-service.enabled) into the dependencies list (placed after lfx-v2-meeting-service).
Helm values charts/lfx-platform/values.yaml	Adds top-level lfx-v2-mailing-list-service block with enabled: true and lfx.domain: k8s.orb.local.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Add LFX v2 Services to Platform Chart #34 — Modifies charts/lfx-platform Chart.yaml and values.yaml to add LFX v2 service dependencies and corresponding values (adds other lfx-v2 services).
• Add committee and meeting services to chart #35 — Adds new lfx-v2 service dependencies and matching values blocks in charts/lfx-platform.
• [LFXV2-534] Add Auth Service #58 — Adds a service dependency and corresponding values block in charts/lfx-platform (similar change pattern).

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	The linked issue LFXV2-618 specifies detailed objectives focused on security configuration, including creating a security plan, defining secret management strategies, implementing secure credential handling, configuring webhook signing secrets, deploying environment-specific secrets, and adding security monitoring. The PR changes—adding the mailing list service as a Helm chart dependency and providing default configuration values—address infrastructure updates but do not implement the core security and secrets management objectives outlined in the issue. The PR appears to be a preliminary step to support the service in the chart, but it does not fulfill the stated acceptance criteria of creating a detailed security plan, configuring secrets, or conducting a security audit.	Consider whether this PR should be aligned with the issue's primary objectives around security and secret configuration, or whether the issue scope should be clarified to separate infrastructure chart updates from security configuration tasks. If the chart update is intended as a prerequisite step, consider creating a separate issue for the infrastructure changes and reserving LFXV2-618 for the security-focused work.
Out of Scope Changes Check	❓ Inconclusive	The changes made to the repository—bumping the chart version, adding the lfx-v2-mailing-list-service dependency, and providing default configuration values—are all directly related to integrating a new service into the Helm chart. From a chart maintenance perspective, these are in-scope changes for adding a new service. However, relative to the linked issue LFXV2-618, which focuses on security configuration and secret management, these changes are primarily infrastructure updates that do not address the main security-related objectives. The PR does not include security plans, secret configurations, or the environment-specific deployment work described in the issue.	Clarify whether this PR should be treated as a prerequisite infrastructure change supporting the issue, or whether it represents an incomplete implementation of LFXV2-618. If it is infrastructure only, consider updating the issue or creating separate tracking for the security-specific work that remains to be completed.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The PR title "LFXV2-618 add lfx-v2-mailing-list-service to platform chart" directly and clearly summarizes the main change in the pull request. The changeset focuses on adding the lfx-v2-mailing-list-service as a new dependency to the Helm chart along with providing default configuration values, which is exactly what the title conveys. The title is concise, specific, and avoids vague language, making it clear to team members scanning the history what the primary change is about.
Description Check	✅ Passed	The pull request description is related to the changeset, as it discusses adding the mailing list service as a dependency, bumping the chart version, and introducing default configuration values. However, there is a discrepancy: the description states the chart version was bumped from 0.3.2 to 0.3.3, while the actual changes show a bump from 0.3.3 to 0.3.4. Despite this inaccuracy, the description is clearly related to the actual changes made and explains the intent behind them, which satisfies the lenient criteria for this check.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch LFXV2-618

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 2b20268 and 03db5c8.

⛔ Files ignored due to path filters (1)

• charts/lfx-platform/Chart.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

• charts/lfx-platform/Chart.yaml (2 hunks)
• charts/lfx-platform/values.yaml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

• charts/lfx-platform/values.yaml
• charts/lfx-platform/Chart.yaml

---

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

[Copilot]

Pull Request Overview

Adds the new lfx-v2-mailing-list-service to the LFX Platform Helm chart by declaring it as a dependency and supplying default values, along with a version bump of the parent chart.

• Added new Helm dependency entry for lfx-v2-mailing-list-service
• Introduced default values (currently enabled) for the new service
• Bumped chart version from 0.3.2 to 0.3.3

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File	Description
charts/lfx-platform/Chart.yaml	Added dependency block for mailing list service and bumped chart version
charts/lfx-platform/values.yaml	Added default values (including enabled flag) for mailing list service

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}