Introduce breeze doctor

[kunaljubce]

Intends to close #45975.

• Introduce breeze doctor to troubleshoot common issues of breeze and reset the environment.
• Remove --project-name parameter from breeze down

---

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

good start :)

[kunaljubce]

@potiuk Do we need to cleanup_python_generated_files() as part of breeze doctor?

[potiuk]

@potiuk Do we need to cleanup_python_generated_files() as part of breeze doctor?

Yeah. Let's just clean absolutely everything we can. Also we might run fix_ownership_using_docker

[kunaljubce]

@potiuk Do we need to cleanup_python_generated_files() as part of breeze doctor?

Yeah. Let's just clean absolutely everything we can. Also we might run fix_ownership_using_docker

@potiuk Done! Please review the PR and add your feedback/suggestions.

[potiuk]

One small comment only. Possibly we should add "confirm" method call for every step? The way it is implemented now, It always performs all cleanup, but maybe it's enough to only do some of that and get interactive. That can be paired with --answer flag:

• breeze doctor -> interactively asking about each step allowing to y -> do it, n -> skip it, q -> quit the process
• breeze doctor --answer y -> perform complete cleanup

[kunaljubce]

One small comment only. Possibly we should add "confirm" method call for every step? The way it is implemented now, It always performs all cleanup, but maybe it's enough to only do some of that and get interactive. That can be paired with --answer flag:

• breeze doctor -> interactively asking about each step allowing to y -> do it, n -> skip it, q -> quit the process
• breeze doctor --answer y -> perform complete cleanup

@potiuk Thanks for your feedback, have added these options now. Lemme know your thoughts :) Also adding @rawwar in this conversation, to try it out and share feedback.

[kunaljubce]

I see the failures, will check them out shortly 😮‍💨

[rawwar]

Awesome! I am just noticing this PR. Thanks for tagging me @kunaljubce

[kunaljubce]

I see the failures, will check them out shortly 😮‍💨

Seems like the failures are related to the apache-kafka==2.8.1 issue, should be resolved now that PR #47204 is merged. Rebased this feature branch with main and rerunning the tests.

[potiuk]

NICE!