Skip to content

Conversation

@hageboeck
Copy link
Member

@hageboeck hageboeck commented Oct 15, 2025

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.

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.
@hageboeck hageboeck requested a review from Ni-Langguth October 15, 2025 11:19
@hageboeck hageboeck self-assigned this Oct 15, 2025
@hageboeck hageboeck requested a review from dpiparo as a code owner October 15, 2025 11:19
@github-actions
Copy link

github-actions bot commented Oct 15, 2025

Test Results

    22 files      22 suites   3d 15h 2m 32s ⏱️
 3 689 tests  3 689 ✅ 0 💤 0 ❌
79 225 runs  79 225 ✅ 0 💤 0 ❌

Results for commit 9e34c62.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@Ni-Langguth Ni-Langguth left a 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

@hageboeck hageboeck force-pushed the upload_macVM branch 2 times, most recently from 5f796ea to 3aabb95 Compare October 15, 2025 15:11
${{ (github.event_name != 'schedule' && github.event_name != 'workflow_dispatch' && join( matrix.overrides, ', ' )) || '' }}
steps:
- name: Node state
Copy link
Member

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?

Copy link
Contributor

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' }}
Copy link
Member

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).

Copy link
Member Author

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 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants