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

[hnajari]

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

[sahilkukreja]

@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]

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

[msaniscalchi]

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]

@msaniscalchi Thanks for closing the issues and for quick response

[sahilkukreja]

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

[hnajari]

Thank you @sahilkukreja

[msaniscalchi]

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

[msaniscalchi]

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]

Relevant PR has been merged.