Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stores: respect replica labels in LabelValues and LabelNames #7310

Merged

Conversation

MichaHoffmann
Copy link
Contributor

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Added acceptance test for a proxy store with replica labels.

Verification

Acceptance Tests

@MichaHoffmann
Copy link
Contributor Author

MichaHoffmann commented Apr 27, 2024

LabelNames and LabelValues tests fail because we dont drop replica label for LabelNames or LabelValues gRPCs

@MichaHoffmann MichaHoffmann force-pushed the mhoffm-acceptance-test-for-proxy-store-with-replicas branch from 3a9d068 to 044e5ed Compare April 27, 2024 07:33
@pull-request-size pull-request-size bot added size/L and removed size/M labels Apr 27, 2024
@MichaHoffmann MichaHoffmann force-pushed the mhoffm-acceptance-test-for-proxy-store-with-replicas branch from c2c9de3 to 83ceb3e Compare April 27, 2024 08:11
@MichaHoffmann MichaHoffmann changed the title Proxy: acceptance test for proxy store with replica labels Stores: acceptance test for stores with replica labels Apr 27, 2024
@MichaHoffmann MichaHoffmann changed the title Stores: acceptance test for stores with replica labels Stores: respect replica labels in LabelValues and LabelNames Apr 27, 2024
@MichaHoffmann MichaHoffmann force-pushed the mhoffm-acceptance-test-for-proxy-store-with-replicas branch from 83ceb3e to a719ea3 Compare April 29, 2024 10:11
@MichaHoffmann MichaHoffmann marked this pull request as ready for review April 29, 2024 10:20
@MichaHoffmann MichaHoffmann merged commit bd74665 into main Apr 29, 2024
20 checks passed
Nashluffy pushed a commit to Nashluffy/thanos that referenced this pull request May 14, 2024
…io#7310)

* Proxy: acceptance test for proxy store with replica labels

Signed-off-by: Michael Hoffmann <[email protected]>

* Stores: handle replica labels in label_value and label_names grpcs

Signed-off-by: Michael Hoffmann <[email protected]>

---------

Signed-off-by: Michael Hoffmann <[email protected]>
Signed-off-by: mluffman <[email protected]>
jnyi pushed a commit to jnyi/thanos that referenced this pull request Jun 1, 2024
…io#7310)

* Proxy: acceptance test for proxy store with replica labels

Signed-off-by: Michael Hoffmann <[email protected]>

* Stores: handle replica labels in label_value and label_names grpcs

Signed-off-by: Michael Hoffmann <[email protected]>

---------

Signed-off-by: Michael Hoffmann <[email protected]>
jnyi pushed a commit to jnyi/thanos that referenced this pull request Jun 4, 2024
…io#7310)

* Proxy: acceptance test for proxy store with replica labels

Signed-off-by: Michael Hoffmann <[email protected]>

* Stores: handle replica labels in label_value and label_names grpcs

Signed-off-by: Michael Hoffmann <[email protected]>

---------

Signed-off-by: Michael Hoffmann <[email protected]>
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