-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
mixedversion: introduce deployment modes #124405
mixedversion: introduce deployment modes #124405
Conversation
Your pull request contains more than 1000 changes. It is strongly encouraged to split big PRs into smaller chunks. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
This commit adds the notion of a "deployment mode" to mixedversion tests. Currently, we only define a `system-only` deployment mode, corresponding to the deployment performed on all mixedversion tests to date: cockroach is started without any tenants (virtual clusters), and client applications talk directly to the "system interface". This commit adds the functionality necessary for tests to be able to select what deployment modes make sense for them, if they verify something specific to some deployment mode. In the future, more deployment modes will be added: tenants in shared-process mode, and external process (serverless-style) tenants. Epic: none Release note: None
4662afe
to
629e407
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
@@ -441,7 +444,7 @@ func Test_stepSelectorFilter(t *testing.T) { | |||
name: "no filter", | |||
predicate: func(*singleStep) bool { return true }, | |||
expectedAllSteps: true, | |||
expectedRandomStepType: runHookStep{}, | |||
expectedRandomStepType: restartWithNewBinaryStep{}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's rather subtle but prng determinism corresponds to both seed
(which hasn't changed) and the number of calls to prng. The latter did indeed change; on L630, we've added a new call to prng.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct!
TFTRs! bors r=herkolategan,DarrylWong,srosenberg |
This commit adds the notion of a "deployment mode" to mixedversion tests. Currently, we only define a
system-only
deployment mode, corresponding to the deployment performed on all mixedversion tests to date: cockroach is started without any tenants (virtual clusters), and client applications talk directly to the "system interface". This commit adds the functionality necessary for tests to be able to select what deployment modes make sense for them, if they verify something specific to some deployment mode.In the future, more deployment modes will be added: tenants in shared-process mode, and external process (serverless-style) tenants.
Epic: none
Release note: None