Skip to content

Conversation

@vlad-diachenko
Copy link
Contributor

bumped version of grafana's memberlist fork to have the fix that allows to disable labels check that is required for the migration.
grafana/memberlist#3

…ws to disable labels check that is required for the migration.
@vlad-diachenko vlad-diachenko requested a review from a team as a code owner July 11, 2022 11:49
@grafanabot
Copy link
Contributor

./tools/diff_coverage.sh ../loki-main/test_results.txt test_results.txt ingester,distributor,querier,querier/queryrange,iter,storage,chunkenc,logql,loki

Change in test coverage per package. Green indicates 0 or positive change, red indicates that test coverage for a package fell.

+           ingester	0%
+        distributor	0%
+            querier	0%
+ querier/queryrange	0%
+               iter	0%
+            storage	0%
+           chunkenc	0%
+              logql	0%
+               loki	0%

Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

LGTM

@vlad-diachenko vlad-diachenko merged commit ca0fd01 into main Jul 11, 2022
@vlad-diachenko vlad-diachenko deleted the buped_version_of_memberlist_fork branch July 11, 2022 15:41
lxwzy pushed a commit to lxwzy/loki that referenced this pull request Nov 7, 2022
…ws to disable labels check that is required for the migration. (grafana#6655)
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.

4 participants