Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Sep 22, 2023


^ 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.

@potiuk
Copy link
Member Author

potiuk commented Sep 22, 2023

I bring back the change from @uranusjr that I reverted because it was failing main (I guess it was because of older docker version - but running it in apache repo to be sure

@potiuk
Copy link
Member Author

potiuk commented Sep 22, 2023

Nope. It turned out that docker context ls --format=json returns just dictionary of context rather than 1-element array of dictionaries.

@hussein-awala hussein-awala merged commit 2e764fb into main Sep 22, 2023
@Taragolis Taragolis deleted the avoid-wsl2-desktop-linux-context branch September 26, 2023 16:01
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Oct 5, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.2 milestone Oct 5, 2023
ephraimbuddy pushed a commit that referenced this pull request Oct 5, 2023
* Avoid WSL2 ones when finding a context for Breeze

* fixup! Avoid WSL2 ones when finding a context for Breeze

---------

Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
(cherry picked from commit 2e764fb)
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.

5 participants