Skip to content

jobs: downgrade panic to log in deprecated path#54114

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
ajwerner:ajwerner/fix-version-upgrade
Sep 9, 2020
Merged

jobs: downgrade panic to log in deprecated path#54114
craig[bot] merged 1 commit intocockroachdb:masterfrom
ajwerner:ajwerner/fix-version-upgrade

Conversation

@ajwerner
Copy link
Copy Markdown
Contributor

@ajwerner ajwerner commented Sep 9, 2020

This must have changed recently. I'll investigate what happened later.

Fixes #54079

Release justification: fixes for high-priority or high-severity bugs in
existing functionality

Release note: None

This must have changed recently. I'll investigate what happened later.

Release justification: fixes for high-priority or high-severity bugs in
existing functionality

Release note: None
@ajwerner ajwerner requested a review from spaskob September 9, 2020 13:25
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@ajwerner
Copy link
Copy Markdown
Contributor Author

ajwerner commented Sep 9, 2020

cc @tbg

@tbg
Copy link
Copy Markdown
Member

tbg commented Sep 9, 2020

Ah, nice. Then I'll hold off on the skip. Thanks for jumping on this.

Copy link
Copy Markdown
Member

@tbg tbg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will also need a backport to the release branch. Let me know what changed to trigger this now when you find out, I have no idea. Maybe @irfansharif's work on the server startup path jiggled things around ever so slightly?

Copy link
Copy Markdown
Contributor

@spaskob spaskob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is concerning.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @spaskob)

@ajwerner
Copy link
Copy Markdown
Contributor Author

ajwerner commented Sep 9, 2020

This one is concerning.

Yes, it is. I'll dig later.

@ajwerner
Copy link
Copy Markdown
Contributor Author

ajwerner commented Sep 9, 2020

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 9, 2020

Build succeeded:

@craig craig bot merged commit d3865d4 into cockroachdb:master Sep 9, 2020
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.

roachtest: acceptance/version-upgrade failed

4 participants