Skip to content

[Kubernetes] Remove all_includes_in_cluster from skypilot config#9671

Merged
DanielZhangQD merged 3 commits into
masterfrom
hailong/sky-5472-remove-config-field
May 20, 2026
Merged

[Kubernetes] Remove all_includes_in_cluster from skypilot config#9671
DanielZhangQD merged 3 commits into
masterfrom
hailong/sky-5472-remove-config-field

Conversation

@DanielZhangQD

@DanielZhangQD DanielZhangQD commented May 20, 2026

Copy link
Copy Markdown
Collaborator

Remove all_includes_in_cluster from skypilot config.

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)
    Set allowed_contexts: all in SkyPilot config
    • SKYPILOT_ALL_KUBERNETES_CONTEXTS_INCLUDES_IN_CLUSTER is not set, in-cluster is included
    • SKYPILOT_ALL_KUBERNETES_CONTEXTS_INCLUDES_IN_CLUSTER set to false, in-cluster is not included
    • SKYPILOT_ALL_KUBERNETES_CONTEXTS_INCLUDES_IN_CLUSTER set to true, in-cluster is included
  • 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)

gemini-code-assist[bot]

This comment was marked as resolved.

@DanielZhangQD DanielZhangQD requested a review from Michaelvll May 20, 2026 04:01
@DanielZhangQD DanielZhangQD marked this pull request as ready for review May 20, 2026 04:07

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@DanielZhangQD DanielZhangQD merged commit 04e9341 into master May 20, 2026
22 checks passed
@DanielZhangQD DanielZhangQD deleted the hailong/sky-5472-remove-config-field branch May 20, 2026 04:43
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