Skip to content

API: ks_tablets#4207

Closed
shlomi-noach wants to merge 12 commits intovitessio:masterfrom
github:api-ks-tablets
Closed

API: ks_tablets#4207
shlomi-noach wants to merge 12 commits intovitessio:masterfrom
github:api-ks-tablets

Conversation

@shlomi-noach
Copy link
Copy Markdown
Contributor

This PR adds an API endpoint to vtctld:

  • /api/ks_tables/my_keyspace
  • /api/ks_tables/my_keyspace/-80

The API returns a list of all tablets for a given keyspace, optionally further filtered by shard.

The context for this PR is https://github.com/github/freno, and in particular github/freno#70: giving the freno throttling tool an easy way to determine the roster of MySQL servers (host & port) it needs to poll for replication lag.

cc @github/database-infrastructure

@sougou
Copy link
Copy Markdown
Contributor

sougou commented Sep 24, 2018

You need to sign your commits :)

@shlomi-noach
Copy link
Copy Markdown
Contributor Author

shlomi-noach commented Oct 2, 2018

You need to sign your commits :)

The steps illustrated in https://github.com/vitessio/vitess/pull/4207/checks?check_run_id=18762836 caused many more commits to be signed by me, which were not, in fact, my commits. This leads to a merge conflict now.

I'm likely to close this PR and open a new one.

@shlomi-noach
Copy link
Copy Markdown
Contributor Author

Superseded by #4232, and I will have local cleanup to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants