Skip to content

upgradejob: initialize the kvdb on TenantDeps#98489

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-fix-upgrade-job-initialization
Mar 13, 2023
Merged

upgradejob: initialize the kvdb on TenantDeps#98489
craig[bot] merged 1 commit intocockroachdb:masterfrom
jeffswenson:jeffswenson-fix-upgrade-job-initialization

Conversation

@jeffswenson
Copy link
Copy Markdown
Collaborator

The KVDB field was created as part of PR #93218, but it was never initialized.

Part of #94843

Release Note: None

The KVDB field was created as part of PR cockroachdb#93218, but it was never
initialized.

Part of cockroachdb#94843

Release Note: None
@jeffswenson jeffswenson requested review from a team and ajwerner March 13, 2023 13:26
@blathers-crl
Copy link
Copy Markdown

blathers-crl bot commented Mar 13, 2023

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@jeffswenson
Copy link
Copy Markdown
Collaborator Author

This change is cherry-picked into the following PRs:
#98445
#98446
#98463

Copy link
Copy Markdown
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

Thank you!

@jeffswenson
Copy link
Copy Markdown
Collaborator Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 13, 2023

Build succeeded:

@craig craig bot merged commit 9ed78bf into cockroachdb:master Mar 13, 2023
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