Skip to content

feat: add nats#8

Open
lusu007 wants to merge 1 commit intomainfrom
feat/nats
Open

feat: add nats#8
lusu007 wants to merge 1 commit intomainfrom
feat/nats

Conversation

@lusu007
Copy link
Contributor

@lusu007 lusu007 commented Feb 8, 2026

Pull Request

Description

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • ♻️ Refactoring
  • 📚 Documentation
  • 🔧 Chore

Related Issues

None

Testing

  • Unit tests pass
  • Manual testing completed
  • New tests added for new functionality

Checklist

  • I have performed a self-review of my own code
  • Tests have been added/updated and pass (if needed)
  • Documentation has been updated (if needed)

@lusu007 lusu007 requested a review from ItsKev February 8, 2026 16:31
@lusu007 lusu007 self-assigned this Feb 8, 2026
Copilot AI review requested due to automatic review settings February 8, 2026 16:31
Copy link

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

Adds NATS as a local-dev infrastructure component managed via Helmfile, with a dedicated values file for a single-node JetStream-enabled setup.

Changes:

  • Add the NATS Helm repository and a new nats release in helmfile.yaml.
  • Introduce values/nats.values.yaml to configure single-replica NATS + JetStream persistence for local dev.
  • Remove the “Service Access” section from README.md.

Reviewed changes

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

File Description
values/nats.values.yaml New NATS chart values for local single-node + JetStream PVC persistence and resource limits.
helmfile.yaml Adds NATS chart repo and deploys a nats release into the infra namespace.
README.md Deletes the prior “Service Access” instructions section.

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

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.

1 participant