assume monkeypatch.chdir does not work in multiprocess spawn

[mattseddon]

No description provided.

[cloudflare-workers-and-pages]

Deploying datachain-documentation with    Cloudflare Pages

Latest commit:	a55bf5a
Status:	✅  Deploy successful!
Preview URL:	https://a6ff053b.datachain-documentation.pages.dev
Branch Preview URL:	https://patch-spawn-path.datachain-documentation.pages.dev

View logs

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.76%. Comparing base (86fc806) to head (a55bf5a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #903   +/-   ##
=======================================
  Coverage   87.76%   87.76%           
=======================================
  Files         130      130           
  Lines       11639    11641    +2     
  Branches     1579     1579           
=======================================
+ Hits        10215    10217    +2     
  Misses       1029     1029           
  Partials      395      395           

Flag	Coverage Δ
datachain	87.69% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[skshetry]

That test only started failing since yesterday, so maybe some dependency got updated?

[skshetry]

It's also interesting that the test only fails on Python 3.12.

[skshetry]

I am able to reproduce this with:

nox -s tests-3.12 -- tests/func/test_query.py::test_shutdown_on_sigterm

[skshetry]

datamodel-code-generator had a new release (0.27.0) that is now bringing more dependencies, including anyio (which comes from httpx). anyio is what is causing issues, but idk what.
I'll try looking into this on Monday.

We can pin to datamodel-code-generator<0.27 temporarily.

koxudaxi/datamodel-code-generator#2288 (comment)

[shcheklein]

@skshetry

We can pin to datamodel-code-generator<0.27 temporarily.

sounds good, thanks

[skshetry]

That has been fixed upstream, so we don't need to pin it. But I do need to look why it hangs with anyio.