ci(macos): full integration lane via colima on macos-15-intel - #447
Open
MattJackson wants to merge 3 commits into
Open
MattJackson wants to merge 3 commits into
MattJackson wants to merge 3 commits into
Conversation
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 19:43
e9a4cbb to
58ba36e
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 20:24
58ba36e to
d234176
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 21:17
d234176 to
eb252de
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 22:10
eb252de to
77f8487
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 23:06
77f8487 to
a4526b0
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 2, 2026 23:54
a4526b0 to
cb63d6a
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 3, 2026 00:01
cb63d6a to
ceefa22
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 3, 2026 00:50
ceefa22 to
53e057f
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 3, 2026 01:10
53e057f to
6a707e5
Compare
MattJackson
changed the title
MattJackson
force-pushed
the
stack/s8
branch
from
September 3, 2026 15:05
7bba903 to
1066eca
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 4, 2026 20:44
1066eca to
c968032
Compare
Contributor
Author
|
@aqrln rebased onto the merged main, all green and MERGEABLE — ready for review whenever you get a chance. Thanks! |
MattJackson
force-pushed
the
stack/s8
branch
from
September 6, 2026 17:27
c968032 to
ff1189e
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 7, 2026 00:01
ff1189e to
3c2a87d
Compare
Contributor
Author
|
Rebased onto the updated stack; green. CI cleanup: dropped a |
MattJackson
force-pushed
the
stack/s8
branch
from
September 8, 2026 18:07
968367f to
f5862d3
Compare
MattJackson
force-pushed
the
stack/s8
branch
from
September 15, 2026 23:02
f5862d3 to
0aa6129
Compare
Replace the build+unit-only macOS job with a full integration lane: an Intel macos-15-intel runner hosts the linux/amd64 SQL Server 2022 image inside a colima (Lima) Linux VM (native amd64, no qemu) and runs the suite across rustls and vendored-openssl (both required). native-tls resolves to Apple Secure Transport, which cannot complete the SQL Server handshake, so it runs continue-on-error (tracked follow-up). Readiness gates on an authenticated SELECT 1 to avoid the 18456 login race. Also declare workflow_dispatch + qa in on:, without which the qa-gated integration lanes (macOS/Linux-next/Windows) could never fire. Validated on a real macos-15-intel runner: rustls + vendored-openssl green, native-tls soft-fails, overall green.
- shorten the on-trigger explanation and drop speculative runner-future notes - remove editorializing/"Required." tags and redundant feature-set restatement - keep the non-obvious notes (colima RAM, krb5 keg-only, 18456 login race)
…s lane - integration-linux-next referenced a `mssql-2025` docker-compose service that does not exist (compose only defines 2022/2019/2017/azure-sql-edge), so the lane could only ever fail and produced false red signal on qa. Removed until a real SQL Server 2025 image + compose service is added. - The macOS `--features=all` matrix entry pulls in native-tls, which on macOS is Apple Secure Transport and cannot complete the SQL Server TLS handshake; it was a permanent soft-fail. Removed it (rustls and vendored-openssl already provide macOS TLS-backend coverage) and documented why, so the macOS lane is now genuinely green rather than perpetually red.
MattJackson
force-pushed
the
stack/s8
branch
from
September 16, 2026 02:51
0aa6129 to
d874f19
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Part of the sync stack — slice s8, based on #446 (
stack/s7).Restore a real macOS integration lane
Hosted macOS runners have no Linux Docker daemon, so macOS had been reduced to build +
cargo test --lib. This hosts the linux/amd64 SQL Server 2022 container inside a colima (Lima) Linux VM on an Intelmacos-15-intelrunner (native amd64, no qemu) and runs the full integration suite. It also declaresworkflow_dispatch+qainon:, without which the qa-gated integration lanes could never fire.Backends (validated on a real
macos-15-intelrunner)continue-on-errorfor visibility, tracked as a follow-upReadiness gates on an authenticated
SELECT 1(a listening port is not readiness — it caused sporadic18456login races under the bulk tests).Answers @aqrln's macOS follow-up on #438.
Please rebase-merge (not squash) to preserve commit attribution.