Skip to content

docs(audits): inventory interop known_failures and pick next-fix target#4026

Merged
oferchen merged 1 commit into
masterfrom
docs/interop-known-failures-inventory
May 14, 2026
Merged

docs(audits): inventory interop known_failures and pick next-fix target#4026
oferchen merged 1 commit into
masterfrom
docs/interop-known-failures-inventory

Conversation

@oferchen
Copy link
Copy Markdown
Owner

Summary

  • Inventory every entry in tools/ci/known_failures.conf (6 entries: 1 unconditional + 5 protocol-locked rules) and tools/ci/upstream_testsuite_known_failures.conf (17 testsuite entries), with file:line references and category breakdown.
  • Flag the iconv-upstream and delta-stats subsections in docs/audits/known-failures-eliminate.md as stale; both were retired post-audit (daf0849e7, 5713e74ea, 8619b63b0, 34dd330a4, 60e83fd96).
  • Recommend daemon / daemon-gzip-download / daemon-gzip-upload (tools/ci/upstream_testsuite_known_failures.conf:28-30) as the single next-fix target: clears 3 testsuite entries in one PR, scope confined to crates/daemon/src/daemon/sections/config_parsing/ plus module_definition/setters.rs, no protocol or platform work needed.

Test plan

  • Verify the new audit renders correctly on GitHub: docs/audits/interop-known-failures-inventory.md
  • Cross-check inventory counts against current tools/ci/known_failures.conf (6) and tools/ci/upstream_testsuite_known_failures.conf (17)
  • Spot-check file:line citations resolve (e.g. tools/ci/known_failures.conf:36, :65, :71, :82, :99; upstream_testsuite_known_failures.conf:28-30)
  • Confirm referenced source paths exist: crates/daemon/src/daemon/sections/module_access/client_args.rs:278-283, crates/transfer/src/setup/capability.rs::build_capability_string

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 14, 2026
Snapshot every entry across `tools/ci/known_failures.conf` and
`tools/ci/upstream_testsuite_known_failures.conf` (23 total: 6 + 17),
group by category, flag the two stale subsections in
`known-failures-eliminate.md` left over from the `iconv-upstream` and
`delta-stats` retirements, and recommend the `daemon` /
`daemon-gzip-download` / `daemon-gzip-upload` cluster as the next
remediation target (clears 3 tests in one PR, scope confined to the
daemon config parser).
@oferchen oferchen force-pushed the docs/interop-known-failures-inventory branch from 790f72c to 501e718 Compare May 14, 2026 04:32
@oferchen oferchen merged commit 3960bd4 into master May 14, 2026
7 checks passed
@oferchen oferchen deleted the docs/interop-known-failures-inventory branch May 14, 2026 04:32
oferchen added a commit that referenced this pull request May 14, 2026
…trix (#4039)

Both `standalone:iconv-upstream` and `standalone:delta-stats` were
removed from `tools/ci/known_failures.conf` but still appeared as open
items in `docs/audits/known-failures-eliminate.md` Section 1.

Move both entries to a new Section 7: retired log with the merge SHAs
that landed the underlying fixes and the matching conf deletions:

- `daf0849e7`, `8619b63b0`, `5713e74ea` retired `iconv-upstream`.
- `34dd330a4`, `60e83fd96` retired `delta-stats`.

Update Section 3's punch list and Section 4's summary to reflect the
new fixable/permanent tally (0 fixable, 6 permanent). Cross-link to
the canonical inventory at
`docs/audits/interop-known-failures-inventory.md` (PR #4026).
oferchen added a commit that referenced this pull request May 18, 2026
…et (#4026)

Snapshot every entry across `tools/ci/known_failures.conf` and
`tools/ci/upstream_testsuite_known_failures.conf` (23 total: 6 + 17),
group by category, flag the two stale subsections in
`known-failures-eliminate.md` left over from the `iconv-upstream` and
`delta-stats` retirements, and recommend the `daemon` /
`daemon-gzip-download` / `daemon-gzip-upload` cluster as the next
remediation target (clears 3 tests in one PR, scope confined to the
daemon config parser).
oferchen added a commit that referenced this pull request May 18, 2026
…trix (#4039)

Both `standalone:iconv-upstream` and `standalone:delta-stats` were
removed from `tools/ci/known_failures.conf` but still appeared as open
items in `docs/audits/known-failures-eliminate.md` Section 1.

Move both entries to a new Section 7: retired log with the merge SHAs
that landed the underlying fixes and the matching conf deletions:

- `daf0849e7`, `8619b63b0`, `5713e74ea` retired `iconv-upstream`.
- `34dd330a4`, `60e83fd96` retired `delta-stats`.

Update Section 3's punch list and Section 4's summary to reflect the
new fixable/permanent tally (0 fixable, 6 permanent). Cross-link to
the canonical inventory at
`docs/audits/interop-known-failures-inventory.md` (PR #4026).
oferchen added a commit that referenced this pull request May 18, 2026
…trix (#4039)

Both `standalone:iconv-upstream` and `standalone:delta-stats` were
removed from `tools/ci/known_failures.conf` but still appeared as open
items in `docs/audits/known-failures-eliminate.md` Section 1.

Move both entries to a new Section 7: retired log with the merge SHAs
that landed the underlying fixes and the matching conf deletions:

- `daf0849e7`, `8619b63b0`, `5713e74ea` retired `iconv-upstream`.
- `34dd330a4`, `60e83fd96` retired `delta-stats`.

Update Section 3's punch list and Section 4's summary to reflect the
new fixable/permanent tally (0 fixable, 6 permanent). Cross-link to
the canonical inventory at
`docs/audits/interop-known-failures-inventory.md` (PR #4026).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant