-
Notifications
You must be signed in to change notification settings - Fork 27
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
[ci] Example Dockerfiles install from source / PyPI #3763
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3763 +/- ##
==========================================
+ Coverage 89.15% 89.17% +0.01%
==========================================
Files 54 54
Lines 6419 6419
==========================================
+ Hits 5723 5724 +1
+ Misses 696 695 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
e02fd94
to
5b4d90e
Compare
@ryan-williams nightly sounds delightful! :) |
Note this addition to OP:
The new GHA passed: I canceled the other GHAs, hence the ❌. |
os: | ||
- ubuntu-24.04 | ||
- ubuntu-24.04-arm | ||
# TODO: currently impossible(?) to build Docker images on macOS runners |
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.
Please file a tracking task (GitHub issue) and include its URL here
@@ -0,0 +1,98 @@ | |||
# Building/Installing TileDB-SOMA in Docker images | |||
|
|||
Sample Dockerfiles that `pip install tiledbsoma` (including installing required system deps). |
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.
Sample Dockerfiles that `pip install tiledbsoma` (including installing required system deps). | |
Sample Dockerfiles that `pip install tiledbsoma` (including installing required system dependencies). |
- run: docker build -f ${{ matrix.name }}.dockerfile -t ${{ matrix.name }} . | ||
working-directory: docker | ||
- run: docker run ${{ matrix.name }} | ||
working-directory: docker |
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.
Please add a stanza to cut a ticket on failure (there are other YAMLs in the same directory you can copy from)
Changes
docker/
: example Dockerfiles thatpip install tiledbsoma
README.md
includes build-debugging tips:pip install --no-clean tiledbsoma
VCPKG_FORCE_SYSTEM_BINARIES=1
(required by vcpkg on ARM)Add
python-dockers.yml
GHATests building and running
docker/*.dockerfile
:python -c 'import tiledbsoma; tiledbsoma.show_package_versions()'
orpython scripts/show-versions.py
on each image.Improve GHA path-filters
I also improved GHAs' path-filtering, so they don't run on changes to other workflow
.yml
s ordocker/**
.However, path-filters apparently don't take effect on the PR that adds them, so I canceled all unrelated GHAs here, to save runner quota.
Notes for Reviewer
Punted: macos Docker builds
python-dockers.yml
builds+runs on Ubuntu ARM and AMD platforms.Example failure
https://github.com/single-cell-data/TileDB-SOMA/actions/runs/13714468862/job/38356765298:
colima start
fails with:If we cat the
ha.stderr.log
before exiting:(source)
xrefs
actions/runner-images/#2150 (Add Docker to macOS)
discussions#69211 (GitHub Actions: Apple silicon (M1) macOS runners are now available in public beta!)
douglascamata/setup-docker-macos-action
setup-docker-macos-action#35
actions/runner-images#6216 (comment)