Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apptainer CI #53

Merged
merged 25 commits into from
Apr 14, 2023
Merged

Apptainer CI #53

merged 25 commits into from
Apr 14, 2023

Conversation

jlenain
Copy link
Collaborator

@jlenain jlenain commented Apr 7, 2023

This PR introduces a new CI workflow where:

  • nectarchain is shipped as a conda environment, and buit as an Apptainer/Singularity container on selected events (releases, pull requests, merges into master),
  • the container is deployed on the GitHub Container Registry, i.e. visible as a package of the cta-observatory organization, on non-pull-request events, such as releases.

Users can then use such containers with:

apptainer shell oras://ghcr.io/cta-observatory/nectarchain:latest

TL;DR

We will now have fully automatic Apptainer containers built and deployed on releases!

@kosack , @maxnoe , would that be deemed acceptable ?

jlenain and others added 21 commits April 7, 2023 23:49
Fix workflow permissions.
Test changing workflow permissions
Try fixing workflow permissions
Try fixing workflow permissions
Try debugging push to ORAS
…l and avoid git cloning the whole repository.
@jlenain jlenain self-assigned this Apr 7, 2023
@jlenain jlenain added the enhancement New feature or request label Apr 7, 2023
@codecov
Copy link

codecov bot commented Apr 7, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (16418f7) 0.53% compared to head (9a2af1f) 0.53%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master     #53   +/-   ##
======================================
  Coverage    0.53%   0.53%           
======================================
  Files          23      23           
  Lines        2434    2434           
======================================
  Hits           13      13           
  Misses       2421    2421           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

singularity/Singularity Outdated Show resolved Hide resolved
jlenain added 3 commits April 8, 2023 18:50
…oning from master. This will also enable the CI to test build failure from a PR triggering the container CI.
@jlenain jlenain requested a review from maxnoe April 13, 2023 08:54
@jlenain jlenain removed the request for review from maxnoe April 14, 2023 21:31
@jlenain jlenain merged commit 76e8131 into cta-observatory:master Apr 14, 2023
@jlenain jlenain deleted the apptainer-ci branch April 14, 2023 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants