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

Natalie .yml test pull #926

Closed

Conversation

nataliejschultz
Copy link
Contributor

Trying to fix docker build error; initial .yml edits

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
Stopping image build test from running with every push. Trying set -e command to see if it makes a difference.
trying again
Attempt at getting an error message and stopping build.
Trying to add verbose output to see logs
-vv added potentially in wrong place.
trying one more location for the -vv command, to see if I can catch the error.
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.
@shankari shankari closed this in a79381e Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant