Skip to content

Conversation

@kishorekg1999
Copy link

What this PR does:
pdates the documentation and code comments for the Ruler's frontend_address configuration to accurately reflect that the Ruler queries both Ingesters and Store Gateways directly when frontend_address is not configured. Previously, the comments incorrectly stated it only queried Ingesters.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Copy link
Member

@SungJin1212 SungJin1212 left a comment

Choose a reason for hiding this comment

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

thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 30, 2025
@kishorekg1999 kishorekg1999 reopened this Dec 30, 2025
Co-authored-by: SungJin1212 <[email protected]>
Signed-off-by: kishorekg1999 <[email protected]>
@kishorekg1999
Copy link
Author

@SungJin1212 the integration tests have been failing. Are those tests flaky ? Is there a way to get an exception ?

@SungJin1212
Copy link
Member

@SungJin1212 the integration tests have been failing. Are those tests flaky ? Is there a way to get an exception ?

yeah, the query fuzz test is flaky now. can you run make doc again?

Signed-off-by: Kishore K G <[email protected]>
@kishorekg1999
Copy link
Author

@SungJin1212 can you please approve the workflow ?

@kishorekg1999
Copy link
Author

@SungJin1212 The test is continuously failing, can you please approve the workflow again ? Is there a way to rerun the failed tests without reopening the PR so that we can avoid workflow approval ?

@SungJin1212
Copy link
Member

@kishorekg1999
Failed tests will be retried automatically, so you can leave it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/documentation lgtm This PR has been approved by a maintainer size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants