Skip to content

Update STFL pipeline#130

Open
Guiliano99 wants to merge 15 commits into
open-quantum-safe:mainfrom
Guiliano99:UpdateSTFLPipeline
Open

Update STFL pipeline#130
Guiliano99 wants to merge 15 commits into
open-quantum-safe:mainfrom
Guiliano99:UpdateSTFLPipeline

Conversation

@Guiliano99
Copy link
Copy Markdown
Contributor

@Guiliano99 Guiliano99 commented Nov 19, 2025

Description

  • Add a new pipelien file to pre-generate XMSS and XMSS^MT keys (cache_xmss_and_xmssmt_keys.yml)
  • Remove the XMSS and XMSS^MT generated keys from the repository.

Motivation and Context

Note

  • The pipeline for generating the cache, must be restarted, because it takes longer than 6 hours.
  • Pipeline starts only after the merge.

- run: ruff format

Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
…S^MT key files for the pipeline.

Signed-off-by: Guiliano99 <guilianolehmann@live.de>
- Generates the needed XMSS and XMSS^MT keys over different runs.

Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
…S^MT keys.

- Restores the pre-generated keys.
- Display what keys were restored.

Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
@Guiliano99 Guiliano99 marked this pull request as draft November 19, 2025 11:33
@Guiliano99 Guiliano99 marked this pull request as ready for review November 19, 2025 11:44
Guiliano99 and others added 3 commits December 2, 2025 10:56
# Conflicts:
#	.github/workflows/python_detailed.yml
#	.github/workflows/python_simplified.yml
Signed-off-by: Guiliano Lehmann <129761072+Guiliano99@users.noreply.github.com>
Signed-off-by: Guiliano99 <guilianolehmann@live.de>
@Guiliano99
Copy link
Copy Markdown
Contributor Author

Should SLH-DSA be included in this PR, or addressed in a follow-up?

@Guiliano99
Copy link
Copy Markdown
Contributor Author

I wouldn't want to add a new dependency to liboqs-python in general. Would this be only for testing? And perhaps we should move the discussion over to #130.

@dstebila Yes, it would only be used for testing, unless you also want to support key export for SLH-DSA, ML-KEM, and ML-DSA (to PEM).

Since I install a stable version of liboqs, this could make the tests safer: issues related to key loading would be identified directly when testing against the latest version. But maybe this makes it more complicated.

@dstebila
Copy link
Copy Markdown
Member

@dstebila Yes, it would only be used for testing, unless you also want to support key export for SLH-DSA, ML-KEM, and ML-DSA (to PEM).

I don't have my own opinions of what the scope of this Python wrapper should be, whether key export to PEM is desirable (I guess #78 is somewhat along these lines), and the extent to which other tools in the ecosystem already provide that functionality. I'm somewhat inclined to keep our scope relatively narrow, in part due to limited resources within the project.

Since I install a stable version of liboqs, this could make the tests safer: issues related to key loading would be identified directly when testing against the latest version. But maybe this makes it more complicated.

I'm not sure I understand. Is this basically talking about compatibility between one version of the liboqs-python wrapper and the next?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants