Skip to content

resolve build dependency conflict for urllib3, unify python version to 3.9.13 - #153

Merged
dtcarls merged 1 commit into
dtcarls:masterfrom
pgranieri:master
Sep 11, 2025
Merged

resolve build dependency conflict for urllib3, unify python version to 3.9.13#153
dtcarls merged 1 commit into
dtcarls:masterfrom
pgranieri:master

Conversation

@pgranieri

@pgranieri pgranieri commented Sep 11, 2025

Copy link
Copy Markdown
Contributor

Problem:

  • The espn_api dependency list is conflicting, and causes a build failure when trying to compose the docker image.
  • The python version is set to different versions across the repo
  • The docker compose file uses a deprecated version tag
[...]
    > [4/4] RUN python setup.py install:
2.777 Searching for certifi>=2017.4.17
2.777 Reading https://pypi.org/simple/certifi/
2.858 Downloading https://files.pythonhosted.org/packages/e5/48/1549795ba7742c948d2ad169c1c8cdbae65bc450d6cd753d124b17c8cd32/certifi-2025.8.3-py3-none-any.whl#sha256=f6c12493cfb1b06ba2ff328595af9350c65d6644968e5d3a2ffd78699af217a5
2.899 Best match: certifi 2025.8.3
2.899 Processing certifi-2025.8.3-py3-none-any.whl
2.901 Installing certifi-2025.8.3-py3-none-any.whl to /usr/local/lib/python3.9/site-packages
2.909 Adding certifi 2025.8.3 to easy-install.pth file
2.911
2.911 Installed /usr/local/lib/python3.9/site-packages/certifi-2025.8.3-py3.9.egg
2.911 error: urllib3 2.5.0 is installed but urllib3<=2.2.3 is required by {'espn-api'}
[...]

Solution:

  • set the urllib3 dependency to be explicitly <=2.2.3, the same version expected by espn_api currently.
  • update the python version to be explicitly 3.9.13 across the repo.
  • remove deprecated version tag from the docker compose file

Testing: was able to successfully run docker compose up -d using the new config, and the github test action workflows passed in my fork on push

$ docker compose up -d --build
[...]
#9 2.742 Finished processing dependencies for gamedaybot==0.3.1
#9 DONE 2.9s

#10 exporting to image
#10 exporting layers
#10 exporting layers 0.2s done
#10 writing image sha256:17f3b228560ea4f93b84eccf5202557103363e9ef0def9bbab6df8b33e726c1a done
#10 naming to docker.io/library/pgranieri_ffcb-gamedaybot done
#10 DONE 0.2s

#11 resolving provenance for metadata file
#11 DONE 0.0s
[+] Running 2/2
 ✔ pgranieri_ffcb-gamedaybot              Built                                                                                                                                        0.0s
 ✔ Container pgranieri_ffcb-gamedaybot-1  Started

Problem: the espn_api dependency list is conflicting, and causes a build
failure when trying to compose the docker image. Additionally, the
python version is set to different versions across the repo
```
[...]
    > [4/4] RUN python setup.py install:
2.777 Searching for certifi>=2017.4.17
2.777 Reading https://pypi.org/simple/certifi/
2.858 Downloading https://files.pythonhosted.org/packages/e5/48/1549795ba7742c948d2ad169c1c8cdbae65bc450d6cd753d124b17c8cd32/certifi-2025.8.3-py3-none-any.whl#sha256=f6c12493cfb1b06ba2ff328595af9350c65d6644968e5d3a2ffd78699af217a5
2.899 Best match: certifi 2025.8.3
2.899 Processing certifi-2025.8.3-py3-none-any.whl
2.901 Installing certifi-2025.8.3-py3-none-any.whl to /usr/local/lib/python3.9/site-packages
2.909 Adding certifi 2025.8.3 to easy-install.pth file
2.911
2.911 Installed /usr/local/lib/python3.9/site-packages/certifi-2025.8.3-py3.9.egg
2.911 error: urllib3 2.5.0 is installed but urllib3<=2.2.3 is required by {'espn-api'}
[...]
```

Solution: set the urllib3 dependency to be explicitly <=2.2.3, the same
version expected by espn_api currently. Additionally, updated the python
version to be explicitly 3.9.13 across the repo. additionally removed
deprecated version tag from the docker compose file

Testing: was able to successfully run `docker compose up -d` using the
new config
```

[+] Running 2/2
    ✔ fantasy_football_chat_bot-gamedaybot              Built                                                                                                                             0.0s
    ✔ Container fantasy_football_chat_bot-gamedaybot-1  Started
```
@dtcarls

dtcarls commented Sep 11, 2025

Copy link
Copy Markdown
Owner

Thank you for the PR. Looks good.

@dtcarls
dtcarls merged commit d2e5ef1 into dtcarls:master Sep 11, 2025
2 checks passed
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.

2 participants