Skip to content
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

ci: Fix lint errors due to mypy with python 3.12 #1890

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

favilo
Copy link
Contributor

@favilo favilo commented Nov 13, 2024

Now that we're using python 3.12 for the lint github action, we need to fix the errors that it is reporting.

Lint errors in question

Now that we're using python 3.12 for the `lint` github action, we need
to fix the errors that it is reporting.
@favilo favilo added the :ci Everything related to CI label Nov 13, 2024
@gbanasiak
Copy link
Contributor

This is caused by aiohttp update, which is a transitive/unpinned dependency in Rally.

After local venv recreation:

-aiohttp==3.10.8
+aiohttp==3.11.0

It's either/or it seems. Either we pin aiohttp and previous version is OK, or we let aiohttp move and then need to remove mypy ignore. WDYT @favilo ?

@favilo
Copy link
Contributor Author

favilo commented Nov 13, 2024

How about we pin to the current version. Then we can remove the ignore and have an updated dependency, without being forced to upgrade from an unpinned one.

Also reorganize the transitive dependencies in pyproject.toml
"elasticsearch[async]==8.6.1",
"elastic-transport==8.4.1",
# License: MIT
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've also moved the license comment to here.

@favilo
Copy link
Contributor Author

favilo commented Nov 14, 2024

Ah, so currently Python 3.8 isn't supported. So I'll pin it to 3.10.8. But I'm going to take this opportunity to just remove python 3.8 support in the near future. I've already got it working with 3.12 just fine.

That will allow us to do a lot more of the improvements we were interested in doing, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ci Everything related to CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants