psa_sim: improve log prints

Mbed-TLS / mbedtls

An open source, portable, easy to use, readable and flexible TLS library, and reference implementation of the PSA Cryptography API. Releases are on a varying cadence, typically around 3 - 6 months between releases.

https://www.trustedfirmware.org/projects/mbed-tls/

Other

5.02k stars 2.5k forks source link

psa_sim: improve log prints #9278

Closed valeriosetti closed 1 week ago

valeriosetti commented 1 week ago

Description

This PR is an extract from #9237 in order to split different topics in different PRs. Here's what this one addresses:

always print ERROR and FATAL messages because they should never occur, but when they do it's important to see them immediately;
keep INFO prints under DEBUG guard;
set client's PRINT as INFO message because otherwise it will mess test_suites's output;
change some error messages from INFO to ERROR because that's what they are.

PR checklist

[ ] changelog not required
[ ] 3.6 backport not required
[ ] 2.28 backport not required
[ ] tests not required

tom-cosgrove-arm commented 1 week ago

~FYI: I still need to add another commit here to update the other Perl script~ Now done

tom-cosgrove-arm commented 1 week ago

The CI error here is

[2024-06-18T22:44:01.073Z] m->m dtls12,no TLS_DHE_RSA_WITH_AES_256_CBC_SHA256 ..................... tests/compat.sh: 876: kill: No such process
[2024-06-18T22:44:01.073Z] 
[2024-06-18T22:44:01.073Z] FAIL
[2024-06-18T22:44:01.073Z]   ! outputs saved to c-srv-139.log, c-cli-139.log

which is not related to this PR

tom-cosgrove-arm commented 1 week ago

As this PR only contains changes under tests/psa-client-server/psasim/ (and OpenCI is green) I am going to bypass the merge queue