Page MenuHomePhabricator

Ensure mjolnir can work on Python 3.9 or later
Closed, ResolvedPublic5 Estimated Story Points

Description

We have to retire our Buster hosts , but search-loader's primary application (mjolnir) is currently configured to use Python 3.7 (the version that ships with Buster).

Bullseye ships with Python 3.9, so that's the minimum version we should be targeting. Creating this ticket to:

  • Update mjolnir to support Python 3.9 or later
  • Verify operation

Details

Related Changes in Gerrit:
Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
search: Update mjolnir to python 3.10repos/data-engineering/airflow-dags!527ebernhardsonwork/ebernhardson/mjolnir-py310main
Update python to 3.10repos/search-platform/mjolnir!7ebernhardsonwork/ebernhardson/libmambamain
Customize query in GitLab

Event Timeline

This needs an updated version of conda, attempts to update the python version currently result in the dependency resolver getting stuck. Will be resolved in T337258

This needs an updated version of conda, attempts to update the python version currently result in the dependency resolver getting stuck. Will be resolved in T337258

The new version of conda-analytics-base is deployed everywhere now, so you should hopefully be unblocked from this. I'm leaving T337258 open whilst I troubleshoot an issue with stat1009, but the libmamba solver is ready to go by default.

btullis@stat1008:~$ source /usr/bin/conda-analytics-activate base
(base) btullis@stat1008:~$ conda info

     active environment : base
    active env location : /opt/conda-analytics
            shell level : 1
       user config file : /home/btullis/.condarc
 populated config files : /opt/conda-analytics/condarc
                          /srv/home/btullis/.conda/condarc
          conda version : 23.7.4
    conda-build version : not installed
         python version : 3.10.12.final.0
       virtual packages : __archspec=1=x86_64
                          __glibc=2.28=0
                          __linux=5.10.0=0
                          __unix=0=0
       base environment : /opt/conda-analytics  (read only)
      conda av data dir : /opt/conda-analytics/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/btullis/.conda/pkgs
                          /usr/lib/anaconda-wmf/pkgs
                          /opt/conda-analytics/pkgs
       envs directories : /home/btullis/.conda/envs
                          /opt/conda-analytics/envs
               platform : linux-64
             user-agent : conda/23.7.4 requests/2.31.0 CPython/3.10.12 Linux/5.10.0-0.deb10.23-amd64 debian/10 glibc/2.28 solver/libmamba conda-libmamba-solver/23.7.0 libmambapy/1.5.1
                UID:GID : 32741:500
             netrc file : None
           offline mode : False


(base) btullis@stat1008:~$
Gehel set the point value for this task to 5.Oct 9 2023, 3:31 PM

Patch looks to work, but we will also want to check the model outputs after deploying the update.

Mentioned in SAL (#wikimedia-operations) [2023-10-24T19:56:36Z] <ebernhardson@deploy2002> Started deploy [airflow-dags/search@c585842]: T346373: Update mjolnir to use python 3.10

Mentioned in SAL (#wikimedia-operations) [2023-10-24T19:57:04Z] <ebernhardson@deploy2002> Finished deploy [airflow-dags/search@c585842]: T346373: Update mjolnir to use python 3.10 (duration: 00m 28s)

Change 973849 had a related patch set uploaded (by Ebernhardson; author: Ebernhardson):

[operations/puppet@production] Source mjolnir deploy repo from gitlab

https://gerrit.wikimedia.org/r/973849

The repo and the airflow side were updated, but we missed updating the search-loader daemons with the new version. Patches above address that issue.

Change 973849 merged by Bking:

[operations/puppet@production] Source mjolnir deploy repo from gitlab

https://gerrit.wikimedia.org/r/973849