-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update to Python 3.12.x (#5736) #5951
base: develop
Are you sure you want to change the base?
Update to Python 3.12.x (#5736) #5951
Conversation
b810ca2
to
c09e7d4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5951 +/- ##
===========================================
- Coverage 85.38% 85.37% -0.01%
===========================================
Files 155 155
Lines 20754 20764 +10
===========================================
+ Hits 17720 17728 +8
- Misses 3034 3036 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
c09e7d4
to
e4c1675
Compare
ed4b5e1
to
858f6fb
Compare
0762297
to
3f7f6ce
Compare
e4c1675
to
3ab9f81
Compare
3ab9f81
to
ca1c534
Compare
c6c31f6
to
9dd4804
Compare
af1fdf3
to
b2bb083
Compare
docker_images.json
Outdated
"docker.io/library/python:3.11.10-slim-bullseye": { | ||
"digest": "sha256:21e2cc29feabc9fa1335feb5efd98ebaf323a5963b8670a2212289b1db74cf73", | ||
"mirror_digest": "sha256:ccb0c6383303002a6d15d2f52a24eb240f46a16f38984b04012a46c6a3b81560", | ||
"docker.io/library/python:3.12.6-slim-bullseye": { |
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.
Looks like you forgot to address #5736 (comment) which is a pretty significant unforced error, since I specifically remember triaging that comment with you to make sure you're aware of it.
d0aeae3
to
4e3b463
Compare
1d3ad1a
to
5fe3df4
Compare
@@ -2319,10 +2319,10 @@ If you add a dependency on a package with native code, you need to build the | |||
wheel manually: | |||
``` | |||
(.venv) ~/workspace/hca/azul$ docker run -it -v ${project_root}/:/root/azul python:3.11.6-bullseye bash | |||
(.venv) ~/workspace/hca/azul$ docker run -it -v ${project_root}/:/root/azul python:3.12.6-slim-bullseye bash |
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.
Fulfills transitive requirement of distutils, which Python 3.12 removed from the standard library
For Python 3.12 support
Resolves error during `make requirements`: × Building wheel for jq (pyproject.toml) did not run successfully. │ exit code: 1 ╰─> [1390 lines of output] running bdist_wheel running build running build_ext ... jq.c:12124:55: error: no member named 'ob_digit' in 'struct _longobject' const digit* digits = ((PyLongObject*)x)->ob_digit; ~~~~~~~~~~~~~~~~~~ ^ jq.c:13608:47: error: no member named 'ob_digit' in 'struct _longobject' const digit* digits = ((PyLongObject*)b)->ob_digit; ~~~~~~~~~~~~~~~~~~ ^ 41 warnings and 7 errors generated. error: command '/usr/bin/clang' failed with exit code 1 [end of output] note: This error originates from a subprocess, and is likely not a problem with pip. ERROR: Failed building wheel for jq
Resolves errors originating from flake8 failing to parse f-strings: E122 continuation line missing indentation or outdented E131 continuation line unaligned for hanging indent E202 whitespace before ')' E202 whitespace before ']' E202 whitespace before '}' E203 whitespace before ':' E221 multiple spaces before operator E222 multiple spaces after operator E225 missing whitespace around operator E231 missing whitespace after ',' E231 missing whitespace after ':' E231 missing whitespace after ';' E271 multiple spaces after keyword E272 multiple spaces before keyword E702 multiple statements on one line (semicolon) E713 test for membership should be 'not in' W604 backticks are deprecated, use 'repr()'
Resolves error during `make requirements_update`: 19.11 Building wheels for collected packages: fastavro 19.11 Building wheel for fastavro (pyproject.toml): started 36.96 Building wheel for fastavro (pyproject.toml): finished with status 'error' 36.97 error: subprocess-exited-with-error 36.97 36.97 × Building wheel for fastavro (pyproject.toml) did not run successfully. 36.97 │ exit code: 1 36.97 ╰─> [73 lines of output] 36.97 <string>:31: SyntaxWarning: invalid escape sequence '\(' 36.97 running bdist_wheel 36.97 running build … 36.97 gcc: internal compiler error: Illegal instruction signal terminated program as 36.97 Please submit a full bug report, 36.97 with preprocessed source if appropriate. 36.97 See <file:///usr/share/doc/gcc-10/README.Bugs> for instructions. 36.97 error: command '/usr/bin/gcc' failed with exit code 4 36.97 [end of output]
Resolves error during test: ERROR: test_openapi (integration_test.OpenAPIIntegrationTest.test_openapi) (component='service') ---------------------------------------------------------------------- Traceback (most recent call last): File "/Users/daniel/repo/azul1/test/integration_test.py", line 1856, in test_openapi validate_spec(spec) File "/Users/daniel/repo/azul1/.venv/lib/python3.12/site-packages/openapi_spec_validator/shortcuts.py", line 17, in validate_spec return validator.validate(spec, spec_url=spec_url) ... AttributeError: '_io.BufferedReader' object has no attribute 'decode'
Resolves DeprecationWarning during schema validation: Traceback (most recent call last): File "/Users/daniel/.pyenv/versions/3.12.6/lib/python3.12/unittest/suite.py", line 306, in _tearDownPreviousClass tearDownClass() AssertionError: ["{message : DeprecationWarning('Automatically retrieving remote references can be a security vulnerability and is discouraged by the JSON Schema specifications. Relying on this behavior is deprecated and will shortly become an error. If you are sure you want to remotely retrieve your reference and that it is safe to do so, you can find instructions for doing so via referencing.Registry in the referencing documentation (https://referencing.readthedocs.org).'), category : 'DeprecationWarning', filename : '/Users/daniel/repo/azul1/.venv/lib/python3.12/site-packages/jsonschema/exceptions.py', lineno : 475, line : None}", ...]
Resolves finding during `make pep8`: F842 local variable 'cs' is annotated but never used
Silences finding during `make pep8`: E122 continuation line missing indentation or outdented
Resolves error during test: AttributeError: 'not_called' is not a valid assertion. Use a spec for the mock if 'not_called' is meant to be an attribute.
5fe3df4
to
92489a0
Compare
Connected issues: #5736
Checklist
Author
develop
issues/<GitHub handle of author>/<issue#>-<slug>
Author (partiality)
p
tag to titles of partial commitspartial
label to PR or this PR completely resolves all connected issuespartial
label1 when the issue title describes a problem, the corresponding PR
title is
Fix:
followed by the issue titleAuthor (reindex, API changes)
r
tag to commit title or this PR does not require reindexingreindex
label to PR or this PR does not require reindexingAPI
or this PR does not modify a REST APIa
(A
) tag to commit title for backwards (in)compatible changes or this PR does not modify a REST APIapp.py
or this PR does not modify a REST APIAuthor (chains)
base
label to the blocking PR or this PR is not chained to another PRchained
label to this PR or this PR is not chained to another PRAuthor (upgrading deployments)
u
tag to commit title or this PR does not require upgrading deploymentsupgrade
label to PR or this PR does not require upgrading deploymentsAuthor (operator tasks)
Author (hotfixes)
F
tag to main commit title or this PR does not include permanent fix for a temporary hotfixprod
branch has no temporary hotfixes for any connected issuesAuthor (before every review)
develop
, squashed old fixupsmake requirements_update
or this PR does not touch requirements*.txt, common.mk, Makefile and DockerfileR
tag to commit title or this PR does not touch requirements*.txtreqs
label to PR or this PR does not touch requirements*.txtmake integration_test
passes in personal deployment or this PR does not touch functionality that could break the ITPeer reviewer (after requesting changes)
Uncheck the Author (before every review) checklists.
Peer reviewer (after approval)
System administrator (after requesting changes)
Uncheck the before every review checklists. Update the
N reviews
label.System administrator (after approval)
demo
orno demo
no demo
no sandbox
N reviews
label is accurateOperator (before pushing merge the commit)
reindex
label andr
commit title tagno demo
upgrade
develop
dev.shared
and ranCI_COMMIT_REF_NAME=develop make -C terraform/shared apply_keep_unused
anvildev.shared
and ranCI_COMMIT_REF_NAME=develop make -C terraform/shared apply_keep_unused
anvilprod.shared
and ranCI_COMMIT_REF_NAME=develop make -C terraform/shared apply_keep_unused
sandbox
label or PR is labeledno sandbox
dev
or PR is labeledno sandbox
anvildev
or PR is labeledno sandbox
anvilprod
or PR is labeledno sandbox
sandbox
deployment or PR is labeledno sandbox
anvilbox
deployment or PR is labeledno sandbox
hammerbox
deployment or PR is labeledno sandbox
sandbox
deployment or PR is labeledno sandbox
anvilbox
deployment or PR is labeledno sandbox
hammerbox
deployment or PR is labeledno sandbox
sandbox
or this PR does not remove catalogs or otherwise causes unreferenced indices indev
anvilbox
or this PR does not remove catalogs or otherwise causes unreferenced indices inanvildev
hammerbox
or this PR does not remove catalogs or otherwise causes unreferenced indices inanvilprod
sandbox
or this PR does not require reindexingdev
anvilbox
or this PR does not require reindexinganvildev
hammerbox
or this PR does not require reindexinganvilprod
sandbox
or this PR does not require reindexingdev
anvilbox
or this PR does not require reindexinganvildev
hammerbox
or this PR does not require reindexinganvilprod
p
if the PR is labeledpartial
Operator (chain shortening)
develop
or this PR is not labeledbase
chained
label from the blocked PR or this PR is not labeledbase
base
base
label from this PR or this PR is not labeledbase
Operator (after pushing the merge commit)
dev
or PR is labeledno sandbox
anvildev
or PR is labeledno sandbox
anvilprod
or PR is labeledno sandbox
dev
1dev
1anvildev
1anvildev
1anvilprod
1anvilprod
1dev.shared
and ranmake -C terraform/shared apply
anvildev.shared
and ranmake -C terraform/shared apply
anvilprod.shared
and ranmake -C terraform/shared apply
dev
anvildev
anvilprod
1 When pushing the merge commit is skipped due to the PR being
labelled
no sandbox
, the next build triggered by a PR whose merge commit ispushed determines this checklist item.
Operator (reindex)
dev
or this PR does not remove catalogs or otherwise causes unreferenced indices indev
anvildev
or this PR does not remove catalogs or otherwise causes unreferenced indices inanvildev
anvilprod
or this PR does not remove catalogs or otherwise causes unreferenced indices inanvilprod
dev
or this PR does not merely remove sources from existing catalogs indev
anvildev
or this PR does not merely remove sources from existing catalogs inanvildev
anvilprod
or this PR does not merely remove sources from existing catalogs inanvilprod
dev
or this PR does not merely add sources to existing catalogs indev
anvildev
or this PR does not merely add sources to existing catalogs inanvildev
anvilprod
or this PR does not merely add sources to existing catalogs inanvilprod
dev
or this PR does not require reindexingdev
anvildev
or this PR does not require reindexinganvildev
anvilprod
or this PR does not require reindexinganvilprod
dev
or this PR does not require reindexingdev
anvildev
or this PR does not require reindexinganvildev
anvilprod
or this PR does not require reindexinganvilprod
dev
deployment or this PR does not require reindexingdev
anvildev
deployment or this PR does not require reindexinganvildev
anvilprod
deployment or this PR does not require reindexinganvilprod
Operator
prod
promotion PR:Shorthand for review comments
L
line is too longW
line wrapping is wrongQ
bad quotesF
other formatting problem