-
Notifications
You must be signed in to change notification settings - Fork 1
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
Consolidation + removing dev GenPipes as GenPipes 5 is a package #17
Conversation
apptainer-version: 1.3.4 | ||
- name: Create sif image | ||
run: apptainer build -F container/wrapper_genpipes/images/genpipes-${{ env.TAG_NAME }}.sif docker://${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ env.TAG_NAME }} | ||
- name: Comit and Push sif image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sif should be publish in a release, not pushed to the repo. That would make it way too big and also make it reach its size limit.
Dockerfile
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should try to run the apt install and updatd in as few RUN call a possible and also make sur to purge the cache and deleted tmp files on the same RUN line. This will make the container image thiner.
You can probably run all the apt and wget on the same line acctually
README.md
Outdated
|
||
Follow installation procedure from the [Singularity install page](https://github.com/hpcng/singularity/blob/master/INSTALL.md) or the [Docker install page](https://docs.docker.com/get-docker/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should point to apptainer directly and let singularity stay there for compatibility but talk about apptainer in the doc.
https://apptainer.org/docs/user/main/quick_start.html
or https://github.com/apptainer/apptainer/blob/main/INSTALL.md
for install link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sif file should be uploaded as a release tgz file
https://github.com/marketplace/actions/upload-files-to-a-github-release
The action crated the sif (will done here), then puts it in the place that looks like this:
./wrapper_genpipes
./wrapper_genpipes/bin
./wrapper_genpipes/bin/container_wrapper.sh
./wrapper_genpipes/images
./wrapper_genpipes/images/genpipes_v<release_number>.sif
./wrapper_genpipes/images/genpipes.sif <symlink to genpipes_v<release_number>.sif >
./wrapper_genpipes/etc
./wrapper_genpipes/etc/wrapper.conf.tpl
run tar -zcvf wrapper_genpipes.tgz wrapper_genpipes
upload wrapper_genpipes.tgz to release.
No description provided.