Skip to content

Fix autopatch env handling for script-set variables#366

Open
cmssmc00 wants to merge 1 commit into
ovg-project:mainfrom
cmssmc00:fix-autopatch-env-in-script
Open

Fix autopatch env handling for script-set variables#366
cmssmc00 wants to merge 1 commit into
ovg-project:mainfrom
cmssmc00:fix-autopatch-env-in-script

Conversation

@cmssmc00

Copy link
Copy Markdown

Fixes #320.
Related to #316.

Summary:

  • Allow ENABLE_KVCACHED to enable vLLM/SGLang autopatching, in addition to KVCACHED_AUTOPATCH.
  • Keep autopatching disabled by default.
  • Update the disabled-path log message to mention both env vars.
  • Add regression tests for script-set ENABLE_KVCACHED and KVCACHED_AUTOPATCH behavior.

Tests:

  • PYTHONPATH=$PWD python -m pytest tests/test_autopatch_env.py -q

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.

Autopatch fails when ENABLE_KVCACHED is set inside a Python script (requires explicit from kvcached import autopatch)

1 participant