Skip to content

Feat/presets#25

Merged
alix-graylog merged 2 commits intomainfrom
feat/presets
Aug 22, 2025
Merged

Feat/presets#25
alix-graylog merged 2 commits intomainfrom
feat/presets

Conversation

@monrax
Copy link
Copy Markdown
Collaborator

@monrax monrax commented Aug 19, 2025

This adds support for the following top-level values:

  • version: set both Graylog and DataNode versions in one go.
  • provider: render provider-specific resources/fields.
  • size: set number of replicas according to a size preset defined in Support Cluster Size Presets #23 (CPU and memory are NOT included in this PR -- they will be addressed in a future PR).

Notes for Reviewers

  • The commit history must be preserved - please use the rebase-merge or standard merge option instead of squash-merge
  • Sync up with the author before merging

How to test?

Version

helm install graylog ./graylog -n graylog --set version='6.3'

kubectl describe sts graylog -n graylog
kubectl describe sts graylog-datanode -n graylog

# both graylog and datanode containers should have 6.3 tag

Providers

Nothing to test for now.

Replicas

helm install graylog ./graylog -n graylog --set size=poc

kubectl get pods -n graylog

# only one graylog pod and one datanode pod should be scheduled

@monrax monrax requested a review from williamtrelawny August 19, 2025 06:59
@monrax
Copy link
Copy Markdown
Collaborator Author

monrax commented Aug 19, 2025

@williamtrelawny this has been rebased on top of the current update/docs branch. Please take a look at PR #24 before reviewing this one!

@monrax monrax marked this pull request as draft August 20, 2025 17:21
@monrax
Copy link
Copy Markdown
Collaborator Author

monrax commented Aug 20, 2025

Turning this into a draft PR until #24 is merged.

@monrax monrax mentioned this pull request Aug 20, 2025
2 tasks
@monrax monrax marked this pull request as ready for review August 22, 2025 16:11
Copy link
Copy Markdown
Collaborator

@alix-graylog alix-graylog left a comment

Choose a reason for hiding this comment

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

Ship it!

@alix-graylog alix-graylog merged commit c1eb646 into main Aug 22, 2025
1 check passed
@williamtrelawny williamtrelawny linked an issue Aug 29, 2025 that may be closed by this pull request
@monrax monrax deleted the feat/presets branch December 5, 2025 11:08
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.

Support Cluster Size Presets

2 participants