[DI] Move scripts used in the CI to the framework #52

New issue

Closed

opened 2024-10-03 04:13:18 -05:00 by mpg · 1 comment

mpg commented

2024-10-03 04:13:18 -05:00

(Migrated from github.com)

[Moved and re-organized from #12 and #16.]

Note: the scope (as a first step) is now only to move to the framework, not to adapt to tf-psa-crypto. Some comments and size estimates are still from the time where the scope included adapting to tf-psa-crypto.

The lists below are temporary and might still evolve.

Table legend:
I: Currently identical between 3.6 and development?
S: Work size for the move and potentially changes needed for TF-PSA-Crypto

Moving to the framework - used by tf-psa-crypto

`name`	I	S	Issue/PR	Comment
`analyze_outcomes.py`	y	S	~~https://github.com/Mbed-TLS/mbedtls-framework/pull/55, https://github.com/Mbed-TLS/mbedtls/pull/9668, https://github.com/Mbed-TLS/mbedtls/pull/9669~~	(done)
`apidoc_full.sh`	y	S	#69	called by doxygen.sh, depends on config.py, needs changes for TF-PSA-Crypto
`assemble_changelog.py`	y	XS	#69	47 adapted to TF-PSA-Crypto
`check-doxy-blocks.pl`	y	s	#69	https://github.com/mbed-tls/tf-psa-crypto/issues/49 to adapt to tf-psa-crypto +doxygen.sh
`check-python-files.sh`	y	xs	#69
`doxygen.sh`	y	xs	#69
`lcov.sh`	y	xs	#93	(41 adapted to tf-psa-crypto)
`min_requirements.py`	y	xs	#84	ci scripts
`output_env.sh`	y	xs	#74	called by all.sh, basic-build-test.sh
`psa_collect_statuses.py`	y	s	#87
`recursion.pl`	y	xs	#69
`run-metatests.sh`	y	s	#87
`run_demos.py`	y	xs	#87
`test_zeroize.gdb`	y	-	#87
`abi_check.py`	n	S	#89	CI scripts
`all.sh`	n	M	~~https://github.com/Mbed-TLS/mbedtls/pull/8226~~	~~see #40~~ (done)
`basic-build-test.sh`	n	S	#90	https://github.com/Mbed-TLS/mbedtls-framework/issues/12#issuecomment-2370807342
`check-generated-files.sh`	n	xs	#71	adaptation to tf-psa-crypto done
`check_files.py`	n	xs	#72	see also https://github.com/mbed-tls/mbedtls-framework/issues/7
`check_names.py`	n	s	#73	M for tf-psa-crypto adaptation
`check_test_cases.py`	n	xs	~~https://github.com/Mbed-TLS/mbedtls-framework/pull/55~~	(done)
`code_style.py`	n	xs	#70	48 for `tf-psa-crypto`
`depends.py`	n	>m	#97 + #98
`generate_features.pl`	y	s	#91
`generate_psa_constants.py`	n	xs	#91	called by builds, check-generated-files.sh
`generate_query_config.pl`	n	m	#91	adapt to mbedtls_config.h split, adapt to tf_psa_crypto_config.h
`list_internal_identifiers.py`	n	xs	#92	+ list-identifiers.sh ci scripts
`test-ref-configs.pl`	n	s+s	~~#9565~~	(removed)
`test_psa_compliance.py`	n	-	#94	adaptation to tf-psa-crypto done
`test_psa_constant_names.py`	n	xs	#94

Moving to the framework - not used by tf-psa-crypto

`name`	I	S	Issue/PR	Comment
~~`generate_ssl_debug_helpers.py`~~	y	xs	#9673	called by builds, check-generated-files.sh
~~`generate_tls13_compat_tests.py`~~	y	xs	#9673
~~`translate_ciphers.py`~~	y	-	#9673	called by compat.sh

Not moving to the framework

`name`	I	S	Comment
`basic.requirements.txt`	y	-	branch specific
`ci.requirements.txt`	y	-	branch specific
`config.py`	n	-	branch specific
`driver.requirements.txt`	y	-	branch specific
`generate_driver_wrappers.py`	n	-	branch specific
`generate_errors.pl`	n		with #8501 no such script in 4.0
`generate_visualc_files.pl`	n
`maintainer.requirements.txt`	y	-	branch specific
`make_generated_files.bat`	a
`run-test-suites.pl`	n	-	make test
`compat.sh`	n	-	not worth the effort at this point
`ssl-opt.sh`	n	-	not worth the effort at this point

[Moved and re-organized from #12 and #16.] Note: the scope (as a first step) is now only to move to the framework, not to adapt to tf-psa-crypto. Some comments and size estimates are still from the time where the scope included adapting to tf-psa-crypto. The lists below are temporary and might still evolve. Table legend: I: Currently identical between 3.6 and development? S: Work size for the move and potentially changes needed for TF-PSA-Crypto Moving to the framework - used by tf-psa-crypto --------------------------------------------- | `name` | I | S | Issue/PR | Comment | | ------ | - | - | -------- | ------- | | `analyze_outcomes.py` | y | S | ~https://github.com/Mbed-TLS/mbedtls-framework/pull/55, https://github.com/Mbed-TLS/mbedtls/pull/9668, https://github.com/Mbed-TLS/mbedtls/pull/9669~ | (done) | | `apidoc_full.sh` | y | S |#69| called by doxygen.sh, depends on config.py, needs changes for TF-PSA-Crypto | | `assemble_changelog.py` | y | XS | #69 | [47](https://github.com/Mbed-TLS/TF-PSA-Crypto/issues/47) adapted to TF-PSA-Crypto | | `check-doxy-blocks.pl` | y | s | #69 | https://github.com/mbed-tls/tf-psa-crypto/issues/49 to adapt to tf-psa-crypto +doxygen.sh | | `check-python-files.sh` | y | xs | #69 | | | `doxygen.sh` | y | xs | #69 | | | `lcov.sh` | y | xs | #93 | ([41](https://github.com/mbed-tls/tf-psa-crypto/issues/41) adapted to tf-psa-crypto) | | `min_requirements.py` | y | xs | #84 | ci scripts | | `output_env.sh` | y | xs | #74 | called by all.sh, basic-build-test.sh | | `psa_collect_statuses.py` | y | s | #87 | | | `recursion.pl` | y | xs | #69 | | | `run-metatests.sh` | y | s | #87 | | | `run_demos.py` | y | xs | #87 | | | `test_zeroize.gdb` | y | - | #87 | | | `abi_check.py` | n | S | #89 | CI scripts | | `all.sh` | n | M | ~https://github.com/Mbed-TLS/mbedtls/pull/8226~ | ~see #40~ (done) | | `basic-build-test.sh` | n | S | #90 | https://github.com/Mbed-TLS/mbedtls-framework/issues/12#issuecomment-2370807342 | | `check-generated-files.sh` | n | xs | #71 | adaptation to tf-psa-crypto done | | `check_files.py` | n | xs | #72 | see also https://github.com/mbed-tls/mbedtls-framework/issues/7 | | `check_names.py` | n | s | #73 | M for tf-psa-crypto adaptation | | `check_test_cases.py` | n | xs | ~https://github.com/Mbed-TLS/mbedtls-framework/pull/55~ | (done) | | `code_style.py` | n | xs | #70 | [48](https://github.com/mbed-tls/tf-psa-crypto/issues/48) for `tf-psa-crypto` | | `depends.py` | n | >m | #97 + #98 | | | `generate_features.pl` | y | s | #91 | | | `generate_psa_constants.py` | n | xs | #91 | called by builds, check-generated-files.sh | | `generate_query_config.pl` | n | m | #91 | adapt to mbedtls_config.h split, adapt to tf_psa_crypto_config.h | | `list_internal_identifiers.py` | n | xs | #92 | + list-identifiers.sh ci scripts | | `test-ref-configs.pl` | n | s+s | ~#9565~ | (removed) | | `test_psa_compliance.py` | n | - | #94 | adaptation to tf-psa-crypto done | | `test_psa_constant_names.py` | n | xs | #94 | | Moving to the framework - not used by tf-psa-crypto ------------------------------------------------- | `name` | I | S | Issue/PR | Comment | | ------ | - | - | -------- | ------- | | ~`generate_ssl_debug_helpers.py`~ | y | xs | #9673 | called by builds, check-generated-files.sh | | ~`generate_tls13_compat_tests.py`~ | y | xs | #9673 | | | ~`translate_ciphers.py`~ | y | - | #9673 | called by compat.sh | Not moving to the framework --------------------------- | `name` | I | S | Issue/PR | Comment | | ------ | - | - | -------- | ------- | | `basic.requirements.txt` | y | - | | branch specific | | `ci.requirements.txt` | y | - | | branch specific | | `config.py` | n | - | | branch specific | | `driver.requirements.txt` | y | - | | branch specific | | `generate_driver_wrappers.py` | n | - | | branch specific | | `generate_errors.pl` | n | | | with #8501 no such script in 4.0 | | `generate_visualc_files.pl` | n | | | | | `maintainer.requirements.txt` | y | - | | branch specific | | `make_generated_files.bat` | a | | | | | `run-test-suites.pl` | n | - | | make test | | `compat.sh` | n | - | | not worth the effort at this point | | `ssl-opt.sh` | n | - | | not worth the effort at this point |

mpg commented

2024-12-09 05:07:56 -06:00

(Migrated from github.com)

Investigation completed; tasks created and estimated.

No milestone

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

gh-Mbed-TLS/mbedtls-framework#52

No description provided.

Rows
Columns