Skip to content

roachtest: jobs/mixed-versions applies from release 20.1.0#48515

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
spaskob:fix-version
May 7, 2020
Merged

roachtest: jobs/mixed-versions applies from release 20.1.0#48515
craig[bot] merged 1 commit intocockroachdb:masterfrom
spaskob:fix-version

Conversation

@spaskob
Copy link
Copy Markdown
Contributor

@spaskob spaskob commented May 6, 2020

Jobs infrastructure was unstable prior to 20.1 in terms of the behavior
of PAUSE/CANCEL JOB commands which were best effort and relied on the
job itself to detect the request. These were fixed by introducing new job
state machine states Status{Pause,Cancel}Requested. This test purpose
is to to test the state transitions of jobs from paused to resumed and
vice versa in order to detect regressions in the work done for 20.1.

Fixes #48141.

Release note: none.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@spaskob spaskob requested review from andreimatei and tbg May 6, 2020 22:33
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.

Mind giving a reason for the MinVersion?

Copy link
Copy Markdown
Contributor Author

@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.

Changed the PR description and added a comment in the code.

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

@spaskob
Copy link
Copy Markdown
Contributor Author

spaskob commented May 7, 2020

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 7, 2020

Build succeeded

@craig craig bot merged commit 6932a67 into cockroachdb:master May 7, 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: jobs/mixed-versions failed

3 participants