-
Notifications
You must be signed in to change notification settings - Fork 305
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-15931 rebuild: fix data corruption caused by partial parity rebu… #14519
Conversation
…ild epoch Rebuild code change: 1. __migrate_fetch_update_parity(), fix a bug when set partial replica rebuild epoch for parity shard rebuild. 2. __migrate_fetch_update_bulk() should carry DIOF_FOR_MIGRATION flag, 3. migrate_fetch_update_parity() parameter fix when calling __migrate_fetch_update_parity(). EC aggregation change: 1. ds_obj_ec_rep_handler() and ds_obj_ec_agg_handler(), the vos_update_begin() should carry VOS_OF_REBUILD to avoid -DER_VOS_PARTIAL_UPDATE failure. 2. give more chance to abort EC agg when rebuild started, to save conflict window. includes backports of DAOS-15007 object: fix EC aggregation's ap_min_unagg_eph set (#13875) DAOS-15262 vos: Fix probe issue in vos iterator (#13918) DAOS-14908 vos: Reduce aggregation conflicts (#14143) Signed-off-by: Jeff Olivier <[email protected]> Signed-off-by: Xuezhao Liu <[email protected]> Signed-off-by: Niu Yawei <[email protected]>
Bug-tracker data: |
mrone, oh, &iod, 1, fetch_eph, update_eph, | ||
DIOF_EC_RECOV_FROM_PARITY | DIOF_FOR_MIGRATION, ds_cont); |
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.
mrone, oh, &iod, 1, fetch_eph, update_eph, | |
DIOF_EC_RECOV_FROM_PARITY | DIOF_FOR_MIGRATION, ds_cont); | |
mrone, oh, &iod, 1, fetch_eph, update_eph, | |
DIOF_EC_RECOV_FROM_PARITY | DIOF_FOR_MIGRATION, ds_cont); |
#define VOS_TX_LOG_FAIL(rc, ...) \ | ||
do { \ | ||
bool __is_err = true; \ | ||
\ |
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.
\ | |
\ |
#define VOS_TX_TRACE_FAIL(rc, ...) \ | ||
do { \ | ||
bool __is_err = true; \ | ||
\ |
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.
\ | |
\ |
Test stage checkpatch completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-14519/1/execution/node/204/log |
…ild epoch (#14512)
Rebuild code change:
EC aggregation change:
includes backports of
DAOS-15007 object: fix EC aggregation's ap_min_unagg_eph set (#13875) DAOS-15262 vos: Fix probe issue in vos iterator (#13918) DAOS-14908 vos: Reduce aggregation conflicts (#14143)
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: