fix(mongodb): let MongoDB container resources be configured - #176
Merged
Conversation
The operator owns the pod template of the StatefulSet it manages and injects its own resource defaults of 500m CPU and 500M memory per container and per init container. mongo-rs.yaml overrode serviceAccountName and affinity but never containers or initContainers, so nothing in the chart - no value, no --set - could change what a MongoDB pod requested. The init containers matter as much as the containers: a pod reserves max(max(initContainer), sum(containers)), so each merely copying a binary into a shared volume at 500m sets the pod's effective floor whenever the containers ask for less. Overriding only the containers is not enough. Adds mongodb.resources, mongodb.agentResources and mongodb.initResources, patched onto the operator-owned template by container name, with defaults sized for a production replica set rather than inherited from the operator. mongod gets 2Gi: WiredTiger sizes its cache from the container limit and an idle 3-member replica set already sits at ~340Mi, so the operator's 500M limit left almost nothing before an OOMKill. CPU goes the other way, since the same replica set idles under 50m per member. A MongoDB pod's reservation therefore moves from 1000m/800M to 300m/1152Mi, and ci/ci-values.yaml shrinks all three blocks to 150m/640Mi for CI. Existing installs get a rolling restart of the replica set on upgrade.
alix-graylog
marked this pull request as ready for review
August 21, 2026 21:00
bemcculley-graylog
approved these changes
Aug 25, 2026
alix-graylog
added a commit
that referenced
this pull request
Aug 27, 2026
No commit in the 1.0.0 to 2.0.0 window carried a breaking-change footer, so release-please had no breaking-changes section to generate. This adds the 16 found by rendering the chart at both versions and diffing resource names, StatefulSet immutable fields, security contexts and values keys. Also adds the mongodb entry from #176, which the parser dropped, corrects "starup" to "startup" in the #169 entry, and gives #160 a description of what it actually does. The publish job generates artifacthub.io/changes from this file, so these notes reach Artifact Hub as written.
alix-graylog
added a commit
that referenced
this pull request
Aug 27, 2026
* chore(main): release graylog 2.0.0 * chore(main): write the 2.0.0 breaking changes into the changelog No commit in the 1.0.0 to 2.0.0 window carried a breaking-change footer, so release-please had no breaking-changes section to generate. This adds the 16 found by rendering the chart at both versions and diffing resource names, StatefulSet immutable fields, security contexts and values keys. Also adds the mongodb entry from #176, which the parser dropped, corrects "starup" to "startup" in the #169 entry, and gives #160 a description of what it actually does. The publish job generates artifacthub.io/changes from this file, so these notes reach Artifact Hub as written. --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: alix-graylog <alix.fullerton@graylog.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sumarry
Exposing mogno container resource's requests and limits to the Graylog charts values.yaml
Description
This allows for creating more minimum viable tests for CICD
Related Issues