search

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.25k stars 2.56k forks source link

Enhancement: Create ready-to-go development environment #5427

Open yuhaoth opened 2 years ago

yuhaoth commented 2 years ago

Suggested enhancement

Below is my suggestion. That can reduce the maintain loading.

Justification

Mbed TLS test environment depends on different version of OpenSSL, GnuTLS and compiler. It is not easy for developer to reproduce and figure out CI fail. We'd better provide a script or image to reproduce CI fail locally.

Docker files in mbedtls-test can be used as environment. But to build them locally still need some time. published images should be better choice. And tests/docker/bionic/Dockerfile might confuse developer, it should be removed.

gilles-peskine-arm commented 1 year ago

From @bensze01 : to grab the exact image OpenCI is using:

docker pull trustedfirmware/ci-amd64-mbed-tls-ubuntu:ubuntu-16.04-$(git hash-object resources/docker_files/ubuntu-16.04/Dockerfile)

(from a checkout of https://github.com/Mbed-TLS/mbedtls-test)

gilles-peskine-arm commented 1 year ago

tests/docker/bionic/Dockerfile might confuse developer, it should be removed.

Done in https://github.com/Mbed-TLS/mbedtls/pull/8131