failed to install pypi packages. googleads has requirement pyyaml <6.0, >=5.1, but you have pyyaml 6.0

Question

failed to install pypi packages. googleads has requirement pyyaml <6.0, >=5.1, but you have pyyaml 6.0

hnajari opened this issue 2 years ago · comments

Describe the bug:
Hi,
I got this error

Steps to Reproduce:
I am using googleads with apache airflow package.
I am trying to upgrade airflow package from 2.2.5 to 2.3.3 version but i got this error

failed to install pypi packages. googleads has requirement pyyaml <6.0, >=5.1, but you have pyyaml 6.0

Expected behavior:

Client library version and API version:
Client library version:
Google Ads API version: 33.0.0

Request/Response Logs:

Anything else we should know about your project / environment:

sahilkukreja · Answer 1 · Sun Oct 16 2022 20:35:06 GMT+0800 (China Standard Time)

@hnajari I am also facing the same issues i have raised the PR to fix the dependency issues

hnajari · Answer 2 · Sun Oct 16 2022 22:13:53 GMT+0800 (China Standard Time)

Thank you @sahilkukreja

msaniscalchi · Answer 3 · Mon Oct 17 2022 22:56:00 GMT+0800 (China Standard Time)

Thanks for the report! I'll investigate this along with the relevant PR shortly.

msaniscalchi · Answer 4 · Mon Oct 17 2022 23:58:36 GMT+0800 (China Standard Time)

I can't reproduce the exact issue on my end–if apache-airflow is installed first, attempting to install googleads afterwards causes the PyYaml installation to be downgraded. This is still workable because the PyYaml dependency for apache-airflow is defined as PyYAML<7,>=5.1. Installing in the opposite order also works as intended because PyYaml 5.4.1 is within the range of versions supported by apache-airflow. I'm not certain why your installation fails in comparison.

Of course, it makes sense to upgrade to 6.0 now that it is available. Given that this is a major version bump for the dependency, I want to run this through some test cases and verify that we won't introduce new breakages before adopting it.

msaniscalchi · Answer 5 · Tue Oct 18 2022 03:07:45 GMT+0800 (China Standard Time)

Relevant PR has been merged.

msaniscalchi · Answer 6 · Tue Oct 18 2022 03:10:06 GMT+0800 (China Standard Time)

Actually, we'll still have to push this to PyPI. I'll close this when that has been updated.

sahilkukreja · Answer 7 · Tue Oct 18 2022 14:31:42 GMT+0800 (China Standard Time)

@msaniscalchi Thanks for merging the PR , by when can we expect PyPI Release ?
FYI. Steps to reproduce the issue
upgrade composer environment to 2.0.28 - Airflow Version 2.3.3
and Then try to install googleads
Composer-versions if you check for Version composer-2.0.28-airflow-2.3.3 in PyPI packages for Python 3 we have PyYAML==6.0 and we cant downgrade pyYAML as its used in other packages

msaniscalchi · Answer 8 · Tue Oct 18 2022 23:02:16 GMT+0800 (China Standard Time)

Thanks for merging the PR , by when can we expect PyPI Release ?

It is dependent on a separate internal review that is ongoing. Given that it is not a complex change, I don't anticipate this should take long. It could happen as early as today, but could take up to a few days if our reviewer can't prioritize it in the short term.

sahilkukreja · Answer 9 · Wed Oct 19 2022 05:02:41 GMT+0800 (China Standard Time)

@msaniscalchi Thanks for closing the issues and for quick response