Skip to content

fix(ci): scope HF dataset timeouts and repair fixture cleanup - #8446

Open
Xuanwo wants to merge 2 commits into
mainfrom
xuanwo/ci-hf-fixtures
Open

Xuanwo wants to merge 2 commits into
mainfrom
xuanwo/ci-hf-fixtures

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Oct 31, 2026

Copy link
Copy Markdown
Member

Which issue does this PR close?

Related to #7577.

Rationale for this change

Two HF dataset behavior-test attempts on 26f2b5cf exhausted the 10-second I/O deadline while writing test fixtures. Temporary dataset cleanup also returned 404 because the deletion request passed the full repository ID as its bare name.

What changes are included in this PR?

  • Allow a positive whole-second OPENDAL_TEST_IO_TIMEOUT in the behavior testkit and use 30 seconds for the HF dataset fixture. Reset the override between fixtures using the existing environment sentinel.
  • Send the namespace and bare repository name separately when deleting temporary datasets. Keep bucket deletion on its existing endpoint.
  • Run regression tests through the actual cleanup entry point in Check CI.

Validation on nier covered the cleanup entry point, testkit lint/format checks, and real Memory behavior with the override, reset sentinel, invalid value, and zero value. The HF diagnostic run passed 109 dataset, 109 bucket, and 109 bucket-HTTP tests, followed by three successful cleanup DELETE requests. The runtime files match this PR's 4c5d89b7; the diagnostic workflow selector and runner are excluded from the PR. Full PR CI will run on publication.

Are there any user-facing changes?

Behavior-test configuration gains an optional I/O timeout. Production timeout defaults and retry counts stay unchanged.

Breaking changes

AI Usage Statement

  • Harness: Codex. Model: GPT-6 family; exact runtime identifier unavailable to this task. Effort: unavailable to this task. Role: implementation, coordination, and validation.
  • Harness: Codex subagent. Model: group/deepseek. Effort: max. Role: independent review and validation on nier.

@Xuanwo
Xuanwo requested a review from tisonkun as a code owner October 31, 2026 02:54
@github-actions github-actions Bot added core integrations/object_store releases-note/fix The PR fixes a bug or has a title that begins with "fix" services/huggingface size:L This PR changes 100-499 lines, ignoring generated files. labels Oct 31, 2026

This branch has not been deployed

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

Labels

core integrations/object_store releases-note/fix The PR fixes a bug or has a title that begins with "fix" services/huggingface size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants