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

10.6 mdev-31905 sj #356

Open
wants to merge 1 commit into
base: 10.6
Choose a base branch
from
Open

10.6 mdev-31905 sj #356

wants to merge 1 commit into from

Conversation

sjaakola
Copy link

No description provided.

@sjaakola sjaakola force-pushed the 10.6-MDEV-31905-sj branch 4 times, most recently from 182c19e to ef77a17 Compare November 9, 2023 13:43
Copy link

@janlindstrom janlindstrom left a comment

Choose a reason for hiding this comment

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

Looks good but added test case has warnings on Jenkings and need investigation are they expected or sign of something is wrong. Is 10.6 really earliers release where this GTID inconsistency reproduces?

@sjaakola
Copy link
Author

test_pr

@sjaakola sjaakola force-pushed the 10.6-MDEV-31905-sj branch 2 times, most recently from a8ce579 to fb5872f Compare November 15, 2023 19:31
@janlindstrom
Copy link

Latest version seems to fail even more.

This commit fixes GTID inconsistency which was injected by mariabackup SST.
Donor node now writes new info file: donor_galera_info, which is streamed
along the mariabackup donation to the joiner node. The donor_galera_info
file contains both GTID and gtid domain_id, and joiner will use these to
initialize the GTID state.

Commit has new mtr test case: galera_3nodes.galera_gtid_consistency, which
exercises potentially harmful mariabackup SST scenarios. The test has also
scenario with IST joining.
@janlindstrom
Copy link

@sjaakola Regression testing is now as good as it can be.

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.

2 participants