Skip to content

Conversation

JoaoBraveCoding
Copy link
Collaborator

What this PR does / why we need it:

Add a new alert to help users notice when LokiStack components haven't reached the ready state.

Which issue(s) this PR fixes:
https://issues.redhat.com/browse/LOG-5470

Special notes for your reviewer:

Checklist

  • Reviewed the CONTRIBUTING.md guide (required)
  • Documentation added
  • Tests updated
  • Title matches the required conventional commits format, see here
    • Note that Promtail is considered to be feature complete, and future development for logs collection will be in Grafana Alloy. As such, feat PRs are unlikely to be accepted unless a case can be made for the feature actually being a bug fix to existing behavior.
  • Changes that require user attention or interaction to upgrade are documented in docs/sources/setup/upgrade/_index.md
  • If the change is deprecating or removing a configuration option, update the deprecated-config.yaml and deleted-config.yaml files respectively in the tools/deprecated-config-checker directory. Example PR

Copy link
Collaborator

@xperimental xperimental left a comment

Choose a reason for hiding this comment

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

Some small comments, but works fine 👍

JoaoBraveCoding and others added 2 commits September 30, 2025 11:03
Co-authored-by: Robert Jacob <[email protected]>
Signed-off-by: Joao Marcal <[email protected]>
@JoaoBraveCoding JoaoBraveCoding changed the title feat(operator): Added LokistackPendingComponents alert feat(operator): Added LokistackComponentsNotReady alert Sep 30, 2025
@JoaoBraveCoding JoaoBraveCoding changed the title feat(operator): Added LokistackComponentsNotReady alert feat(operator): Added LokistackComponentsNotReadyCritical alert Sep 30, 2025
@xperimental xperimental changed the title feat(operator): Added LokistackComponentsNotReadyCritical alert feat(operator): Add warning alert for when LokiStack is not getting ready Oct 2, 2025
@xperimental xperimental enabled auto-merge (squash) October 2, 2025 14:20
@xperimental xperimental merged commit c47fe46 into grafana:main Oct 2, 2025
71 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants