Skip to content

Fix cleanup of JupyterCodeExecutor temp output dir#7369

Open
zerone0x wants to merge 1 commit intomicrosoft:mainfrom
zerone0x:orbit/e1e6ddcf
Open

Fix cleanup of JupyterCodeExecutor temp output dir#7369
zerone0x wants to merge 1 commit intomicrosoft:mainfrom
zerone0x:orbit/e1e6ddcf

Conversation

@zerone0x
Copy link

Summary

  • ensure the Jupyter code executor recreates and removes its managed temporary output directory when stopping
  • cover the default path cleanup with a regression test

Testing

  • .venv/bin/python -m pytest python/packages/autogen-ext/tests/code_executors/test_jupyter_code_executor.py -k output_dir

Fixes #7217

@zerone0x
Copy link
Author

@microsoft-github-policy-service agree

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 this pull request may close these issues.

JupyterCodeExecutor leaks temporary output directory on stop — inconsistent with other executors

1 participant