Skip to content

Conversation

@kyuds
Copy link
Collaborator

@kyuds kyuds commented Aug 10, 2025

Extending #6561 to:

  • sky exec
  • sky stop/down
  • sky jobs launch

NOTE: Because we don't actually expose the managed jobs sdk, I added it to sky __init__.py. Please review if my placement is appropriate, or lmk if there is another official way we import the sdk.

Tested (run the relevant ones):

  • Code formatting: install pre-commit (auto-check on commit) or bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: /smoke-test (CI) or pytest tests/test_smoke.py (local)
  • Relevant individual tests: /smoke-test -k test_name (CI) or pytest tests/test_smoke.py::test_name (local)
  • Backward compatibility: /quicktest-core (CI) or pytest tests/smoke_tests/test_backward_compat.py (local)

@kyuds kyuds requested a review from SeungjinYang August 10, 2025 02:40
@kyuds kyuds changed the title [Docs] Add SDK Examples to QuickStart [Docs] Add SDK Examples to QuickStart (2) Aug 11, 2025
SeungjinYang
SeungjinYang previously approved these changes Aug 11, 2025
Copy link
Collaborator

@SeungjinYang SeungjinYang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kyuds!

@SeungjinYang
Copy link
Collaborator

/quicktest-core

@SeungjinYang
Copy link
Collaborator

/smoke-test

@SeungjinYang SeungjinYang dismissed their stale review August 11, 2025 23:52

slightly worried about quicktest core failure, try rebasing and running quicktest core again

@kyuds
Copy link
Collaborator Author

kyuds commented Aug 12, 2025

/quicktest-core

@kyuds kyuds requested a review from SeungjinYang August 12, 2025 00:35
Copy link
Collaborator

@SeungjinYang SeungjinYang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kyuds!

@kyuds kyuds merged commit fe53873 into skypilot-org:master Aug 12, 2025
16 checks passed
@kyuds kyuds deleted the docs/quickstart-sdk2 branch August 12, 2025 00:43
@SeungjinYang SeungjinYang mentioned this pull request Aug 12, 2025
5 tasks
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.

2 participants