Skip to content

fix: add replicas configuration for KV buckets#70

Open
AlanSherman wants to merge 1 commit intomainfrom
asherman/nats-replicas
Open

fix: add replicas configuration for KV buckets#70
AlanSherman wants to merge 1 commit intomainfrom
asherman/nats-replicas

Conversation

@AlanSherman
Copy link

No description provided.

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>
Copilot AI review requested due to automatic review settings March 18, 2026 22:57
@AlanSherman AlanSherman requested review from a team and emsearcy as code owners March 18, 2026 22:57
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 explicit replicas configuration for the JetStream KeyValue (KV) buckets managed by the lfx-v1-sync-helper Helm chart, allowing KV bucket replication factor to be set via chart values.

Changes:

  • Added natsResources.*.replicas defaults for the three KV buckets in values.yaml.
  • Rendered the spec.replicas field for those KeyValue resources in the Helm template.

Reviewed changes

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

File Description
charts/lfx-v1-sync-helper/values.yaml Introduces default replicas: 1 values for each KV bucket.
charts/lfx-v1-sync-helper/templates/nats-resources.yaml Emits spec.replicas for each KeyValue resource using the configured values.

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

You can also share your feedback on Copilot code review. Take the survey.

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