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

upgrademanager: change host cluster to storage cluster #94669

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

healthy-pod
Copy link
Contributor

Release note: None
Epic: none

@healthy-pod healthy-pod requested a review from a team January 3, 2023 23:07
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@healthy-pod healthy-pod requested a review from knz January 4, 2023 05:51
Copy link
Contributor

@knz knz 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 :)

@healthy-pod
Copy link
Contributor Author

bors r=knz

@craig
Copy link
Contributor

craig bot commented Jan 4, 2023

Build succeeded:

@craig craig bot merged commit 4dc80b0 into cockroachdb:master Jan 4, 2023
healthy-pod pushed a commit to healthy-pod/cockroach that referenced this pull request Jan 9, 2023
In cockroachdb#94669, we changed usages of `host cluster` to
`storage cluster` because `host cluster` is specific
to `CC Serverless`.

This change updates the expected error in `multitenant-upgrade`
and some other usages of `host cluster`.

Epic: none
Release note: None

Closes cockroachdb#94748
craig bot pushed a commit that referenced this pull request Jan 11, 2023
94947: roachtest: fix multitenant-upgrade expected error r=healthy-pod a=healthy-pod

In #94669, we changed usages of `host cluster` to
`storage cluster` because `host cluster` is specific to `CC Serverless`.

This change updates the expected
error in `multitenant-upgrade` and some other usages of `host cluster`.

Epic: none
Release note: None

Closes #94748

Co-authored-by: healthy-pod <[email protected]>
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