-
Notifications
You must be signed in to change notification settings - Fork 306
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
DAOS-16001 placement: fix cases for delay_rebuild #14558
Conversation
Bug-tracker data: |
@cdavis28 I haven't been able to reproduce the corruption with this cherry-pick |
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.
LGTM. No errors found by checkpatch.
Test stage Unit Test on EL 8.8 completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-14558/1/testReport/ |
Test stage Unit Test with memcheck on EL 8.8 completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-14558/1/testReport/ |
Features: rebuild Allow-unstable-test: true Required-githooks: true Signed-off-by: Xuezhao Liu <[email protected]>
f186a50
to
3ae5b27
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.
LGTM. No errors found by checkpatch.
Losing track of pending code changes... |
Yes, this is the one that fixes corruption on reintegration with delay_rebuild |
my PR #14557 with a few CI test case failures, but I checked looks not really related. I changed a few test yaml file and test again. no change in functional code compared to this PR. |
Placement fixes for delayed rebuild * Note: this is just a partial backport of stuff that changed since we backported it (See #14558) before it landed on master branch. Signed-off-by: Xuezhao Liu <[email protected]>
Placement fixes for delayed rebuild * Note: this is just a partial backport of tests that changed since we backported it (See #14558) before it landed on master branch. Signed-off-by: Xuezhao Liu <[email protected]>
Features: rebuild
Allow-unstable-test: true
Required-githooks: true
Before requesting gatekeeper:
Features:
(orTest-tag*
) commit pragma was used or there is a reason documented that there are no appropriate tags for this PR.Gatekeeper: