Skip to content

[all] Default to python 3.11, drop support for 3.9 - #14918

Merged
hail-ci-robot merged 17 commits into
hail-is:mainfrom
chrisvittal:deps/upgrade-python/3.11
Jul 14, 2025
Merged

hail-ci-robot merged 17 commits into
hail-is:mainfrom
chrisvittal:deps/upgrade-python/3.11

Conversation

@chrisvittal

@chrisvittal chrisvittal commented Jun 12, 2025 •

Copy link
Copy Markdown
Collaborator

This better aligns us with the rest of the scientific python ecosystem as well as ensures we're off 3.9 before end of life.

Fixes #14859

Security Assessment

Delete all except the correct answer:

  • This change potentially impacts the Hail Batch instance as deployed by Broad Institute in GCP

Impact Rating

Delete all except the correct answer:

  • This change has a low security impact

Impact Description

Upgrading the python major version for everything.

Appsec Review

  • Required: The impact has been assessed and approved by appsec

@chrisvittal chrisvittal self-assigned this Jun 24, 2025
chrisvittal added a commit to chrisvittal/hail that referenced this pull request Jun 26, 2025
…hail-is#14927)

In hail-is#14859 and hail-is#14918, I was getting segfaults in libsass, that went away
when I upgraded to ubuntu 24.04. This change separates out the upgrade
as there should be no issues running python3.9 on ubuntu 24.04 until we
upgrade our default version of python.

## Security Assessment

Delete all except the correct answer:
- This change potentially impacts the Hail Batch instance as deployed by
Broad Institute in GCP

### Impact Rating

Delete all except the correct answer:
- This change has a low security impact

### Impact Description
Core OS image upgrade.

### Appsec Review

- [x] Required: The impact has been assessed and approved by appsec
@chrisvittal
chrisvittal force-pushed the deps/upgrade-python/3.11 branch 4 times, most recently from fd9b2d7 to 0f5944a Compare July 1, 2025 19:52
@chrisvittal
chrisvittal requested review from a team and cjllanwarne July 1, 2025 19:53
@chrisvittal
chrisvittal force-pushed the deps/upgrade-python/3.11 branch from 0f5944a to 7614d8a Compare July 1, 2025 19:59
This better aligns us with the rest of the scientific python ecosystem
as well as ensures we're off 3.9 before end of life.
Update ENV directives to use 'VAR=VALUE' syntax as opposed to the no
longer recommended 'VAR VALUE' syntax.

Save the PYTHON_VERSION arg in the hail-ubuntu image to an environment
variable so that other images can inherit the value.
new function as of python3.10 and new ruff lint
There's new lints for these too.
@chrisvittal
chrisvittal force-pushed the deps/upgrade-python/3.11 branch from eaba103 to 3ba39f0 Compare July 2, 2025 20:37
@chrisvittal
chrisvittal force-pushed the deps/upgrade-python/3.11 branch from d1b3235 to dd1bdb6 Compare July 3, 2025 17:46

@ehigham ehigham left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hooray the people will be pleased

@sarahgibs sarahgibs left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved. Thanks for updating python to a version with a longer support timeline.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants