Skip to content

Conversation

alanprot
Copy link
Member

What this PR does:
Expose documentation for change merged on #5855

This feature is still experimental but we can document it.

Which issue(s) this PR fixes:
Fixes #

Checklist

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

@alanprot alanprot requested a review from yeya24 April 25, 2024 00:44
@@ -176,9 +176,6 @@ func NewLifecycler(
}

zone := cfg.Zone
if zone != "" {
level.Warn(logger).Log("msg", "experimental feature in use", "feature", "Zone aware replication")
Copy link
Contributor

Choose a reason for hiding this comment

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

Wondering if we need to clean up other doc in Cortex website?

Copy link
Contributor

Choose a reason for hiding this comment

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

But we can clean up all in another pr

@yeya24 yeya24 merged commit 41f2d0f into cortexproject:master Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants