-
Notifications
You must be signed in to change notification settings - Fork 118
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
Natalie .yml test pull #926
Closed
nataliejschultz
wants to merge
22
commits into
e-mission:master
from
nataliejschultz:natalie_testbranch
Closed
Natalie .yml test pull #926
nataliejschultz
wants to merge
22
commits into
e-mission:master
from
nataliejschultz:natalie_testbranch
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
testing to see if image_build_push runs
Testing setup_config.sh echoes and attempting to produce error out of test-with-docker.yml
Commented out the docker -compose command because it was taking much longer than expected.
Adding a few more echoes to find exact location of error + commenting out unnecessary parts of docker build test
Re-doing echo command to find error.
another push to try and find error
Attempt at getting an error message and stopping build.
Trying to add verbose output to see logs
trying to edit the dockerfile to output verbose.
-x did not increase output info, so removed from dockerfile. Modifying setup_config to see if it will exit or echo. Also adding set -e at the top.
Remove exit 1 code and close if statement.
set -e seems to cause the scrip to skip over the part where the error is occurring. Running regularly to see if it errors out.
seeing if the run fails without installing conda at all.
Test to make sure that build fails due to inability to set conda environment variable.
Found out that last error (created in dockerfile) was erroring every time. Created a version in the setup_config.sh script and have verified that it will succeed if run with enough memory. Making sure that it fails in GH actions due to low memory.
testing set -e, yet again. Worked properly locally for some reason but historically hasn't worked in GH actions.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Trying to fix docker build error; initial .yml edits