Skip to content

Fix Apptainer CI: replace obsolete flag and modernize workflow - #25

Merged
cokelaer merged 2 commits into
mainfrom
copilot/check-apptainer-ci-status
Mar 20, 2026
Merged

Fix Apptainer CI: replace obsolete flag and modernize workflow#25
cokelaer merged 2 commits into
mainfrom
copilot/check-apptainer-ci-status

Conversation

Copilot AI commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

The Apptainer CI workflow was failing because --use-apptainer no longer exists in sequana_pipetools; the replacement is --apptainer-prefix <path>. The workflow also had stale action versions and no caching, making runs slow and unreliable.

Changes

  • Fix broken CLI flag: --use-apptainer--apptainer-prefix ~/.apptainer/cache
  • Update action versions: checkout@v3v4, setup-python@v3v5
  • Add workflow_dispatch: enables manual trigger from the GitHub UI
  • Add caching for APT packages, Apptainer binary, and Apptainer image cache (~/.apptainer/cache) to avoid redundant installs/downloads across runs
  • Simplify matrix: drop Python 3.9 (keep 3.10 only), consistent with other active sequana pipeline repos
  • Upgrade pip explicitly before installing dependencies

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

…refix, update action versions, add caching and workflow_dispatch

Co-authored-by: cokelaer <778821+cokelaer@users.noreply.github.com>
Copilot AI changed the title [WIP] Check CI status for Apptainer and assess obsolescence Fix Apptainer CI: replace obsolete flag and modernize workflow Mar 20, 2026
Copilot AI requested a review from cokelaer March 20, 2026 08:29
@cokelaer
cokelaer marked this pull request as ready for review March 20, 2026 09:27
@cokelaer
cokelaer merged commit a692f76 into main Mar 20, 2026
5 checks passed
@coveralls

coveralls commented Mar 20, 2026

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23334950165

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.0%

Totals Coverage Status
Change from base Build 23318316953: 0.0%
Covered Lines: 56
Relevant Lines: 70

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants