Validate tenantID on single resolver by SungJin1212 · Pull Request #6727 · cortexproject/cortex · GitHub
Skip to content

Validate tenantID on single resolver#6727

Merged
yeya24 merged 1 commit into
cortexproject:masterfrom
SungJin1212:Add-tenant-validation-to-single-resolver
May 22, 2025
Merged

Validate tenantID on single resolver#6727
yeya24 merged 1 commit into
cortexproject:masterfrom
SungJin1212:Add-tenant-validation-to-single-resolver

Conversation

@SungJin1212

@SungJin1212 SungJin1212 commented Apr 29, 2025

Copy link
Copy Markdown
Member

Currently, a tenantID validation has been applied only for the MultiResolver.
This PR adds a validation of the tenantID to the single resolver. The validation rules follow the docs.

Which issue(s) this PR fixes:
Fixes #

Checklist

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

Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>

@CharlieTLe CharlieTLe left a comment

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.

Thanks!

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label May 3, 2025
@SungJin1212 SungJin1212 requested a review from yeya24 May 21, 2025 01:06

@yeya24 yeya24 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@yeya24 yeya24 merged commit 5df4598 into cortexproject:master May 22, 2025
@yeya24

yeya24 commented May 22, 2025

Copy link
Copy Markdown
Contributor

@SungJin1212

SungJin1212 commented May 23, 2025

Copy link
Copy Markdown
Member Author

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

Labels

lgtm This PR has been approved by a maintainer size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants