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

sync disks before freeing loop device #7593

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

swissiety
Copy link
Contributor

Description

see comments in #7527
freeing loop devices seems unstable and fails freeing the loop device.
syncing disks helped at least addressing (loop) device problems in my last contribution (fix lvm,luks extension)

How Has This Been Tested?

  • copied the config from the repo
  • compiled with mentioned failing command from the daily builds
    shell ./compile.sh armbian-images build BOARD=uefi-x86 BRANCH=current BUILD_DESKTOP=yes BUILD_MINIMAL=no DESKTOP_APPGROUPS_SELECTED= DESKTOP_ENVIRONMENT=gnome DESKTOP_ENVIRONMENT_CONFIG_NAME=config_base ENABLE_EXTENSIONS=image-output-qcow2,v4l2loopback-dkms EXPERT=yes IMAGE_XZ_COMPRESSION_RATIO=1 KERNEL_CONFIGURE=no RELEASE=bookworm SHARE_LOG=yes SHOW_DEBUB=yes

multiple compile runs, could not reproduce getting an error anymore - but not confident that this was it.

@swissiety swissiety requested a review from a team as a code owner December 18, 2024 14:04
@github-actions github-actions bot added size/small PR with less then 50 lines Needs review Seeking for review Framework Framework components labels Dec 18, 2024
@igorpecovnik
Copy link
Member

not confident that this was it.

Yeah, this is not the problem. I ran tests on CI infrastructure where several images are assembled in parallel. And it fails at once. I use this branch to test against: https://github.com/armbian/build/tree/test

Tests:

  1. With this fix - no changes - failing quickly : https://github.com/armbian/os/actions/runs/12397113158
  2. With reverting all three commits - no fails (of this kind): https://github.com/armbian/os/actions/runs/12397514328

@igorpecovnik igorpecovnik added Work in progress Unfinished / work in progress Bugfix Pull request is fixing a bug labels Dec 18, 2024
@swissiety swissiety force-pushed the try2fix_freeing_loop_device branch from c9cd1b4 to 6c401f2 Compare December 20, 2024 17:33
@swissiety swissiety force-pushed the try2fix_freeing_loop_device branch from 6c401f2 to 18c6584 Compare December 20, 2024 17:37
@swissiety
Copy link
Contributor Author

@igorpecovnik is there a way I can run the changes on the gh-actions infrastructure?

@igorpecovnik
Copy link
Member

I run it and it fails :( Let me give you rights to run this ...

@igorpecovnik
Copy link
Member

Once you accept invitation, there is a branch "test" in the https://github.com/armbian/build/ (you will be able to push there)

And then just start this:
https://github.com/armbian/os/actions/workflows/complete-artifact-matrix-nightly.yml

and choose test branch:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix Pull request is fixing a bug Framework Framework components Needs review Seeking for review size/small PR with less then 50 lines Work in progress Unfinished / work in progress
Development

Successfully merging this pull request may close these issues.

2 participants