← Back to issue list

Change ROS spread testing strategy

View original Github issue

Metadata

Project
snapcraft
Number
#5296
Type
issue
State
closed
Author
mr-cal
Labels
Status: Triaged Type: Enhancement
Created
Updated
Closed

Current evaluation

ROS spread-test CI strategy was updated in merged PR #5418 (commit a5682c276) to reduce test-suite duration.

Resolution: closed resolved

Reason: Merged PR #5418, commit a5682c276 (“ci: speed up spread test suite”), directly references and addresses #5296.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done The ROS spread tests are slow and don't need to be run with every PR. They should only run when: * ROS-related code changes * In the weekly spread tests ### Why it needs to get done To speed up CI runs

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved ROS spread-test CI strategy was updated in merged PR #5418 (commit a5682c276) to reduce test-suite duration.
stealth/space-bunny-alpha — closed resolved Addressed by merged PR #5418 (commit a5682c276), “ci: speed up spread test suite”; the commit history references this issue.
qwen/qwen3.6-35b-a3b — — Abandoned without implementation. The proposal to restrict ROS spread tests to ROS-specific changes or weekly runs was closed, leaving CI optimization unaddressed. No resolution or merge details were provided.
qwen/qwen3.6-35b-a3b — — Closed without implementation. A Jira ticket was created to track the proposed CI optimization for ROS spread tests, but no code changes or merge details were provided.
qwen3.6-35b-a3b-mtp-q6 — — Closed without implementation. The enhancement to restrict ROS spread tests to ROS changes or weekly runs was abandoned.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: canonical/snapcraft#5418 (confidence 99%)

    Merged commit a5682c276 implements the spread-test CI speedup requested by this issue.

Related issues

Issue Project State Summary Similarity
#4113 tests: update linter-ros2-humble-mixed spread test snapcraft merged Merged #4113 as commit 2735e2c3c, removing seven outdated ROS 2 Humble mixed spread-test expectations after extension libraries were removed.
73%
#4369 test(spread): make ros linter test less strict snapcraft merged Merged in #4369 as commit 8d0c451a4, relaxing the ROS spread linter test while retaining libpng and ros2-humble warning checks; fixes #4256.
73%