Skip to content

Commit 1d2ed74

Browse files
committed
snowflake-connector-python: fix matrix collapse and cp314t curve test
The bare `include:` python list shared no key with the `version` matrix axis, so each entry merged into every combination in order and the last one (cp314t) silently overwrote matrix.python for every leg -- only cp314t ever built, for every version, since the version-axis refactor (#1937). Restore a real `python:` axis alongside it. That surfaced the actual cp314t (and now cp312/cp313/cp314) failure: test_auth_keypair_ecdsa_unsupported_curve generates a SECP192R1 key as test setup, and our riscv64 cryptography wheel's OpenSSL build has no legacy provider, so cryptography itself raises UnsupportedAlgorithm before the connector's own curve check ever runs. Deselect it.
1 parent b1ffce0 commit 1d2ed74

1 file changed

Lines changed: 11 additions & 2 deletions

File tree

‎.github/workflows/build-snowflake-connector-python.yml‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,11 @@ jobs:
5353
matrix:
5454
version: ${{ fromJSON(needs.setup.outputs.versions) }}
5555
# Per-interpreter (not abi3): the Cython extension links the version-specific ABI.
56+
python: ["cp312", "cp313", "cp314", "cp314t"]
57+
# `include` entries that share no key with another axis merge into every
58+
# combination in order, so without the `python` axis above the last entry
59+
# here (cp314t) silently overwrote matrix.python for every version and the
60+
# cp312/cp313/cp314 legs never ran.
5661
include:
5762
- python: "cp312"
5863
pytest_dist: "-n auto --dist loadfile"
@@ -133,7 +138,10 @@ jobs:
133138
# The first three -k names need a native minicore blob, which upstream
134139
# ships for eight platforms but not riscv64; the last five chmod a path
135140
# and expect the EACCES that root - which cibuildwheel runs the tests
136-
# as - never gets.
141+
# as - never gets. test_auth_keypair_ecdsa_unsupported_curve generates a
142+
# SECP192R1 key as test setup; our riscv64 cryptography wheel's OpenSSL
143+
# build has no legacy provider, so cryptography itself raises
144+
# UnsupportedAlgorithm before the connector's own curve check runs.
137145
CIBW_TEST_COMMAND: >-
138146
python -c "from snowflake.connector.nanoarrow_arrow_iterator import PyArrowRowIterator" &&
139147
pytest -m "unit and not sso and not pandas and not lambda and not aio"
@@ -144,7 +152,8 @@ jobs:
144152
and not test_log_debug_config_file_parent_dir_permissions
145153
and not test_read_only
146154
and not test_config_file_inaccessible_path
147-
and not test_put_error"
155+
and not test_put_error
156+
and not test_auth_keypair_ecdsa_unsupported_curve"
148157
${{ matrix.pytest_dist }}
149158
--ignore {package}/test/unit/aio
150159
--ignore {package}/test/unit/test_ocsp.py

0 commit comments

Comments
 (0)