-
Notifications
You must be signed in to change notification settings - Fork 536
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[k8s] Disable autostop for controller on kubernetes #3521
Merged
Merged
Changes from all commits
Commits
Show all changes
96 commits
Select commit
Hold shift + click to select a range
34598a2
playing around
romilbhardwaj 95b1eb0
wip with hacks
romilbhardwaj f93e6dd
wip refactor get_endpoints
romilbhardwaj c8276bb
working get_endpoints
romilbhardwaj c713293
wip
romilbhardwaj aa286e8
fixed circular import
romilbhardwaj 9f768a2
Working for ingress and loadbalancer svc
romilbhardwaj 1848b3a
lint
romilbhardwaj ba0ff60
add purging from #3094
romilbhardwaj 13f7241
Use local catalog on the controller too
romilbhardwaj 996742a
use externalip if available
romilbhardwaj 7c8b5cd
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj b222186
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj d852c61
add dshm_size_limit
romilbhardwaj b9014df
optimize dependency installation
romilbhardwaj 282bcef
Add todo
romilbhardwaj d10fb24
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj 3cb948f
optimize ingress
romilbhardwaj 4a7e10a
fix
romilbhardwaj 9e0d910
fix
romilbhardwaj 9e51793
Merge branch 'k8s_ingress_optimization' of https://github.com/skypilo…
romilbhardwaj 68bb6e8
remove autostop timing
romilbhardwaj b72a814
Fix URLs for raw IP:ports
romilbhardwaj a22b6d3
fixes
romilbhardwaj 4637f91
wip
romilbhardwaj d27ae5f
SA wip
romilbhardwaj cdc2e3f
Allow use of service accounts through remote_identity field
romilbhardwaj 643fb77
Make purge work for no clusters in kubeconfig
romilbhardwaj 7649fcb
Handle ingress namespace not present
romilbhardwaj 8c76eb6
setup optimizations and critical SA key fix
romilbhardwaj 2e20600
fix docs
romilbhardwaj de6a99c
fix docs
romilbhardwaj 03e44eb
Add support for skypilot.co/external-ip annotation for ingress
romilbhardwaj 7436a7e
Remove dshm_size_limit
romilbhardwaj 3cc5352
Undo kind changes
romilbhardwaj d836f17
Update service account docs
romilbhardwaj 1b559d2
minor docs
romilbhardwaj cda27b2
update comment
romilbhardwaj 23d730b
is_same_cloud to cloud_in_list
romilbhardwaj cba92e3
refactor query_ports to use head_ip
romilbhardwaj 789cefe
autodown + http prefixing in callers
romilbhardwaj 55b63de
fix ssh key issues when user hash is reused
romilbhardwaj 73ad2e6
linting
romilbhardwaj 31418d2
lint
romilbhardwaj 0684f2b
lint, HOST_CONTROLLERS
romilbhardwaj e7f232e
add serve smoke tests for k8s
romilbhardwaj 2a1b916
disallow file_mounts and workdir if no storage cloud is enabled
romilbhardwaj f242037
minor
romilbhardwaj 9b12cf1
lint
romilbhardwaj 87330bd
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj 3ca5d47
update fastchat to use --host 127.0.0.1
romilbhardwaj 02ef50a
extend timeout
romilbhardwaj 259917f
docs comments
romilbhardwaj 8066a2a
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj 665a12e
rename to port
romilbhardwaj fcc63fe
add to core.py
romilbhardwaj df21a0b
docstrs
romilbhardwaj 5d8c9c7
add docs on exec based auth
romilbhardwaj 937feb6
expand elif
romilbhardwaj 33dfb01
add lb comment
romilbhardwaj 87c77df
refactor
romilbhardwaj 99547db
refactor
romilbhardwaj 3d24b41
fix docs build
romilbhardwaj 750d4d4
add PODIP mode support
romilbhardwaj 87d4d25
make ssh services optional
romilbhardwaj a31a3bc
nits
romilbhardwaj e3bb4d7
Revert "make ssh services optional"
romilbhardwaj 12096fd
Revert "add PODIP mode support"
romilbhardwaj c33572e
nits
romilbhardwaj 803df4a
use 0.0.0.0 when on k8s; use common impl for other clouds
romilbhardwaj a0ba0d1
return dict instead of raising errors in core.endpoints()
romilbhardwaj fb88218
lint
romilbhardwaj 1d968a6
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj e22908a
merge fixes
romilbhardwaj 34a719c
merge fixes
romilbhardwaj 017dcb5
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj cd08d0f
merge fixes
romilbhardwaj 05f1996
lint
romilbhardwaj 97b1d56
fix smoke tests
romilbhardwaj 9a8ccd1
fix smoke tests
romilbhardwaj 3b7d33f
comment
romilbhardwaj e029ebc
add enum for remote identity
romilbhardwaj 7b201f2
lint
romilbhardwaj f31553b
disable autostop for kubernetes
romilbhardwaj 8a54faf
add skip_status_check
romilbhardwaj 5fb4232
remove zone requirement
romilbhardwaj 2e8afe9
fix timings for test
romilbhardwaj 29b05aa
Merge branch 'serve_k8s_final' of https://github.com/skypilot-org/sky…
romilbhardwaj 736e087
silence curl download
romilbhardwaj e3768b4
move jq from yaml to test_minimal
romilbhardwaj 26cc380
move jq from yaml to test_minimal
romilbhardwaj 3b7b935
add assert
romilbhardwaj 7658964
lint
romilbhardwaj f2b9d48
Merge branch 'serve_k8s_final' of https://github.com/skypilot-org/sky…
romilbhardwaj 4e3a3cd
lint
romilbhardwaj 9c09025
Merge branch 'master' of https://github.com/skypilot-org/skypilot int…
romilbhardwaj File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: would be nice to add
()