Updating containers to Galaxy v20.05 +CI for pushing them#2025
Updating containers to Galaxy v20.05 +CI for pushing them#2025hexylena merged 23 commits intogalaxyproject:masterfrom
Conversation
|
I am already encountering the limitations of github actions (14gb is not enough to build and push some images) with big topics like, proteomics, transcriptomics etc ... these will still need to be pushed manually or to be moved to a Jenkins server. |
|
can we put the github action yaml onto a whitelist of travis yaml linting? |
|
@bedroesb sure. Just edit the makefile, maybe - find . -name "*.yaml" | xargs -L 1 -I '{}' sh -c "yamllint {}" \
+ find . -name "*.yaml" | grep -v .github | xargs -L 1 -I '{}' sh -c "yamllint {}" \ |
|
@hexylena Thank you! easy fix :) |
|
@hexylena If there are no comments I think this is ready to be merged :) |
| # run tutorial install as user galaxy | ||
| su - $GALAXY_USER | ||
| # wait for database to finish starting up | ||
| STATUS=$(psql 2>&1) |
There was a problem hiding this comment.
shouldn't psql hang if it's alive/working? Maybe psql -l would be better since you know that command terminates?
There was a problem hiding this comment.
I took most if this part over from the galaxy-stable tools-wrapper (I am not super familiar with psql), because that was the only way I could successfully install the tools on the new versions of Galaxy. should we make edits there too?
There was a problem hiding this comment.
Huh! That's unexpected. psql just opens an interactive postgres session, so it intentionally hangs forever waiting for user input. If it works there (and here) then no reason to change it. Just was surprising.
|
@hexylena Deleting that image was very sloppy of me, luckily you caught it! |
|
@bedroesb honestly it shouldn't've passed travis, that image I think is used in the metagenomics intro slides. Weird! Oh well, looks good otherwise. |
This PR will contain:
-> a job for every topic