← Back to issue list

test(spread): mark doc tests as manual

View original Github issue

Metadata

Project
rockcraft
Number
#801
Type
pull request
State
open
Author
tigarmo
Labels
—
Created
Updated
Closed
—

Current evaluation

Marks 24 docs spread test task files as manual so they stop running on every code change. Maintainer-authored, never reviewed, no activity for 413 days; branch predates the google→openstack spread migration.

Suggested action: close stale

Reason: The PR has 0 reviewers, 0 comments and 0 CI checks, and no activity for 413 days. Its branch is based on pre-migration spread config: the repo since merged #1279 ('test(spread): migrate from google to openstack') and #1206, which added a dedicated path-conditional `.github/workflows/spread-docs.yaml` running `spread openstack:...docs...` on `docs/**/code/**` plus a weekly cron — addressing this PR's 'don't run on every code change' motivation differently. Docs test files were also heavily rewritten since (e.g. #1222 removed a tutorial, #1213 moved to ubuntu 26 base, #1330 FastAPI updates), so the 24-file diff would likely not apply cleanly. Inactivity alone isn't the reason; the demonstrable architectural divergence is.

Evaluated by: mr-cal-bot

Impact: 28 Quick Win: 3 Complexity: 32 Confidence: 72 Actionability: 14

Issue body

These tests take a disproportional amount of time to run, and don't need to run on every code change. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 28
Quick Win: 3
Complexity: 32
Confidence: 72
Actionability: 14
close stale Marks 24 docs spread test task files as manual so they stop running on every code change. Maintainer-authored, never reviewed, no activity for 413 days; branch predates the google→openstack spread migration.
qwen/qwen3.6-35b-a3b
Impact: 20
Quick Win: 2
Complexity: 10
Confidence: 85
Actionability: 10
close stale Marks doc tests in spread as manual to reduce CI runtime. PR is 596 days old with no activity for 400 days, no reviewers, no comments, and no CI checks.
qwen/qwen3.8-27b
Impact: 30
Quick Win: 22
Staleness: 72
Complexity: 25
Confidence: 55
needs review Marks doc spread tests as manual so they don't run on every code change. Pending review, 0 comments, no CI. Docs tests now run via a separate conditional workflow (spread-docs.yaml) on a weekly schedule and only when docs code changes.
qwen/qwen3.6-35b-a3b
Staleness: 90
Complexity: 15
Confidence: 85
close stale Marks documentation spread tests as manual to reduce CI execution time. Inactive for over a year, lacks CLA and CI runs, and test configuration has likely evolved since submission.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 5
Confidence: 85
close stale Marks documentation tests as manual to reduce CI execution time. PR has been inactive for over 326 days with no review or CI feedback.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 5
Confidence: 85
close stale Marks spread documentation tests as manual to reduce CI execution time. The PR has been inactive for over 323 days with no reviews or CI status.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/rockcraft#1279 (confidence 70%)

    Merged (132b18c4) 'test(spread): migrate from google to openstack' — rewrote spread.yaml and the CI invocations this PR's branch predates

  • Superseded By: canonical/rockcraft#1206 (confidence 55%)

    Merged (3624f64c) 'ci: make workflows conditional, add docs-only spread tests' — created a separate docs spread workflow triggered on docs code paths and a weekly schedule instead of marking doc suites manual

Related issues

Issue Project State Summary Similarity
#1279 test(spread): migrate from google to openstack rockcraft merged PR #1279 by tigarmo migrated rockcraft's spread test setup from Google to OpenStack (spread.yaml, spread workflows, 14 files, +192/-17); merged as commit 132b18c4. Proxy-related spread failures were knowingly deferred.
73%
#637 Work/spread openstack rockcraft closed Draft PR #637 to run spread tests on OpenStack was closed unmerged as superseded by merged PR #1279 (commit 132b18c4), which migrated spread tests from Google to OpenStack.
72%
#2126 ci(docs): spread test documentation charmcraft merged Merged PR #2126 in commit f1b73a6e, adding CI for documentation spread tests and adapting spread tasks for Google-hosted runners.
71%
#3530 tests: manual spread task for MaaS (CRAFT-225) snapcraft merged Merged as a03923d6f: added a manual Spread task for MaaS, with the reported run completing both tasks successfully.
70%