Skip to content

Run the entire installation of the compat layer in a container, without requiring root privileges#167

Merged
trz42 merged 20 commits intoEESSI:mainfrom
bedroge:full_installation_in_container
Mar 30, 2023
Merged

Run the entire installation of the compat layer in a container, without requiring root privileges#167
trz42 merged 20 commits intoEESSI:mainfrom
bedroge:full_installation_in_container

Conversation

@bedroge
Copy link
Copy Markdown
Collaborator

@bedroge bedroge commented Mar 7, 2023

No description provided.

@@ -0,0 +1,17 @@
FROM debian:11-slim

Check warning

Code scanning / Scorecard

Pinned-Dependencies

score is 7: containerImage not pinned by hash Remediation tip: pin your Docker image by updating debian to debian@sha256:f81bf5a8b57d6aa1824e4edb9aea6bd5ef6240bcc7d86f303f197a2eb77c430f Click Remediation section below for further remediation help
RUN apt-get update
RUN apt-get install -y gcc g++ make diffutils libgmp-dev perl wget
RUN apt-get install -y git python3-pip python3-cryptography
RUN pip3 install --upgrade pip

Check warning

Code scanning / Scorecard

Pinned-Dependencies

score is 7: pipCommand not pinned by hash Click Remediation section below to solve this issue
RUN apt-get install -y gcc g++ make diffutils libgmp-dev perl wget
RUN apt-get install -y git python3-pip python3-cryptography
RUN pip3 install --upgrade pip
RUN pip3 install ansible

Check warning

Code scanning / Scorecard

Pinned-Dependencies

score is 7: pipCommand not pinned by hash Click Remediation section below to solve this issue
@trz42
Copy link
Copy Markdown
Contributor

trz42 commented Mar 30, 2023

Noted that the bootstrap-prefix.sh script seems to be still the old one from ~ two years ago? In #160 it has been updated to what was available from Gentoo Prefix then.

Should we update this too? Maybe not in this PR?

Copy link
Copy Markdown
Contributor

@trz42 trz42 left a comment

Choose a reason for hiding this comment

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

Looks like solid work to me.

echo " -a | --arch ARCH - architecture to build a compatibility layer for"
echo " [default/required: current host's architecture]"
echo " -c | --container IMG - image file or URL defining the container to use"
echo " [default: ${CONTAINER}"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Default lacks closing ] bracket.

Copy link
Copy Markdown
Contributor

@trz42 trz42 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@trz42 trz42 merged commit bdf67d2 into EESSI:main Mar 30, 2023
@bedroge bedroge deleted the full_installation_in_container branch April 21, 2023 13:38
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.

3 participants