-
Notifications
You must be signed in to change notification settings - Fork 212
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
Fix newsi relinks ordering bug #4601
base: main
Are you sure you want to change the base?
Conversation
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.
Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 3/564 tests failed ⚠.
The first 10 failing tests are:
online_compaction
truncatesc
truncatesc_ufid_on_generated
b4ca270
to
4ef8a22
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.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 1/564 tests failed ⚠.
The first 10 failing tests are:
truncatesc_ufid_on_generated
Signed-off-by: Mark Hannum <[email protected]>
4ef8a22
to
ca3a6b9
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.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 4/564 tests failed ⚠.
The first 10 failing tests are:
insert_lots_ssl_generated [setup failure]
phys_rep_tiered
sc_resume
truncatesc_ufid_on_generated
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.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 2/564 tests failed ⚠.
The first 10 failing tests are:
diskspace_nollmeta_nostripe_generated
auth
Pagelogs are placed on filed-queues in a slightly different order when snapshot isolation is run directly on master. This PR fixes the issue, and adds the 'newsi.testopts' file to the newsi_on_master test.