fix(policy): replace deprecated HuggingFace inference endpoint in preset#1454
Conversation
api-inference.huggingface.co returns HTTP 410 Gone — HuggingFace migrated their Inference Providers API to router.huggingface.co. Sandbox agents using the HuggingFace preset for inference would have their requests routed to a dead endpoint. Closes NVIDIA#1453 Signed-off-by: latenighthackathon <latenighthackathon@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe HuggingFace network policy preset updates the inference API endpoint from the deprecated Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
Replace
api-inference.huggingface.cowithrouter.huggingface.coin the HuggingFace policy preset. The old endpoint returns HTTP 410 Gone — HuggingFace migrated their Inference Providers API torouter.huggingface.co.Sandbox agents using the HuggingFace preset for inference would have their requests routed to a dead endpoint.
Testing
Confirm the issue (before the fix)
1. Verify the preset references the deprecated endpoint:
2. Verify the endpoint is dead:
3. Verify the replacement endpoint is live:
Confirm the fix
4. Verify the preset now uses the new endpoint:
5. Verify no reference to the deprecated endpoint remains:
Run the test suite
Closes #1453
Signed-off-by: latenighthackathon latenighthackathon@users.noreply.github.com
Summary by CodeRabbit