Skip to content

Fix tck not properly parsing ci.json#875

Merged
jormundur00 merged 2 commits intomasterfrom
jormundur00/gh-855
Dec 10, 2025
Merged

Fix tck not properly parsing ci.json#875
jormundur00 merged 2 commits intomasterfrom
jormundur00/gh-855

Conversation

@jormundur00
Copy link
Copy Markdown
Member

@jormundur00 jormundur00 commented Dec 10, 2025

What does this PR do?

This PR resolves an issue where internal TCK builds failed to retrieve the necessary native-image build arguments (such as optimization flags like -Ob) from the central ci.json configuration file, causing tests to run with incorrect or default configurations.

Also in this PR, we add a hard-fail when ci.json is missing.

Fixes: #855

@jormundur00 jormundur00 self-assigned this Dec 10, 2025
@jormundur00 jormundur00 added the bug Something isn't working label Dec 10, 2025
@vjovanov
Copy link
Copy Markdown
Member

We need to test all metadata manually.

@jormundur00
Copy link
Copy Markdown
Member Author

We need to test all metadata manually.

Due to the change in ci.json (because --verbose should be used, not -verbose), the test-all-metadata workflow ran automatically, and successfully completed.

@jormundur00 jormundur00 merged commit 25015d3 into master Dec 10, 2025
62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: test-changed-metadata.yml uses -O2 instead of -Ob, as specified by ci.json

2 participants