Skip to content

ci(ansible): split artifacts health check into its own workflow#7054

Merged
xmfcx merged 1 commit intomainfrom
ci/ansible-split-artifacts-workflow
Apr 24, 2026
Merged

ci(ansible): split artifacts health check into its own workflow#7054
xmfcx merged 1 commit intomainfrom
ci/ansible-split-artifacts-workflow

Conversation

@xmfcx
Copy link
Copy Markdown
Contributor

@xmfcx xmfcx commented Apr 23, 2026

  • Split the artifacts role out of health-check-ansible into a new dedicated workflow health-check-ansible-artifacts so the large $HOME/autoware_data download runs on its own schedule (only when artifact-related paths change).
  • Move health-check-ansible from self-hosted runners back to GitHub-hosted ubuntu-latest and drop the now-redundant apt/tzdata/git bootstrap steps (handled by ansible/scripts/install-ansible.sh), skipping the artifacts tag.
  • Report installation disk footprint in both workflows via $GITHUB_STEP_SUMMARY (delta for the install, total size for the artifacts download).

Why

The artifacts download dominates disk and network in the health check, yet most ansible edits don't touch it. Isolating it gives us targeted triggers (via files_ignore: ansible/roles/artifacts/**), frees the main check to run on GitHub-hosted runners again, and makes disk usage visible per job.


Test plan

@xmfcx xmfcx added the type:ci Continuous Integration (CI) processes and testing. label Apr 23, 2026
@xmfcx xmfcx requested a review from isamu-takagi as a code owner April 23, 2026 14:39
@xmfcx xmfcx added the run:health-check Run health-check label Apr 23, 2026
@xmfcx xmfcx requested a review from mitsudome-r as a code owner April 23, 2026 14:39
@xmfcx xmfcx self-assigned this Apr 23, 2026
@xmfcx xmfcx added run:health-check Run health-check and removed run:health-check Run health-check labels Apr 23, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@xmfcx xmfcx force-pushed the ci/ansible-split-artifacts-workflow branch from 96d70c8 to 2f9ff5b Compare April 23, 2026 15:42
Separate the heavy artifacts download from the ansible install health-check.
The main workflow now skips the artifacts tag and runs on ubuntu-latest,
while a new workflow runs only the artifacts role and reports its size.

Signed-off-by: Mete Fatih Cırıt <mfc@autoware.org>
@xmfcx xmfcx force-pushed the ci/ansible-split-artifacts-workflow branch from 2f9ff5b to 18183c9 Compare April 24, 2026 07:04
@xmfcx xmfcx merged commit 2440d27 into main Apr 24, 2026
25 checks passed
@xmfcx xmfcx deleted the ci/ansible-split-artifacts-workflow branch April 24, 2026 07:26
@xmfcx
Copy link
Copy Markdown
Contributor Author

xmfcx commented Apr 24, 2026

image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:health-check Run health-check type:ci Continuous Integration (CI) processes and testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants