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

Open
1 task
bolives-hax opened this issue Nov 11, 2024 · 1 comment
Open
1 task

s390x: CRC32/checksum tests failing #3185

bolives-hax opened this issue Nov 11, 2024 · 1 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.

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
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