Skip to content

Conversation

@zpoint
Copy link
Collaborator

@zpoint zpoint commented Aug 28, 2025

  • Skip tests that require server restart or changing USER_ENV, which is not supported via API endpoints and service accounts token(The user will fixed if we use that, USER_ENV won't work then)
  • Change some GPU types
  • Still working on it, PR for sync and convenience tests.

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)

@zpoint zpoint marked this pull request as draft August 28, 2025 07:27
@zpoint
Copy link
Collaborator Author

zpoint commented Aug 28, 2025

/smoke-test --kubernetes --env-file s3://buildkite-configfile-test/config.yaml

* only enable aws for remote api server in test_docker_storage_mounts

* fix is_remote_server_test()
@zpoint zpoint requested a review from kevinmingtarja August 29, 2025 02:22
@zpoint
Copy link
Collaborator Author

zpoint commented Aug 29, 2025

More context from @kevinmingtarja zpoint#82 (comment)

@zpoint zpoint marked this pull request as ready for review August 29, 2025 02:22
@kevinmingtarja
Copy link
Collaborator

/smoke-test --kubernetes --env-file s3://buildkite-configfile-test/config.yaml

@zpoint
Copy link
Collaborator Author

zpoint commented Aug 29, 2025

/smoke-test --kubernetes --env-file s3://buildkite-configfile-test/config.yaml

Copy link
Collaborator

@kevinmingtarja kevinmingtarja left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @zpoint! This gets us to 64/93 tests 🟢. I'll follow up with another PR soon.

@kevinmingtarja kevinmingtarja merged commit e56647b into skypilot-org:master Aug 29, 2025
16 of 17 checks passed
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