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

Investigating mysterious Docker errors #107

Merged
merged 2 commits into from
Feb 3, 2022

Conversation

jhudsl-robot
Copy link
Collaborator

Purpose/implementation Section

What changes are being implemented in this Pull Request?

Investigating the Docker login works in Docker test github action yml: https://github.com/opencasestudies/OCS_Guide/actions/runs/1786353460

But isn't working in the render-bookdown.yml: https://github.com/opencasestudies/OCS_Guide/runs/5045929531?check_suite_focus=true

Only difference I can see so far is that the order of the steps are different (this shouldn't affect it but let's see)

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2022

No spelling errors! 🎉
Comment updated at 2022-02-03 with changes from 82d5622

@jhudsl-robot
Copy link
Collaborator Author

This fixes it! Basically I didn't realize that the docker github actions use the container specified as the global docker image for the action so it looked for docker to be installed on that image which it is not.

To get around this we just have to return to the original method of specifying docker run for each command we want run in the docker image of this repository.

@jhudsl-robot
Copy link
Collaborator Author

I will be making these changes upstream on the OTTR_Template repo as well.

@carriewright11
Copy link
Collaborator

still confused as to why the OCSdata package isn't getting recognized

@carriewright11 carriewright11 merged commit 5885be7 into main Feb 3, 2022
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.

2 participants