Add YAML schemas for system tests components, tests, and metadata#722
Add YAML schemas for system tests components, tests, and metadata#722kishansinghifs1 wants to merge 1 commit into
Conversation
|
This pull request has been mentioned on preCICE Forum on Discourse. There might be relevant details there: https://precice.discourse.group/t/gsoc-2026-kishan-singh-system-tests-improvements/2775/1 |
There was a problem hiding this comment.
Thank you for demonstrating that #403 is possible, @kishansinghifs1. It looks like it works for tests.yaml and components.yaml.
How did you generate the schema?
There have been several changes since this PR was opened (I could only look at it closer now), and we would anyway need to add the respective headers to all related YAML files.
Would you still like to pursue this PR?
There was a problem hiding this comment.
The schema header still needs to be added to the various component templates.
You write that
The component template schema documents the expected Docker Compose service fragment structure; note that Jinja2 template syntax ({{ }}, {% %}) will still produce warnings in the editor since it's not valid YAML.
What exactly breaks? It is still a regular comment, no?
There was a problem hiding this comment.
There have recently been some changes in the schema in develop since #842: the repository is now an optional property of each build_args (actually, a required one of any property ending in _REF).
There was a problem hiding this comment.
This would still need to be added to the respective metadata.yaml files, right? Or can we centrally make it being picked up automatically for every file?
There was a problem hiding this comment.
There have been a few new parameters since this was opened: max_time, max_time_windows, timeout.
Sorry @MakisH due to the time constraint I would not be able to work further over the respective issue. Thank you!! |
Add YAML schemas for components, tests, and metadata : #403
Checklist:
changelog-entries/<PRnumber>.md.