Skip to content

ci: fix flaky Apptainer install by replacing PPA with direct GitHub release download#57

Merged
cokelaer merged 1 commit intomainfrom
copilot/fix-apptainer-workflow-failure
May 5, 2026
Merged

ci: fix flaky Apptainer install by replacing PPA with direct GitHub release download#57
cokelaer merged 1 commit intomainfrom
copilot/fix-apptainer-workflow-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

add-apt-repository ppa:apptainer/ppa intermittently times out on GitHub-hosted runners, causing CI failures unrelated to test code.

Changes

  • Install method: Replace PPA-based install with a direct .deb download from Apptainer's GitHub releases (auto-detects latest version), adding apptainer --version to confirm success
  • Cache cleanup: Drop brittle caching of /usr/bin/apptainer, /usr/lib/apptainer, /etc/apptainer, and the unused APT archives cache; keep only ~/.apptainer/cache
  • Action version: Bump actions/cache@v3actions/cache@v4
- name: Install Apptainer
  run: |
    APPTAINER_VERSION=$(curl -s https://api.github.com/repos/apptainer/apptainer/releases/latest \
      | grep '"tag_name"' | cut -d'"' -f4 | sed 's/v//')
    wget -q "https://github.com/apptainer/apptainer/releases/download/v${APPTAINER_VERSION}/apptainer_${APPTAINER_VERSION}_amd64.deb"
    sudo apt-get install -y "./apptainer_${APPTAINER_VERSION}_amd64.deb"
    apptainer --version

Mirrors the fix applied in sequana/fastqc#41.

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@coveralls
Copy link
Copy Markdown

coveralls commented May 5, 2026

Coverage Report for CI Build 25362873438

Coverage remained the same at 73.481%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 181
Covered Lines: 133
Line Coverage: 73.48%
Coverage Strength: 1.47 hits per line

💛 - Coveralls

@cokelaer cokelaer merged commit 9ca04f1 into main May 5, 2026
4 checks passed
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