-
Notifications
You must be signed in to change notification settings - Fork 222
testing out linux incremental build #5445
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…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.
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.
…for CTests and cache build directory
🧪 Test Results DashboardSummary
|
| Run | XML File | Status |
|---|---|---|
| run1 | results.xml |
✅ Found |
| run2 | results.xml |
✅ Found |
| run3 | results.xml |
✅ Found |
…and environment variable usage
…al build workflow
Collaborator
|
CI Results for 5f6b5aa:
|
…tency with Docker volume structure
… workspace preparation
…w for consistency
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.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Do not merge yet. Place holder for testing self hosted Github runners