-
Notifications
You must be signed in to change notification settings - Fork 16
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
Archive home directory using multi-stage build #781
Merged
Merged
Changes from 22 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
0bc3277
Tar home build
danielhollas 9f30615
Add .dockerignore
danielhollas 99ee0ce
WIP: Smaller image
danielhollas 45c4596
Still doesn't work
danielhollas a685221
WIP: Multistage
danielhollas fd1ef03
Pre-download pseudos
danielhollas 0521d29
Fixes
danielhollas 68dc3c0
Build deps stage
danielhollas 94254ed
Use uv cache
danielhollas 7c30427
Indepedent build steps
danielhollas dba9fbb
Trace startup script
danielhollas 192e558
Smaller timeout, print logs when startup fails
danielhollas f735c2b
Use AIIDALAB_APPS
danielhollas 0bbe0b1
Fix permissions?
danielhollas 48b3990
Don't remove home dir
danielhollas af72528
Merge branch 'main' into smaller-image
danielhollas 475635e
Smaller diff
danielhollas 2804752
Fix
danielhollas e9cd720
Separate stage for build_deps
danielhollas 9a18ab7
Better caching behaviour
danielhollas 2fd78f8
Tweaks
danielhollas 456c4a2
Min cache
danielhollas 21a1917
Fix integration tests on PRs from forks
danielhollas bf57f63
Revert "Fix integration tests on PRs from forks"
danielhollas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
#!/bin/bash | ||
set -eux | ||
|
||
home="/home/${NB_USER}" | ||
|
||
# Untar home archive file to restore home directory if it is empty | ||
if [[ $(ls -A ${home} | wc -l) = "0" ]]; then | ||
if [[ ! -f $HOME_TAR ]]; then | ||
echo "File $HOME_TAR does not exist!" | ||
exit 1 | ||
fi | ||
if [[ ! -d ${QE_APP_FOLDER} ]]; then | ||
echo "Folder $QE_APP_FOLDER does not exist!" | ||
exit 1 | ||
fi | ||
|
||
echo "Extracting $HOME_TAR to $home" | ||
tar -xf $HOME_TAR -C "$home" | ||
|
||
echo "Copying directory '$QE_APP_FOLDER' to '$AIIDALAB_APPS'" | ||
cp -r "$QE_APP_FOLDER" "$AIIDALAB_APPS" | ||
else | ||
echo "$home folder is not empty!" | ||
ls -lrta "$home" | ||
fi | ||
set +eux |
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also happened in stage 4 but I understand it is not avoidable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. It is unfortunate that we need to install all the dependencies to install qe codes and pseudos. But
uv
is so fast and I am re-using its cache that in terms of speed it doesn't matter much.