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

update: added google-cloud-storage v2.18.2 support #594

Merged
merged 3 commits into from
Aug 27, 2024

Conversation

CagriYonca
Copy link
Contributor

  • added google-cloud-storage v2.18.2 support for python 3.8, 3.9 and 3.10.

Python 3.11 and 3.12 has the dependency conflict below:
image

@CagriYonca CagriYonca added this to the H2-2024 milestone Aug 22, 2024
@CagriYonca CagriYonca requested review from pvital and GSVarsha August 22, 2024 16:47
@CagriYonca CagriYonca self-assigned this Aug 22, 2024
Signed-off-by: Cagri Yonca <cagriyonca@gmail.com>
@CagriYonca CagriYonca force-pushed the module_version_update branch from 29dcf9e to b6b3b31 Compare August 22, 2024 16:57
Copy link
Member

@pvital pvital left a comment

Choose a reason for hiding this comment

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

Python 3.8 will be declared EOL in about two months when 3.13 is released. I prefer to find a way to also support it on 3.11 and 3.12 since these will be used more and more for stable apps.

What happens if we specify on the requirements.txt file for 3.11 and 3.12 the google-api-core >= 2.15.0?

@CagriYonca CagriYonca force-pushed the module_version_update branch 3 times, most recently from f27b782 to 23a25f2 Compare August 23, 2024 10:55
… for python 3.11 and 3.12

Signed-off-by: Cagri Yonca <cagriyonca@gmail.com>
@CagriYonca CagriYonca force-pushed the module_version_update branch from 23a25f2 to 5e34e95 Compare August 23, 2024 10:58
@pvital
Copy link
Member

pvital commented Aug 23, 2024

Wonderful @CagriYonca!

Please change the following in addition:

  1. remove any entry of the Google Cloud packages from the requirements file for Python 3.11, 3.12, and 3.13.
  2. add an extra job for Python 3.13.
  3. update the pipelines for Tekton as well.

@CagriYonca CagriYonca force-pushed the module_version_update branch 3 times, most recently from 9118791 to b7dcbb0 Compare August 26, 2024 07:04
@CagriYonca
Copy link
Contributor Author

Wonderful @CagriYonca!

Please change the following in addition:

  1. remove any entry of the Google Cloud packages from the requirements file for Python 3.11, 3.12, and 3.13.
  2. add an extra job for Python 3.13.
  3. update the pipelines for Tekton as well.
  • Removed Google Cloud packages from reqs file for 3.11, 3.12 and 3.13.
  • Python 3.13 is already not supported due to grpcio dependency.
  • Tekton pipeline is updated.

@CagriYonca CagriYonca force-pushed the module_version_update branch 4 times, most recently from fdf4e6b to 0284776 Compare August 26, 2024 08:28
Signed-off-by: Cagri Yonca <cagriyonca@gmail.com>
@CagriYonca CagriYonca force-pushed the module_version_update branch from 0284776 to ccde04b Compare August 26, 2024 11:02
@CagriYonca CagriYonca requested a review from pvital August 26, 2024 13:59
Copy link
Member

@pvital pvital left a comment

Choose a reason for hiding this comment

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

It looks good to me.

@CagriYonca CagriYonca merged commit 22098d6 into master Aug 27, 2024
14 of 15 checks passed
@CagriYonca CagriYonca deleted the module_version_update branch August 27, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants