Skip to content

mongodb additional config#12

Merged
ihaardik merged 1 commit intomainfrom
mongodb-additional-fix-config
Apr 5, 2026
Merged

mongodb additional config#12
ihaardik merged 1 commit intomainfrom
mongodb-additional-fix-config

Conversation

@Hakandede
Copy link
Copy Markdown
Member

No description provided.

@ihaardik ihaardik merged commit ccd40d1 into main Apr 5, 2026
6 checks passed
Copy link
Copy Markdown

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 support for passing extra mongod configuration through the MongoDBCommunity Helm template, enabling users to customize MongoDB behavior via chart values.

Changes:

  • Conditionally renders spec.additionalMongodConfig in the MongoDBCommunity CR based on .Values.mongodb.additionalMongodConfig.

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

Comment on lines +110 to +112
{{- if .Values.mongodb.additionalMongodConfig }}
additionalMongodConfig:
{{- toYaml .Values.mongodb.additionalMongodConfig | nindent 4 }}
Copy link

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

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

This introduces a new values key (mongodb.additionalMongodConfig) but the chart’s values.yaml / README don’t document it, and values.schema.json doesn’t constrain its type. Please add this option to values.yaml (with an empty map default) and document it in the README; consider adding a schema entry as an object to give users validation/autocomplete.

Copilot uses AI. Check for mistakes.
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