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

Add optional singularity filesystem encryption #63

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

iqmtestd
Copy link
Contributor

and fix various problems with singularity image generation:

  • klayout binary is included
  • simplify install_deb_packages()
  • no more shellcheck errors
  • use docker in place of debootstrap

Issue #1306

@cla-bot cla-bot bot added the cla-signed label Sep 27, 2023
@iqmtestd iqmtestd self-assigned this Sep 27, 2023
@github-actions
Copy link

github-actions bot commented Sep 27, 2023

Test results

       5 files  ±0         5 suites  ±0   4m 3s ⏱️ +14s
   333 tests ±0     332 ✔️ ±0  1 💤 ±0  0 ±0 
1 665 runs  ±0  1 660 ✔️ ±0  5 💤 ±0  0 ±0 

Results for commit ec09491. ± Comparison against base commit 51817e1.

♻️ This comment has been updated with latest results.

and fix various problems with singularity image generation:
 * klayout binary is included
 * simplify install_deb_packages()
 * no more shellcheck errors
 * use docker in place of debootstrap

Issue #1306
@iqmtestd iqmtestd force-pushed the fix_ci_and_add_singularity_encryption branch from 8e1c099 to ec09491 Compare September 27, 2023 12:21
Copy link
Contributor

@qpavsmi qpavsmi left a comment

Choose a reason for hiding this comment

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

Approved, thanks for fixing shellcheck errors

@iqmtestd iqmtestd merged commit 89f1d37 into main Sep 27, 2023
18 checks passed
@iqmtestd iqmtestd deleted the fix_ci_and_add_singularity_encryption branch September 27, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants