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

podman pull, inside qm, fails on OStree image: no space left on device #584

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

nsednev
Copy link
Collaborator

@nsednev nsednev commented Sep 25, 2024

Fix: ostree images, pulling container larger than 0.9M

Container with size > 1G is failing and resulting no disk space on podman pull
resolve #574

The fix creates a workaround for OStree images to properly use prepare.sh.
Rebased from #575
Adds OStree image specific checkup to the prepare.sh and sets image_copy_tmp_dir="/var/tmp.dir", so the prepare_images section could be used also for OStree images.

@nsednev
Copy link
Collaborator Author

nsednev commented Sep 25, 2024

@Yarboa @dougsland Please review this fix.

@aesteve-rh
Copy link
Collaborator

The commit itself could get a better title/body description.

@nsednev nsednev changed the title Rebase from 575 Can't create ffi-qm container inside the QM partition on OStree image - no space left on device. Sep 25, 2024
@nsednev
Copy link
Collaborator Author

nsednev commented Sep 25, 2024

The commit itself could get a better title/body description.

Fixed.

@aesteve-rh
Copy link
Collaborator

Fixed.

I meant the commit message, not the PR. You could use a similar text though.

Also, please try to avoid creating a new commit when you rebase/merge from main, so that we can keep a clean git history.

@Yarboa Yarboa changed the title Can't create ffi-qm container inside the QM partition on OStree image - no space left on device. podman pull fails on OStree image: no space left on device Sep 25, 2024
@Yarboa Yarboa changed the title podman pull fails on OStree image: no space left on device podman pull, inside qm, fails on OStree image: no space left on device Sep 25, 2024
tests/ffi/common/prepare.sh Outdated Show resolved Hide resolved
tests/ffi/common/prepare.sh Show resolved Hide resolved
tests/ffi/common/prepare.sh Outdated Show resolved Hide resolved
@aesteve-rh
Copy link
Collaborator

aesteve-rh commented Sep 26, 2024

@nsednev when you receive requests from reviewers, you should apply them over the commit that caused the issue.

Try following these steps:

  1. Rebase your branch with an up-to-date local main. This will remove the merge commit that you've addded:
$ git rebase main
warning: skipped previously applied commit ab9b8c9
hint: use --reapply-cherry-picks to include skipped commits
hint: Disable this message with "git config advice.skippedCherryPicks false"
Successfully rebased and updated refs/heads/VROOM-22955.
  1. Then you can do:
git rebase -i HEAD~5

You should see only your commits, keep the first pick, the other commits switch to squash:
image

Save and exit editing the file. Next you will be prompted with the commit message and body selection. Remove the additional text from the commits, and leave a meaningful header and body, and the SoB line.

  1. Now you should be only one commit ahead of main. So you can just force push:
git push -f

Next time you get a review and want to change something in the last commit you can just do git add and then git commit --amend --no-edit. Or a rebase to edit a previous commit if the reviewer wants you to change a commit that is not the HEAD of your branch.

Let me know if you have any problems during the process.

@aesteve-rh
Copy link
Collaborator

Well, almost... You've rebased over the last commit from main.

It's fine, happens. You'll probably need to do a soft reset of your commit, rebase from main, and commit your changes again. I can be of assist if you need. Ping me if so.

tests/ffi/common/README.md Outdated Show resolved Hide resolved
@dougsland
Copy link
Collaborator

I will let the others review but to my eyes looks good, thanks for the efforts to make it great.

@dougsland
Copy link
Collaborator

@aesteve-rh @Yarboa feel free to merge.

@aesteve-rh
Copy link
Collaborator

lgtm, thanks!

@dougsland dougsland merged commit 26ba84b into containers:main Sep 27, 2024
10 checks passed
@nsednev nsednev deleted the VROOM-22955 branch September 27, 2024 09:28
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.

Can't create ffi-qm container inside the QM partition on OStree image - no space left on device.
4 participants