Skip to content

Add /ring to ingester & distributor, ability to remove token #162

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Nov 25, 2016
Merged

Add /ring to ingester & distributor, ability to remove token #162

merged 2 commits into from
Nov 25, 2016

Conversation

tomwilkie
Copy link
Contributor

Fixes #94

@tomwilkie
Copy link
Contributor Author

From #159:

IIRC, this time is from the ingester's clock, but the state is inferred from the distributor's clock.

Either way, this page should either show the distributor's opinion of the current time outside the table (so we have a basis for comparison), or the delta between the current time and the last heartbeat in the table.

@tomwilkie tomwilkie mentioned this pull request Nov 25, 2016
Merged
@tomwilkie tomwilkie merged commit 00ef739 into master Nov 25, 2016
@tomwilkie tomwilkie deleted the slash-ring branch November 25, 2016 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jml jml jml approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tomwilkie @jml