Skip to content

ci: fix Apptainer install by dropping Launchpad PPA#31

Merged
cokelaer merged 2 commits into
mainfrom
copilot/fix-apptainer-workflow-error
May 1, 2026
Merged

ci: fix Apptainer install by dropping Launchpad PPA#31
cokelaer merged 2 commits into
mainfrom
copilot/fix-apptainer-workflow-error

Conversation

Copilot AI commented May 1, 2026

Copy link
Copy Markdown
Contributor

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

Changes

  • Install: Replace PPA-based install (software-properties-commonadd-apt-repositoryapt install) with a direct download of the latest .deb from the official Apptainer GitHub releases; add apptainer --version to confirm.
  • Cache: Remove brittle caching of system paths (/usr/bin/apptainer, /usr/lib/apptainer, /etc/apptainer) and the unused APT archives cache. Retain only ~/.apptainer/cache, moved before the install step.
  • Action version: Bump actions/cache@v3actions/cache@v4.
# Before
- name: Install Apptainer
  if: steps.cache-apptainer.outputs.cache-hit != 'true'
  run: |
    sudo apt-get install -y software-properties-common
    sudo add-apt-repository -y ppa:apptainer/ppa  # ← flaky
    sudo apt-get update && sudo apt-get install -y apptainer

# After
- 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

@codacy-production

codacy-production Bot commented May 1, 2026

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

coveralls commented May 1, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 25227416281

Coverage remained the same at 80.0%

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: 70
Covered Lines: 56
Line Coverage: 80.0%
Coverage Strength: 1.6 hits per line

💛 - Coveralls

@cokelaer

cokelaer commented May 1, 2026

Copy link
Copy Markdown
Contributor

@copilot check the original PR from sequana/fastqc. there was an update wrt apptainer installation

Copilot AI commented May 1, 2026

Copy link
Copy Markdown
Contributor Author

@copilot check the original PR from sequana/fastqc. there was an update wrt apptainer installation

Updated in 2806c9c. The install now matches sequana/fastqc#41 exactly — downloading the latest .deb directly from GitHub releases rather than using Ubuntu's default apt repos:

- 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

Copilot AI requested a review from cokelaer May 1, 2026 18:34
@cokelaer cokelaer merged commit b20eb54 into main May 1, 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