merge from upstream#1
Merged
Merged
Conversation
- Port 5000/tcp missing: log full ports dict, list possible causes (firewall, direct_port_count too low), and suggest check command - 120s no-response timeout: distinguish between port never reachable (startup/port failure) vs connection lost mid-test (crash/OOM) - Add actionable suggestions for each failure case - Log polling URL at start so users can manually curl to debug - Enrich Error_testresults.log with port, IP, and connection state - Fix misleading "crashed during stress test" message when server never responded at all (stress test never ran) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Improve self-test diagnostics on port timeout and mapping failures
Fix/dep issues
added new show instances-v1 command to get instances with pagination
…343) * docs: add upcoming breaking change notice to show instances endpoint
* add missing transfer credit arg to create team
Function to hit update_worker on the autoscaler
Added inactivity timeout to endpoint
Add Deployments CLI
consolidate SDK and CLI into unified vastai package
#352) make serverless code needs serverless reviewers,
make user confirm instance deletion
docs: Add SKILL.md for Claude Code and AI agent integration
* feat: implement request tracking for execution time metrics * fix: update tests to reflect new behavior
* fix create instance raw output * fix: show instances v1 returns empty data with raw arg
* delete search invoices, add show charges, update show invoices
* fix cli crashing on windows, add OS smoketest
…ration (#365) feat(CLN-3107): implement accept price increase command and API integration Co-authored-by: Abiola Akinnubi <abiola.akinnubi@gmail.com>
* add metrics cli commands for hosts
…tes (#370) adds timeouts, error handling, retry logic, random bugfixes, doc updates
Many of the fixes in the recent CLI+SDK unification PR were originally written by @begna112 on his pr/cli-fixes branch. The squash commit omitted the Co-Authored-By trailer; this empty commit records the attribution. Co-Authored-By: begna112 <begna112@gmail.com>
remove benchmark command, rm auto pip package updater
fixed cli so that expired tfa session keys are handled when they return the 404 error
… Quirks (#364) * docs: add agent entry points — CLAUDE.md, README AI Agents section, --help pointer - CLAUDE.md: routes agents to vastai/SKILL.md (CLI) and vastai_sdk/SKILL.md (SDK) - README.md: add ## AI Agents section with npx skills install command - vast.py: extend root --help epilog with link to vastai/SKILL.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: use raw GitHub URL for SKILL.md in --help epilog Raw URL returns plain markdown directly — better for agents fetching it programmatically. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: add agent quick-start to OpenAPI spec info.description Replaces generic API intro with agent-oriented content: - npx skills install command + SKILL.md links - Auth instructions - Key API quirks (gpu_ram units, instance response format, user ID pattern, SSH timing, onstart limit) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * openapi: update Key Quirks — replace instance/user-id quirks with new_contract and poll trap Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: fix API key URL in CLI SKILL.md — update to /manage-keys/ Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: add SKILL.md pointer to vastai/cli/main.py epilog Addresses review comment on PR #364 — the change belongs in vastai/cli/main.py (canonical location) not just vast.py (deprecated). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Removes the `wget` + `chmod +x` installation path from the CLI skill docs. The agent-trust-hub security audit flagged fetching and executing remote scripts as a high-risk pattern. PyPI (`pip install vastai`) is the recommended and only supported install method for agent use. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
security audit fix: remove wget install from SKILL.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
12 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.