scimma / blast

Django web app for the automatic characterization of supernova hosts
MIT License
1 stars 2 forks source link

Revise GitHub Action workflows #199

Closed manning-ncsa closed 6 months ago

manning-ncsa commented 6 months ago

Fixes # .

Description of the Change

Revise GitHub Action that automates Docker image build to push to NCSA Harbor instance. Triggers on any changes in /app instead of only Dockerfile and requirements.txt. Disables pull request trigger. Use build cache when running Continuous Integration workflow to improve speed of testing.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.56%. Comparing base (3dd4bfc) to head (66bd995). Report is 37 commits behind head on main.

:exclamation: Current head 66bd995 differs from pull request most recent head e97d6aa. Consider uploading reports for the commit e97d6aa to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #199 +/- ## ========================================== - Coverage 53.62% 53.56% -0.07% ========================================== Files 28 28 Lines 3355 3426 +71 ========================================== + Hits 1799 1835 +36 - Misses 1556 1591 +35 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

manning-ncsa commented 6 months ago

@djones1040 we should merge this when you are ready prior to merging #197 so that we can test the updated Actions.

manning-ncsa commented 6 months ago

Looks like this is failling with a "no space left on device" error. Somehow we need to tell the step running Docker Compose not to build the image from scratch but to use the image built in the previous step (the build that should be faster due to the cache).