Bug in docker file - Githubissues

quic / gunyah-support-scripts

Other

10 stars 6 forks source link

Bug in docker file #6

Closed vsrivatsa1 closed 10 months ago

vsrivatsa1 commented 1 year ago

I think dockerfile-hyp needs to have this fix:

COPY ./utils/* "${BASE_DIR}/utils/"
COPY --chown=$USER:$USER ./utils/* "${BASE_DIR}/utils/"

Otherwise some of the cp commands fail with perm denied:

cp ${BASE_DIR}/utils/kern-test.sh ${TOOLS_DIR}/qemu/  && \

quic-yvasi commented 1 year ago

Thanks Srivatsa, I didn't see issue on internal Linux machines, I will check it out on external machine and will incorporate the fixes in next release.

Thanks Yug

quic-yvasi commented 1 year ago

Hi Srivatsa, thanks for raising this, I found that I found similar issue on one of my home machines.

I was seeing issue of the ownership and mode not changing despite an explicit command to do so. Seems like this is issue with Ubuntu provided docker.io installation (https://github.com/moby/moby/issues/46161).

So, your change should fix the problem you are facing and any further issues of permission denials are causing by wrong variant of docker installation.

Thanks Yug

quic-yvasi commented 10 months ago

Fixes released in commit 6a959c887f3ccf0396a71f60c25497d99b3efa35