Skip to content

feat(chart): add .Values.extraResources#322

Merged
choufraise merged 2 commits into
WirelessCar:mainfrom
madestreel:feature/add-extra-resources
Jun 9, 2026
Merged

feat(chart): add .Values.extraResources#322
choufraise merged 2 commits into
WirelessCar:mainfrom
madestreel:feature/add-extra-resources

Conversation

@madestreel

Copy link
Copy Markdown

Goal is to allow users of the helm chart to deploy extra resources along nauth. This is useful to pre-configure a NatsCluster or run the operator-bootstrap within a job.

@madestreel madestreel requested a review from a team as a code owner June 4, 2026 08:35

@choufraise choufraise left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the contribution. This is a useful addition for the NAuth chart.

I’m good with the direction, but I’d like the review comments addressed before we merge. Once those are fixed, this should be straightforward to approve.

Comment thread charts/nauth/templates/extra_resources.yaml Outdated
Comment thread charts/nauth/values.yaml
Comment thread charts/nauth/values.yaml
@choufraise

Copy link
Copy Markdown
Collaborator

@madestreel , please also ensure that the commit(s) are signed-off (please see CONTRIBUTING.md)

@madestreel madestreel force-pushed the feature/add-extra-resources branch 7 times, most recently from 570117a to 524fb0c Compare June 8, 2026 13:42
@madestreel

Copy link
Copy Markdown
Author

@choufraise Hi, thanks for the suggestions. I did the required changes and signed the commits

@choufraise

choufraise commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

@madestreel , sorry for the inconvenience, but may I please ask you to rebase this branch on top of origin/main? 🙏 We will need to fix the PR workflow for external PRs, see #331, but I can't say when we'll find the time to fix this.

@madestreel madestreel force-pushed the feature/add-extra-resources branch from 524fb0c to 358bb6f Compare June 9, 2026 06:54
Maxime de Streel added 2 commits June 9, 2026 09:00
Signed-off-by: madestreel <maximedestreel@gmail.com>
Signed-off-by: madestreel <maximedestreel@gmail.com>
@madestreel madestreel force-pushed the feature/add-extra-resources branch from 358bb6f to bc7722c Compare June 9, 2026 07:00
@choufraise choufraise merged commit bc7722c into WirelessCar:main Jun 9, 2026
6 checks passed
@madestreel madestreel deleted the feature/add-extra-resources branch June 9, 2026 09:25
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.

2 participants