Skip to content

Conversation

@tijcolem
Copy link
Collaborator

Do not merge yet. Place holder for testing self hosted Github runners

tijcolem and others added 4 commits June 25, 2025 14:22
…orkflow

Quote Docker volume paths to handle special characters and fix git safe.directory path. Also improve formatting of git commands and use explicit build folder name instead of variable.
Comment out unused environment variables and directly specify volume paths in the container configuration for better clarity and maintainability.
@anchapin anchapin added Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge. labels Jun 25, 2025
anchapin and others added 24 commits June 25, 2025 14:52
Update container paths to use consistent environment variables and change the Conan build output directory from OS-build-release to OS-build-release-v2. Add OPENSTUDIO_SOURCE_VOLUME variable for better path management.
Simplify the Docker volume configuration by directly specifying the full path to the OpenStudio source volume instead of using a derived environment variable.
Print the BRANCH_NAME environment variable to help with troubleshooting the GitHub Actions workflow execution context.
Update branch name variable to properly handle pull request builds by using github.head_ref when available, falling back to github.ref_name for direct branch builds.
Replace BRANCH_NAME with PR_NUMBER environment variable and improve extraction logic to handle both pull request numbers and branch name formats.
Use github.ref instead of the previous complex expression for better reference handling in the CI pipeline.
Update the PR_NUMBER environment variable to correctly extract the pull request number from GitHub event context, falling back to github.ref when not available.
Updates the incremental build workflow by removing unnecessary Conan-related environment variables and fixing the volume mount path to use /home/root instead of /home/ubuntu/conan-data.
Add diagnostic commands to display available disk space and list home directory contents in the incremental build workflow, helping to troubleshoot CI environment issues.
Add ls command to check /home/root directory and tail command to prevent
workflow from exiting prematurely during troubleshooting.
Map the Conan 2 cache directory correctly to preserve cache between builds
The workflow no longer needs the `tail -f /dev/null` command that was likely used for debugging purposes.
The change disables the automatic profile detection step in the Conan package manager configuration, while preserving the rest of the installation process.
Refactor GitHub workflow by introducing environment variables for Docker volumes and paths. Enable CMake configuration step with detailed build options for OpenStudio. Fix directory structure references for better maintainability.
Correct the reference to OPENSTUDIO_SOURCE_NAME by adding the 'env.' prefix to properly access the environment variable in the CMake configuration step.
Corrects the indentation of the CMake command in the incremental build workflow, improving readability while maintaining the same functionality.
The change removes the Python setup step from the GitHub Actions workflow by commenting it out, as it may not be necessary for the incremental build process.
Add tail -f /dev/null command to pause execution in the GitHub workflow, allowing for debugging and investigation of the build environment before proceeding with the build process.
@github-actions
Copy link

github-actions bot commented Jun 27, 2025

🧪 Test Results Dashboard

Summary

Metric Value
Total Tests 4086
Passed 4076
Failed 10
Errors 0
Skipped 0
Success Rate 99.8%
Generated 2025-06-27 23:01:38 UTC

⚠️ Minor Issues Detected

🔍 Failed Tests (10 failures)

Linux-c++ (10 failures)

gbXMLFixture.ForwardTranslator_exampleModelValid.gbXMLFixture.ForwardTranslator_exampleModelValid (run1)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run1)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run1)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run1)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run2)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run2)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run2)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run3)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run3)

Error Message:

Failed

Full Details:

No details available
CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run3)

Error Message:

Failed

Full Details:

No details available

📊 Test Run Information

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

@ci-commercialbuildings
Copy link
Collaborator

ci-commercialbuildings commented Jun 27, 2025

CI Results for 5f6b5aa:

@anchapin anchapin merged commit 37db4c4 into develop Jun 27, 2025
2 of 3 checks passed
@jmarrec jmarrec deleted the self-hosted branch July 4, 2025 11:15
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.

4 participants