Skip to content

Comments

Mark azure sdist warning as knowing when installing#27258

Merged
jedcunningham merged 1 commit intoapache:mainfrom
potiuk:fix-warning-when-sdist-installing
Oct 25, 2022
Merged

Mark azure sdist warning as knowing when installing#27258
jedcunningham merged 1 commit intoapache:mainfrom
potiuk:fix-warning-when-sdist-installing

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Oct 25, 2022

When installing azure as sdist, there is a warning emitted about urllib pyopenssl deprecation. This warning should be fixed in the future versions of azure libraries, but for now we should mark it as "known" warning.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

When installing azure as sdist, there is a warning emitted about
urllib pyopenssl deprecation. This warning should be fixed in
the future versions of azure libraries, but for now we should
mark it as "known" warning.
@potiuk
Copy link
Member Author

potiuk commented Oct 25, 2022

cc: @uranusjr -> I guess this one cannot be easily fixed in your quest of removing warnings (and anyhow it only seems to be printed when sdist-installing azure provider (see some of the recent main failures of "sdist provider package installing" jobs).

@potiuk
Copy link
Member Author

potiuk commented Oct 25, 2022

(Actually it's also generated when installing it from wheels).

@jedcunningham jedcunningham merged commit 3938b28 into apache:main Oct 25, 2022
@potiuk potiuk deleted the fix-warning-when-sdist-installing branch December 2, 2022 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants