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

cherrypick-1.1 ui: fix undefined.replica_id errors #18941

Merged
merged 1 commit into from
Oct 2, 2017

Conversation

BramGruneir
Copy link
Member

Cherry-picking #18916
cc @cockroachdb/release

And as a consequence of this, also display when a replica is awaiting being gced.

Fixes #18035.

And as a concequence of this, also display when a replica is awaiting being gced.

Fixes cockroachdb#18035.
@BramGruneir BramGruneir requested a review from a team October 2, 2017 16:49
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@BramGruneir
Copy link
Member Author

There is some debate as to if this is valid for 1.1. I think it is, as currently the range debug page will fail to load on any range that has some replicas awaiting GC.

@couchand couchand changed the title ui: fix undefined.replica_id errors cerrypick-1.1 ui: fix undefined.replica_id errors Oct 2, 2017
@couchand couchand changed the title cerrypick-1.1 ui: fix undefined.replica_id errors cherrypick-1.1 ui: fix undefined.replica_id errors Oct 2, 2017
@bdarnell
Copy link
Contributor

bdarnell commented Oct 2, 2017

:lgtm:


Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@BramGruneir BramGruneir merged commit fc895f3 into cockroachdb:release-1.1 Oct 2, 2017
@BramGruneir BramGruneir deleted the cherrypick-18916 branch October 2, 2017 17:20
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.

4 participants