-
Notifications
You must be signed in to change notification settings - Fork 184
Description
Bug Report
Environment
What operating system is being used to run coreos-assembler?
Fedora 43
What operating system is being assembled?
fedora coreos
Is coreos-assembler running in Podman or Docker?
podman
If Podman, is coreos-assembler running privileged or unprivileged?
rootless with --privileged
Expected Behavior
When defining external tests for kola, the file kola.json should be used as default metadata for all tests in the test folder.
Actual Behavior
Only in binary metadatas (json and yaml) are used and kola.json is parsed, but never used
Reproduction Steps
- Create an external test. Put metadata in the kola.json file
- Run kola list -E /path/to/project
- On the ext.* tests, the metadata are not used
This procedure is for quick check, running or listing the tests is the same. Using label is a quick way to check.
Other Information
After reading the code I found the issue. in mantle/kola/harness.go, the function registerExternalTest will check if the binary have metadata in the binary using the metadataFromTestBinary function. If the result is nil, then kola.json is used, if not, the in binary metadata are used.
The commit 119bc7f that introduced the exclusive metadata, modified the function metadataFromTestBinary by initializing the meta variable with this expression :
meta := &externalTestMeta{Exclusive: true}Causing the function to never return a nil value, and thus kola.json will never be used.