Skip to content

Release CI follow-ups: tighter coverage #747

Closed

Description

@elijahbenizzy

Follow-ups identified during code review of #745. Not blocking, but worth doing.

Files / scripts

  • Rename examples/deep-researcher/utils.py — currently shares its basename with 4 other ASF-owned utils.py files. The new .rat-excludes uses basename matching, so all 5 are skipped. A future regression in any of the 4 ASF files would silently pass RAT.
  • Thread --skip-signing through cmd_verify in scripts/apache_release.py. Today running the verify subcommand against a --skip-signing build crashes on missing .asc files.
  • Run RAT on the wheel too, not just the tarballs. Wheel is also redistributed.
  • Pin the RAT JAR by SHA256 in .github/workflows/release-validation.yml. Currently the workflow downloads from Maven Central without integrity verification.

Smoke test (scripts/ci_smoke_server.py)

  • Replace time.sleep(2) after running the tracked app with a poll-then-assert loop on /api/v0/projects (currently flake-prone).
  • Forward SIGTERM to the uvicorn child via process group (currently only kills the burr CLI wrapper; child can be orphaned on dev boxes).
  • Add --cleanup flag and default to cleaning the workspace locally (CI keeps it via GITHUB_ACTIONS env, currently always retained).

CI coverage gaps

  • Add a matrix cell that does bare pip install <wheel> (no [learn] extras) and at least imports burr and burr.tracking.server.run. Catches optional-extra dep leakage into base.
  • Add a step that builds the wheel from the sdist tarball (what voters do) and asserts the result matches the CI-built wheel. Catches sdist-vs-wheel drift.
  • Hit GET / in the smoke test to confirm the UI build is actually served (not just present in the wheel).

Hygiene

  • Add a pre-commit hook that grep-checks new files for ASF headers (RAT in CI is the safety net but slow).
  • Add a weekly cron job running this workflow against main to catch dep-drift between RCs.

Activity

  1. added
    kind/improvementImproving something that already exists
    status/acceptedTriaged, validated, ready to work on
    and removed on Apr 23, 2026
  2. andreahlert commented on Apr 23, 2026

    @andreahlert
    Collaborator

    This issue is a follow-up of #745 and part of the release process automation epic #715.

  3. github-actions commented on May 13, 2026

    @github-actions

    @elijahbenizzy, this issue has been inactive for 19 days.
    Are you still working on it? Drop a comment to let us know.

    If there is no update within 7 days, the assignment will be removed
    so someone else can pick it up
    (per the assignment policy).

  4. github-actions commented on May 27, 2026

    @github-actions

    @elijahbenizzy, this issue has been inactive for 14 days.
    Are you still working on it? Drop a comment to let us know.

    If there is no update within 7 days, the assignment will be removed
    so someone else can pick it up
    (per the assignment policy).

  5. github-actions commented on Jun 10, 2026

    @github-actions

    @elijahbenizzy, this issue has been inactive for 14 days.
    Are you still working on it? Drop a comment to let us know.

    If there is no update within 7 days, the assignment will be removed
    so someone else can pick it up
    (per the assignment policy).

  6. t1mato commented on Jun 23, 2026

    @t1mato
    Contributor

    Hello, would love to work on this issue if it hasn't been finished already.

    @elijahbenizzy @andreahlert

  7. github-actions commented on Jul 8, 2026

    @github-actions

    @elijahbenizzy, @t1mato, this issue has been inactive for 14 days.
    Are you still working on it? Drop a comment to let us know.

    If there is no update within 7 days, the assignment will be removed
    so someone else can pick it up
    (per the assignment policy).

  8. added 4 commits that reference this issue on Jul 12, 2026
    48d6e40
    2f4abb2
    ec376de
    ca76895
  9. added a commit that references this issue on Jul 12, 2026
    19d3fcf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

kind/improvementImproving something that already existspriority/mediumImportant but not urgentstatus/acceptedTriaged, validated, ready to work on

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions