Skip to content

feat/logstash_stack_monitoring#235

Merged
NavidSassan merged 5 commits intomainfrom
feat/logstash_stack_monitoring
Apr 28, 2026
Merged

feat/logstash_stack_monitoring#235
NavidSassan merged 5 commits intomainfrom
feat/logstash_stack_monitoring

Conversation

@ebuerki-lf
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Member

@NavidSassan NavidSassan left a comment

Choose a reason for hiding this comment

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

also update the changelog (mention the new variables)

Comment thread roles/logstash/templates/etc/logstash/logstash.yml.j2
@NavidSassan
Copy link
Copy Markdown
Member

and make sure that the commit message follows our guidelines (force push to fix)


tags:
- 'always'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

missing another empty newline (2 blank lines between blocks)

Comment thread roles/logstash/README.md
@NavidSassan NavidSassan merged commit 47fa201 into main Apr 28, 2026
4 checks passed
@NavidSassan NavidSassan deleted the feat/logstash_stack_monitoring branch April 28, 2026 15:14
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