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

Remove misleading message from CI #24650

Merged
merged 1 commit into from
Jun 25, 2022

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jun 25, 2022

The CI jobs do not usually have mypy_cache volume created and
almost all jobs print misleading "mypy_cache_volume" error at
the beginning.

This is a noise - we are not interested in this stderr printed
message - we are only interested in returncode from checking
if it exists.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 newsfragement file, named {pr_number}.significant.rst, in newsfragments.

The CI jobs do not usually have mypy_cache volume created and
almost all jobs print misleading "mypy_cache_volume" error at
the beginning.

This is a noise - we are not interested in this stderr printed
message - we are only interested in returncode from checking
if it exists.
@potiuk potiuk force-pushed the remove-misleading-volume-missing-error branch from a61593b to bb4a4a9 Compare June 25, 2022 08:33
@potiuk
Copy link
Member Author

potiuk commented Jun 25, 2022

Gets rid of those:

Screenshot 2022-06-25 at 10 33 22

@potiuk
Copy link
Member Author

potiuk commented Jun 25, 2022

Same here "selective checks" failing is fixed.

@potiuk potiuk merged commit ac10402 into apache:main Jun 25, 2022
@potiuk potiuk deleted the remove-misleading-volume-missing-error branch June 25, 2022 09:27
potiuk added a commit to potiuk/airflow that referenced this pull request Jun 29, 2022
The CI jobs do not usually have mypy_cache volume created and
almost all jobs print misleading "mypy_cache_volume" error at
the beginning.

This is a noise - we are not interested in this stderr printed
message - we are only interested in returncode from checking
if it exists.

(cherry picked from commit ac10402)
@ephraimbuddy ephraimbuddy added this to the Airflow 2.3.3 milestone Jun 30, 2022
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants