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

tests/kola: bump minDisk in autosave-xfs tests #2359

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Apr 11, 2023

The actual agcount on a 64G disk depends on the agsize baked in the image, which in turn depends on how large the rootfs was calculated to be (see estimate-commit-disk-size in cosa).

Since the starting rootfs size will differ on different arches (notably, be larger than x86_64) and minor variations are greatly amplified, the agcount may be lower than expected for a 64G disk and in turn the test can fail. Even on x86_64, the rootfs size can change e.g. in CI builds where we bake non-stripped binaries.

Bump the requested disk size to 100G in the test to make it more foolproof.

Closes: coreos/fedora-coreos-tracker#1458

The actual agcount on a 64G disk depends on the agsize baked in the
image, which in turn depends on how large the rootfs was calculated to
be (see `estimate-commit-disk-size` in cosa).

Since the starting rootfs size will differ on different arches (notably,
be larger than x86_64) and minor variations are greatly amplified, the
agcount may be lower than expected for a 64G disk and in turn the test
can fail. Even on x86_64, the rootfs size can change e.g. in CI builds
where we bake non-stripped binaries.

Bump the requested disk size to 100G in the test to make it more
foolproof.

Closes: coreos/fedora-coreos-tracker#1458
@jlebon
Copy link
Member Author

jlebon commented Apr 11, 2023

Verified these tests now pass on an aarch64 machine!

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe dustymabe enabled auto-merge (rebase) April 11, 2023 15:42
@dustymabe dustymabe merged commit 07f87e3 into coreos:testing-devel Apr 11, 2023
1 check passed
@jlebon jlebon deleted the pr/bump-autosave-xfs branch April 23, 2023 23: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
2 participants