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

Fix COPY commands in Dockerfile #137

Closed
thekaveman opened this issue Aug 25, 2022 · 0 comments · Fixed by #138
Closed

Fix COPY commands in Dockerfile #137

thekaveman opened this issue Aug 25, 2022 · 0 comments · Fixed by #138
Assignees
Labels
bug Something isn't working

Comments

@thekaveman
Copy link
Member

Follow up to #123.

The COPY commands I added are adding all the files directly to the /home/calitp/app directory, rather than copying the subdirectories.

I discovered this while trying to test Benefits devcontainer against the published image, getting the following error:

/bin/bash: bin/start.sh: No such file or directory

This is because start.sh is sitting directly in /home/calitp/app, not /home/calitp/app/bin.

Found out by opening bash inside the (Benefits version) of the server:

docker compose run --entrypoint bash server

Then listing the directory:

calitp@46d38f74c1b2:~/app$ ls -al
total 52
drwxr-xr-x 1 calitp root   4096 Aug 25 15:50 .
drwxr-xr-x 1 calitp calitp 4096 Aug 23 19:33 ..
-rw-r--r-- 1 root   root      0 Aug 25 15:50 __init__.py
-rw-r--r-- 1 root   root   1155 Aug 25 15:50 app.py
-rw-r--r-- 1 root   root   1796 Aug 25 15:50 database.py
-rw-r--r-- 1 root   root    834 Aug 25 15:50 hash.py
-rwxr-xr-x 1 root   root     73 Aug 25 15:50 init.sh
-rw-r--r-- 1 root   root     95 Aug 25 15:50 requirements.txt
-rw-r--r-- 1 root   root    601 Aug 25 15:50 settings.py
-rw-r--r-- 1 root   root   2253 Aug 25 15:50 setup.py
-rwxr-xr-x 1 root   root    158 Aug 25 15:50 start.sh
-rw-r--r-- 1 root   root    607 Aug 25 15:50 teardown.py
-rw-r--r-- 1 root   root   5281 Aug 25 15:50 verify.py
@thekaveman thekaveman added the bug Something isn't working label Aug 25, 2022
@thekaveman thekaveman self-assigned this Aug 25, 2022
@thekaveman thekaveman moved this to In Progress in Digital Services Aug 25, 2022
Repository owner moved this from In Progress to Done in Digital Services Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant