Skip to content

gha/add llvmlite osx 64 conda builder #1190

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 22 commits into from
Apr 17, 2025

Conversation

swap357
Copy link
Contributor

@swap357 swap357 commented Apr 2, 2025

this PR adds osx64 conda builder workflow for llvmlite.
It needs same fixes for osx-64 as llvmdev workflow PR - https://github.com/numba/llvmlite/pull/1167/files
This PR is a superset of #1167

@swap357 swap357 marked this pull request as ready for review April 2, 2025 00:32
@swap357 swap357 requested a review from esc April 2, 2025 00:32
@swap357 swap357 mentioned this pull request Apr 2, 2025
20 tasks
@esc
Copy link
Member

esc commented Apr 8, 2025

I am testing the llvmlite_osx-64_conda_builder using the ID 14313666110 from https://github.com/numba/llvmlite/actions/runs/14313666110 at:

https://github.com/esc/llvmlite/actions/runs/14333921715

however, it seems there is an issue trying to download the artifact:

Run actions/download-artifact@v4
  with:
    name: llvmdev_osx-64
    path: llvmdev_conda_packages
    run-id: 14[3](https://github.com/esc/llvmlite/actions/runs/14333921715/job/40176074824#step:6:3)13666110
    repository: esc/llvmlite
    github-token: ***
    merge-multiple: false
  env:
    INPUT_RUN_POST: true
    CONDA: /Users/runner/miniconda3
Downloading single artifact
Error: Unable to download artifact(s): Not Found

sadly I don't know what the issue is here. Could it be that I need to have run the llvmdev in my fork also?

@esc
Copy link
Member

esc commented Apr 8, 2025

Tried a build of llvmdev on main of the esc/llvmlite fork:

https://github.com/esc/llvmlite/actions/runs/14334012418

The used the workflow ID 14334012418 to launch:

https://github.com/esc/llvmlite/actions/runs/14337845023

@esc
Copy link
Member

esc commented Apr 9, 2025

@swap357 thank you for the pushes, I will re-review this now.

@esc
Copy link
Member

esc commented Apr 9, 2025

Testing protocol and results:

a) merged this PR to esc/llvmlite@main: esc@0b15a08

b) used a GHA produced llvmdev from https://github.com/esc/llvmlite/actions/runs/14334012418

c) ran: https://github.com/esc/llvmlite/actions/runs/14354807935

This was green. The code in this PR is fine. But it can not yet be merged until the following PRs are also green:

#1187

numba/numba#10023

and the PR for numba osx-64 wheels that has ye to appear.

As a result I will label this as "waiting on CI".

@seibert
Copy link
Contributor

seibert commented Apr 17, 2025

#1187 has been merged. It looks like numba/numba#10023 is also working, and I don't know if we need to wait for the numba osx-64 wheel PR to be created before this one is. I'm going to keep this all moving forward, and if we need to open some smaller PRs to fix up issues found later, we can do that.

@seibert seibert dismissed esc’s stale review April 17, 2025 20:05

esc's requested change has been made, but he's not available this week to update his review.

Copy link
Contributor

@seibert seibert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes @esc requested seem to have been made. Approving.

@seibert seibert merged commit 2fbac84 into numba:main Apr 17, 2025
40 checks passed
@swap357
Copy link
Contributor Author

swap357 commented Apr 17, 2025

numba wheel builder PR has been created and tested here - numba/numba#10038

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

Successfully merging this pull request may close these issues.

3 participants