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

s390x: CRC32/checksum tests failing #3185

Closed
1 task
bolives-hax opened this issue Nov 11, 2024 · 3 comments
Closed
1 task

s390x: CRC32/checksum tests failing #3185

bolives-hax opened this issue Nov 11, 2024 · 3 comments
Labels
bug This issue is a bug. p2 This is a standard priority issue pending-release This issue will be fixed by an approved PR that hasn't been released yet.

Comments

@bolives-hax
Copy link

Describe the bug

sdk-aws-log.txt

While adding s390x (IBM system Z10-Z15 mainframe arch) support to NixOS, i encountered the following issue trying to build this package during the test phase: (see log)

I suspect endianess is to blame here but im not sure, disabling some tests like:

    rm tests/aws-cpp-sdk-core-tests/utils/memory/AWSMemoryTest.cpp
    rm tests/aws-cpp-sdk-core-tests/utils/event/EventStreamDecoderTest.cpp
    rm tests/aws-cpp-sdk-core-tests/utils/event/EventStreamTest.cpp
    rm tests/aws-cpp-sdk-core-tests/utils/HashingUtilsTest.cpp

makes the build complete without any errors/failures. Building the same package for other architectures works thus I suspect due to the nature of these issues its an endianess related problem.

I couldn't confirm if the lib works like intended on s390x but building my system against it seemed to not trigger any issues so its either not getting used or despite failing these tests works.

Regression Issue

  • Select this option if this issue appears to be a regression.

Expected Behavior

I expected the tests not to fail like on other architectures

Current Behavior

various issues such as:

ESC[0;31m[  FAILED  ] ESC[mHashingUtilsTest.TestCRC32CFromStream (154 ms)
/build/source/tests/aws-cpp-sdk-core-tests/utils/HashingUtilsTest.cpp:406: Failure
Expected equality of these values:
  expectedHexHash
    Which is: "91267e8a"
  hexHash.c_str()
    Which is: "1c6a2029"
ESC[0;31m[  FAILED  ] ESC[mHashingUtilsTest.TestCRC32FromStream (156 ms)

taking place

Reproduction Steps

I assume that its highly unlikely someone will reproduce it as most likely im the only person on earth atm running NixOS on a mainframe as support only got recently added by me. But I will still share one of the releases I made a while ago that relies on these tests being disabled

https://github.com/bolives-hax/nixos-s390x-images/releases/tag/1.0.0

I think it may be easier to ask me for a s390x container that getting this to run in qemu(emul). If one still wants to give the emulator a try I'd suggest going with a distro that offers prebuild compilers/build packages for s390x

Possible Solution

No response

Additional Information/Context

No response

AWS CPP SDK version used

aws-sdk-cpp-1.11.336

Compiler and Version used

gcc GNU 13.3.0

Operating System and version

NixOS 24.11 / master

@bolives-hax bolives-hax added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Nov 11, 2024
@DmitriyMusatkin
Copy link
Contributor

this will be fixed once cpp sdk integrates this awslabs/aws-checksums@b23b2e2.
big endian is currently not supported

@jmklix jmklix added pending-release This issue will be fixed by an approved PR that hasn't been released yet. p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Nov 11, 2024
@jmklix
Copy link
Member

jmklix commented Dec 12, 2024

The fixed was merged when the crt version was bumped in this PR. Please update to the latest version of this sdk

@jmklix jmklix closed this as completed Dec 12, 2024
Copy link

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. p2 This is a standard priority issue pending-release This issue will be fixed by an approved PR that hasn't been released yet.
Projects
None yet
Development

No branches or pull requests

3 participants