Skip to content

storage: fix use of context with closed trace#25581

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
a-robinson:renewleasectx
May 16, 2018
Merged

storage: fix use of context with closed trace#25581
craig[bot] merged 1 commit intocockroachdb:masterfrom
a-robinson:renewleasectx

Conversation

@a-robinson
Copy link
Copy Markdown
Contributor

Fixes #25575

Release note: None

@a-robinson a-robinson requested review from a team and bdarnell May 16, 2018 15:43
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@bdarnell
Copy link
Copy Markdown
Contributor

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from Reviewable

@a-robinson
Copy link
Copy Markdown
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request May 16, 2018
24945: server: exempt healthcheck endpoint from authentication check r=couchand a=vilterp

Otherwise, you get the red "connection lost" banner until you log in.

This will have no effect until the auth mux is put on the request path (#24944).

Fixes #24942 

25581: storage: fix use of context with closed trace r=a-robinson a=a-robinson

Fixes #25575

Release note: None

Co-authored-by: Pete Vilter <vilterp@cockroachlabs.com>
Co-authored-by: Alex Robinson <alexdwanerobinson@gmail.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 16, 2018

Build succeeded

@craig craig bot merged commit 248b38e into cockroachdb:master May 16, 2018
mvijaykarthik pushed a commit to mvijaykarthik/cockroach that referenced this pull request Sep 25, 2018
craig bot pushed a commit that referenced this pull request Sep 25, 2018
30614: backport-2.0: storage: fix use of context with closed trace r=tschottdorf a=mvijaykarthik

Backport 1/1 commits from #25581

---

Fixes #25575

Release note: None

Co-authored-by: Alex Robinson <alexdwanerobinson@gmail.com>
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.

3 participants