Skip to content

[LFXV2-1064] Add survey service chart dependency#109

Merged
andrest50 merged 2 commits intomainfrom
andrest50/surveys-service-chart
Feb 5, 2026
Merged

[LFXV2-1064] Add survey service chart dependency#109
andrest50 merged 2 commits intomainfrom
andrest50/surveys-service-chart

Conversation

@andrest50
Copy link
Contributor

@andrest50 andrest50 commented Feb 4, 2026

Summary

  • Added lfx-v2-survey-service chart dependency to the lfx-platform Helm chart
  • Configured the survey service with default settings and reloader annotations

Ticket

LFXV2-1064

🤖 Generated with Claude Code

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: Andres Tobon <andrest2455@gmail.com>
@andrest50 andrest50 requested review from a team and emsearcy as code owners February 4, 2026 18:30
Copilot AI review requested due to automatic review settings February 4, 2026 18:30
@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • charts/lfx-platform/Chart.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Walkthrough

This change adds a new Helm dependency declaration for lfx-v2-survey-service to the lfx-platform chart, including the dependency specification in Chart.yaml and its corresponding configuration values in values.yaml.

Changes

Cohort / File(s) Summary
Helm Chart Dependency Addition
charts/lfx-platform/Chart.yaml, charts/lfx-platform/values.yaml
Added new chart dependency lfx-v2-survey-service with OCI repository reference, version constraint ~0.1.0, and enabled condition. Added corresponding values configuration mirroring the existing voting service pattern, including domain and Stakater reloader annotation for automatic pod restarts.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly matches the main change: adding the lfx-v2-survey-service chart dependency to the Helm chart.
Description check ✅ Passed The description accurately covers the changeset, including the dependency addition and configuration with reloader annotations.
Linked Issues check ✅ Passed All requirements from LFXV2-1064 are met: the dependency is added to Chart.yaml, configuration is in values.yaml, service is enabled with reloader annotations.
Out of Scope Changes check ✅ Passed All changes are directly related to adding the lfx-v2-survey-service dependency; no out-of-scope modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch andrest50/surveys-service-chart

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the lfx-v2-survey-service as a new chart dependency to the LFX platform Helm chart, enabling survey functionality within the platform. The service is configured with default settings consistent with other similar services.

Changes:

  • Added lfx-v2-survey-service dependency to Chart.yaml with version constraint ~0.1.0
  • Configured survey service in values.yaml with domain settings and reloader annotations

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
charts/lfx-platform/Chart.yaml Added lfx-v2-survey-service dependency configuration with OCI repository reference
charts/lfx-platform/values.yaml Added survey service configuration with domain settings and reloader annotations for automatic pod restarts

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: Andres Tobon <andrest2455@gmail.com>
@andrest50 andrest50 merged commit df34f0e into main Feb 5, 2026
4 checks passed
@andrest50 andrest50 deleted the andrest50/surveys-service-chart branch February 5, 2026 16:20
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.

3 participants