Skip to content

CI: keep day-in-the-life test on the radio path only#411

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/day-in-the-life-test-ci-target
Draft

CI: keep day-in-the-life test on the radio path only#411
Copilot wants to merge 4 commits into
mainfrom
copilot/day-in-the-life-test-ci-target

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 30, 2026

Pull Request Title (e.g., Feature: Add user authentication)

Description

The new day_in_the_life integration test was being picked up by the UART job, which caused the wrong CI lane to execute a radio-only workload. This update narrows the UART job filter so the test runs only in the dedicated radio path.

  • CI routing

    • Exclude day_in_the_life from integration-uart
    • Leave the dedicated integration-radio day-in-the-life step unchanged
  • Effect

    • Prevents UART CI from failing on the new long-duration radio cadence test
    • Keeps coverage ownership aligned with the job that provisions the radio harness
- name: Run UART Integration Tests
  run: |
    make test-integration FILTER="not day_in_the_life"

Related Issues/Tickets

N/A

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Z Tests
  • Manual testing (describe steps)

Screenshots / Recordings (if applicable)

N/A

Checklist

  • Written detailed sdd with requirements, channels, ports, commands, telemetry defined and correctly formatted and spelled
  • Have written relevant integration tests and have documented them in the sdd
  • Have done a code review with
  • Have tested this PR on every supported board with correct board definitions

Further Notes / Considerations

The follow-up CI run was triggered after the workflow edit. GitHub reported the run as action_required, with no jobs started, which appears to be external to this workflow change.

Copilot AI changed the title Developing Day in the Life test CI target using over the radio test harness CI: keep day-in-the-life test on the radio path only May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants