-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[CI] Don't upload build caches to S3 for Mac15 and Mac26. #20113
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
base: master
Are you sure you want to change the base?
Conversation
The source and build caches aren't used since 4c3335e, so there is no urgent need to generate them at all. Since the Mac VMs sometimes hang for several hours at the upload step, it seems less fragile to not upload or download anything.
baff6a8 to
04d2dfc
Compare
Test Results 22 files 22 suites 3d 15h 2m 32s ⏱️ Results for commit 9e34c62. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about adding these lines to Node state to find out even more about the environment?
echo $SHELL
echo $0
python3 -m site
pkgutil --pkg-info=com.apple.pkg.CLTools_Executables
5f796ea to
3aabb95
Compare
3aabb95 to
9e34c62
Compare
| ${{ (github.event_name != 'schedule' && github.event_name != 'workflow_dispatch' && join( matrix.overrides, ', ' )) || '' }} | ||
| steps: | ||
| - name: Node state |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this step meant to be merged?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we would like to add some diagnostics for the node to the CI runs.
| shell: bash -leo pipefail {0} | ||
| if: github.event_name == 'push' | ||
| run: ".github/workflows/root-ci-config/build_root.py | ||
| if: ${{ github.event_name == 'push' && !matrix.platform == 'mac15' && !matrix.platform == 'mac26' }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to link/relate closer this with the disabling of the use? (I.e. to avoid someone later one trying to renable the use by forgetting to revert this).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might be able to write some info in the environment variables, like so:
<download cached artifacts>
echo ROOT_CI_UPLOAD_ARTIFACTS=1 >> $GITHUB_ENV
This environment variable could be used to trigger the upload step. What do you think @pcanal ?
For Mac15 and Mac26, the source and build caches aren't used since 4c3335e, so there is no urgent need to generate them at all. Since the Mac VMs sometimes hang for several hours at the upload step, it seems less fragile to not upload or download anything.