Skip to content

DAOS-19591 container: two fixes for EC aggregation boundary reporting - #19014

Open
liuxuezhao wants to merge 1 commit into
release/2.6from
lxz/cont_agg_eph_sync_26
Open

DAOS-19591 container: two fixes for EC aggregation boundary reporting#19014
liuxuezhao wants to merge 1 commit into
release/2.6from
lxz/cont_agg_eph_sync_26

Conversation

@liuxuezhao

Copy link
Copy Markdown
Contributor

1.cont_refresh_vos_agg_eph_one() now creates the VOS container when it
does not exist locally (-DER_NONEXIST). That can happen when the
container was created while the target was excluded and the following
reintegration/extend migrated no record for it. Without the VOS
container the target never reports its EC aggregation epoch, so the
container service leader keeps the boundary of the whole container
pinned. As the leader only refreshes containers still present in RDB,
creating the VOS container here is safe.
Since container creation can yield and needs a deep stack, switch
ds_cont_tgt_refresh_agg_eph() from ds_pool_task_collective() to
ds_pool_thread_collective() with DSS_ULT_DEEP_STACK.

2.Track ea_start_ts, the time this leader started tracking a container,
and suppress the "Sluggish EC boundary reporting" warning when the
boundary jump is larger than the tracking age. Such a jump cannot have
been observed by this leader; it comes from the stale epoch loaded from
RDB at leader step-up and is not a real stall.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

1.cont_refresh_vos_agg_eph_one() now creates the VOS container when it
  does not exist locally (-DER_NONEXIST). That can happen when the
  container was created while the target was excluded and the following
  reintegration/extend migrated no record for it. Without the VOS
  container the target never reports its EC aggregation epoch, so the
  container service leader keeps the boundary of the whole container
  pinned. As the leader only refreshes containers still present in RDB,
  creating the VOS container here is safe.
  Since container creation can yield and needs a deep stack, switch
  ds_cont_tgt_refresh_agg_eph() from ds_pool_task_collective() to
  ds_pool_thread_collective() with DSS_ULT_DEEP_STACK.

2.Track ea_start_ts, the time this leader started tracking a container,
  and suppress the "Sluggish EC boundary reporting" warning when the
  boundary jump is larger than the tracking age. Such a jump cannot have
  been observed by this leader; it comes from the stale epoch loaded from
  RDB at leader step-up and is not a real stall.

Signed-off-by: Xuezhao Liu <xuezhao.liu@hpe.com>
@liuxuezhao
liuxuezhao requested review from a team as code owners September 4, 2026 08:59
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Ticket title is 'Aurora daos_user: engine printing continuously in log "cont_agg_eph_sync() 66e9ae82/2416cc43: Sluggish EC boundary report from rank 117, 165933 Seconds." '
Status is 'In Review'
Labels: 'ALCF'
https://daosio.atlassian.net/browse/DAOS-19591

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@liuxuezhao
liuxuezhao requested review from a team as code owners September 4, 2026 09:28
@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Test RPMs on EL 8.6 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-19014/2/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-19014/2/display/redirect

1 similar comment
@daosbuild3

Copy link
Copy Markdown
Collaborator

Test stage Functional on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-19014/2/display/redirect

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@daosbuild3

Copy link
Copy Markdown
Collaborator

@liuxuezhao
liuxuezhao force-pushed the lxz/cont_agg_eph_sync_26 branch from 1a6341f to 9ae4adc Compare September 7, 2026 08:12
@liuxuezhao
liuxuezhao removed request for a team September 7, 2026 08:15
@daosbuild3

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants