Skip to content

Conversation

@jlausuch
Copy link
Contributor

@jlausuch jlausuch commented Mar 27, 2023

ALP produces a non-compressed RAW images for the encrypted flavor. Without this change, it's set as ASSET_1 instead of HDD_1: ASSET_1=ALP-Micro.x86_64-0.1-Default-encrypted-Build2.2.raw and the test fails:
https://openqa.opensuse.org/tests/3193166

Follow up from #209

ASSET_256=ALP-Bedrock.x86_64-0.1-Default-encrypted-Build2.2.raw.sha256 \
BUILD=2.2 \
CHECKSUM_ASSET_1=$(cut -b-64 /var/lib/openqa/factory/other/ALP-Bedrock.x86_64-0.1-Default-encrypted-Build2.2.raw.sha256 | grep -E '[0-9a-f]{5,40}' | head -n1) \
CHECKSUM_HDD_1=$(cut -b-64 /var/lib/openqa/factory/other/ALP-Bedrock.x86_64-0.1-Default-encrypted-Build2.2.raw.sha256 | grep -E '[0-9a-f]{5,40}' | head -n1) \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the important change to review, the other files are just build number changes.

DISTRI=alp \
FLAVOR=Default-encrypted \
FULLURL=1 \
HDD_1=ALP-Bedrock.x86_64-0.1-Default-encrypted-Build2.2.raw \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will fix this error: https://openqa.opensuse.org/tests/3193166
where HDD_1 was missing.

elif [[ $destiso =~ \.iso$ ]]; then
echo \" ''' + openqa_call_start_ex1(checksum, 'ISO') + '''\"
elif [[ $destiso =~ \.(hdd|qcow2|raw\.xz|raw\.gz|vhdx\.xz|vmdk\.xz)$ ]]; then
elif [[ $destiso =~ \.(hdd|qcow2|raw|raw\.xz|raw\.gz|vhdx\.xz|vmdk\.xz)$ ]]; then
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the main feature change.

@jlausuch jlausuch requested a review from andrii-suse March 27, 2023 07:38
ALP produces a non-compressed RAW images for the encrypted flavor.
Without this change, it's set as ASSET_1 instead of HDD_1:
ASSET_1=ALP-Micro.x86_64-0.1-Default-encrypted-Build2.2.raw
and the test fails:
https://openqa.opensuse.org/tests/3193166
@andrii-suse andrii-suse merged commit 5f7fb76 into os-autoinst:master Mar 27, 2023
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.

2 participants