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

Error running breeze setup regenerate-command-images #30465

Closed
rohan472000 opened this issue Apr 4, 2023 · 5 comments · Fixed by #30464
Closed

Error running breeze setup regenerate-command-images #30465

rohan472000 opened this issue Apr 4, 2023 · 5 comments · Fixed by #30464

Comments

@rohan472000
Copy link
Contributor

          after running `pre-commit run` , it asked me to run `breeze setup regenerate-command-images` but while running it I got below errors

PS C:\pycharm\codes\airflow-repo-clone\airflow> breeze setup regenerate-command-images
Traceback (most recent call last):
File "C:\pycharm\codes\airflow-repo-clone\airflow\dev\breeze\src\airflow_breeze\commands\main_command.py", line 122, in check_for_python_emulation
system_machine = subprocess.check_output(["uname", "-m"], text=True).strip()
File "C:\python3.10\setup\lib\subprocess.py", line 420, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "C:\python3.10\setup\lib\subprocess.py", line 501, in run
with Popen(*popenargs, **kwargs) as process:
File "C:\python3.10\setup\lib\subprocess.py", line 966, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\python3.10\setup\lib\subprocess.py", line 1435, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "C:\python3.10\setup\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\python3.10\setup\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "c:\users\anand.local\bin\breeze.exe_main
.py", line 7, in
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\core.py", line 1130, in call
return self.main(*args, **kwargs)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\rich_click\rich_group.py", line 21, in main
rv = super().main(*args, standalone_mode=False, **kwargs)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\core.py", line 1055, in main
rv = self.invoke(ctx)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\core.py", line 1654, in invoke
super().invoke(ctx)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "C:\Users\anand.local\pipx\venvs\apache-airflow-breeze\lib\site-packages\click\decorators.py", line 26, in new_func
return f(get_current_context(), *args, **kwargs)
File "C:\pycharm\codes\airflow-repo-clone\airflow\dev\breeze\src\airflow_breeze\commands\main_command.py", line 114, in main
check_for_python_emulation()
File "C:\pycharm\codes\airflow-repo-clone\airflow\dev\breeze\src\airflow_breeze\commands\main_command.py", line 146, in check_for_python_emulation
except TimeoutOccurred:
UnboundLocalError: local variable 'TimeoutOccurred' referenced before assignment

Originally posted by @rohan472000 in #30405 (comment)

@boring-cyborg
Copy link

boring-cyborg bot commented Apr 4, 2023

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

potiuk added a commit to potiuk/airflow that referenced this issue Apr 4, 2023
When breeze is run on Windows it fails with FileNotFoundException
when running uname during emulation check. This is now fixed alongside
fixing TimeoutError misplacement - after moving it to local import,
an exception triggered before importing it causes UnboundLocaalError.

Related: apache#30405 (comment)

Fixes: apache#30465
@potiuk
Copy link
Member

potiuk commented Apr 4, 2023

Fix in #30464 . BTW, there will likely be more errors like that. Windows is our target (but not fully supported yet) env, so if you can report (or maybe even fix and make PRs) similar errors for windows, that would be cool. Fixing is best of course @rohan472000

@rohan472000
Copy link
Contributor Author

Sure @potiuk I will try that definitely.

potiuk added a commit that referenced this issue Apr 4, 2023
When breeze is run on Windows it fails with FileNotFoundException
when running uname during emulation check. This is now fixed alongside
fixing TimeoutError misplacement - after moving it to local import,
an exception triggered before importing it causes UnboundLocaalError.

Related: #30405 (comment)

Fixes: #30465
@potiuk
Copy link
Member

potiuk commented Apr 5, 2023

Fix has been merged. It should work now natively

@rohan472000
Copy link
Contributor Author

@potiuk well i m using wsl2, but I encountered some problems in Ubuntu distro whereas Ubuntu 22.0.4 is working fine with commands, also after any installation or any changes, I need to restart my terminal. Docs doesn't cover every step to install packages which I will try to add into my next PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants