Skip to content

Conversation

@kyuds
Copy link
Collaborator

@kyuds kyuds commented Aug 7, 2025

Currently, quickstart in the docs only shows the yaml file example using the CLI. This PR implements the same quickstart example using the skypilot sdk

Disclaimer: tested the code in the quickstart example without the A100:8 accelerator parameter. Im gpu poor... Code works when using sky.Resources(infra='aws')

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
Copy link
Collaborator Author

kyuds commented Aug 7, 2025

** will update example for the above mentioned PR if said PR gets merged first.

* SSH access from SkyPilot API server host to all remote machines
* All nodes within a SSH Node Pool must have access to port 6443 to its peers (e.g., same VPC).
* Port 6443 doesn't have to be open to machines outside of the network
* All nodes within a SSH Node Pool must have access to port 6443 to its peers (e.g., same VPC). Port 6443 doesn't have to be open to machines outside of the network.
Copy link
Collaborator Author

@kyuds kyuds Aug 7, 2025

Choose a reason for hiding this comment

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

apparently indented bullet points did not work :(... Added this fix here as a side thing


.. code-block:: yaml
.. tab-item:: CLI
:sync: cli
Copy link
Collaborator

Choose a reason for hiding this comment

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

I actually did not know about this :sync:, TIL! Gonna modify some of my older docs with this nifty little trick.

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! verified the python code works on my end as well.

@SeungjinYang SeungjinYang merged commit 5639f08 into skypilot-org:master Aug 8, 2025
15 checks passed
@kyuds kyuds deleted the docs/quickstart-sdk branch August 8, 2025 23:49
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