Skip to content

Conversation

@anchapin
Copy link
Collaborator

Fixing packaging issues and added docker trigger step.

Alex Chapin added 5 commits December 30, 2025 08:48
When CTest failures trigger a rerun, the exit code from the initial failure
was previously retained even if the rerun passed. This caused the CI job
to fail despite successful retries.

This change ensures that overall_exit_code is reset to 0 if the rerun
command succeeds, while preserving the failure code if the rerun also fails.
Updated Linux, Mac, and Windows build jobs.
Updated the 'upload-artifact' path in linux, macos, and windows build jobs to specifically match 'OpenStudio-*.tar.gz'.
This prevents dependency tarballs (EnergyPlus, Radiance, etc.) from being included in the OpenStudio installer artifact, ensuring the uploaded asset is a single file as expected.
@anchapin anchapin added the Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge. label Dec 30, 2025
…s and ensuring proper signing of nested components
@github-actions
Copy link

github-actions bot commented Dec 30, 2025

🧪 Test Results Dashboard

Summary

Metric Value
Total Tests 4150
Passed 4150
Failed 0
Errors 0
Skipped 0
Success Rate 100.0%
Generated 2025-12-31 00:45:48 UTC

✅ All Tests Passed!

📊 Test Run Information

Run XML File Status
run1 results.xml ✅ Found
run3 results.xml ✅ Found
run2 results.xml ✅ Found

@ci-commercialbuildings
Copy link
Collaborator

ci-commercialbuildings commented Dec 30, 2025

CI Results for 98fd426:

@anchapin anchapin merged commit 231a2e3 into develop Dec 31, 2025
2 of 3 checks passed
@anchapin anchapin deleted the ci_fixes branch December 31, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants