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

Merge feat-remote-write-metadata to v2.17.1-grafana #4

Merged
merged 3 commits into from
Apr 3, 2020

Conversation

rfratto
Copy link
Member

@rfratto rfratto commented Apr 3, 2020

Mostly unchanged from the shared-interner branch but I rebased my code out of it so it's just the remote_write metadata propagation code now.

Similarly to the WAL watcher, its purpose is to observe the scrape manager and pull metadata. Then, send it to a remote storage.

Signed-off-by: gotjosh <josue@grafana.com>
@rfratto rfratto requested a review from gotjosh April 3, 2020 15:02
@gotjosh
Copy link

gotjosh commented Apr 3, 2020

Gonna take a moment to apply your feedback (and anything that might come up out of the current issue in ops-tools). Give me a couple of hours before emerging this one.

Signed-off-by: gotjosh <josue@grafana.com>
@gotjosh
Copy link

gotjosh commented Apr 3, 2020

I'm not sure how you managed to spot that extra space, great find.

@rfratto rfratto merged commit 09b8b10 into v2.17.1-grafana Apr 3, 2020
@rfratto
Copy link
Member Author

rfratto commented Apr 3, 2020

I'm going to rename the release branch to release-2.17.1-grafana so it works better with Go tooling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants