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

workload/schemachanger: stabilize schema changer workload #83838

Merged
merged 2 commits into from
Jul 7, 2022

Conversation

fqazi
Copy link
Collaborator

@fqazi fqazi commented Jul 5, 2022

This pull request will do the following, to help stabilize the workload under CI:

  1. Temporarily disable survive/primary region-related database alters to avoid (workload/schemachanger: re-enable survive/primary region operations again #83831)
  2. Add retry logic for unknown schema errors which are due to (sql: replace internalLookupCtx with descs.Collection interfaces #64673)

fqazi added 2 commits July 5, 2022 14:41
Previously, the survive/primary region operations inside
the schemachange workload would hit would hit "error: attempting
to update zone configuration for database schemachange which contains
a modified field "voter_constraints". This led to CI failures, while
the root cause is somewhere inside our zone config code. To address
this we are temporarily disabling these ops stability reasons.

Release note: None
Previously, the schemachange workload did not detect unknown
schema error due to cockroachdb#6467. This was another spot that is frequently
being hit in our CI environment. To address this, this patch
will introduce the unknown schema retry logic.

Release note: None
@fqazi fqazi requested a review from ajwerner July 5, 2022 19:46
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
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.

:lgtm: thanks!

Reviewed 1 of 1 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @fqazi)

@fqazi
Copy link
Collaborator Author

fqazi commented Jul 6, 2022

@ajwerner TFTR!

bors r+

@craig
Copy link
Contributor

craig bot commented Jul 7, 2022

Build succeeded:

@craig craig bot merged commit a0b1222 into cockroachdb:master Jul 7, 2022
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