Skip to content

Conversation

tomuben
Copy link
Collaborator

@tomuben tomuben commented Jul 1, 2021

No description provided.

@tomuben tomuben requested a review from tkilias July 1, 2021 20:55
@tkilias
Copy link
Collaborator

tkilias commented Jul 2, 2021

It is clear why the test don't work at the moment. You mount the host directory to a different path in the CentOS container. You need to mount it to the same path as on the host.

@tkilias
Copy link
Collaborator

tkilias commented Jul 2, 2021

Please also check the readme. The prerequisite section got a bit messy in the last PR.

Copy link
Collaborator

@tkilias tkilias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe clean up after installing with apt and yum to keep the docker image smaller.and if not yet done, use --no-install-recommended

"deb [arch=amd64 signed-by=/usr/share/keyrings/docker-archive-keyring.gpg] https://download.docker.com/linux/ubuntu \
$(lsb_release -cs) stable" | tee /etc/apt/sources.list.d/docker.list > /dev/null

RUN apt-get -y update && apt-get -y install docker-ce docker-ce-cli containerd.io
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here is the clean missing

"deb [arch=amd64 signed-by=/usr/share/keyrings/docker-archive-keyring.gpg] https://download.docker.com/linux/ubuntu \
$(lsb_release -cs) stable" | tee /etc/apt/sources.list.d/docker.list > /dev/null

RUN apt-get -y update && apt-get -y install docker-ce docker-ce-cli containerd.io
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here is the clean missing

@tomuben tomuben merged commit 6c312bb into main Jul 6, 2021
@tomuben tomuben deleted the ci/#50_Add_CI_checks_for_other_operating_systems branch July 6, 2021 16:03
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