Skip to content

Commit

Permalink
Change cloud implementation feature name
Browse files Browse the repository at this point in the history
  • Loading branch information
ucbstudent committed Dec 10, 2024
1 parent 74bcb7d commit 01dc978
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion sky/clouds/cloud.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ class CloudImplementationFeatures(enum.Enum):
STORAGE_MOUNTING = 'storage_mounting'
HOST_CONTROLLERS = 'host_controllers' # Can run jobs/serve controllers
AUTO_TERMINATE = 'auto_terminate' # Pod/VM can stop or down itself
ENVOY = 'envoy_load_balancer'
HOST_ENVOY_LOAD_BALANCER = 'host_envoy_load_balancer'


class Region(collections.namedtuple('Region', ['name'])):
Expand Down
7 changes: 4 additions & 3 deletions sky/clouds/kubernetes.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,10 @@ class Kubernetes(clouds.Cloud):
'tiers are not '
'supported in '
'Kubernetes.',
clouds.CloudImplementationFeatures.ENVOY: 'Envoy load balancer is not '
'supported on Kubernetes '
'controllers.'
clouds.CloudImplementationFeatures.HOST_ENVOY_LOAD_BALANCER:
'Envoy load balancer is not '
'supported on Kubernetes '
'controllers.'
}

IMAGE_CPU = 'skypilot:custom-cpu-ubuntu-2004'
Expand Down
4 changes: 3 additions & 1 deletion sky/serve/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,9 @@ def up(
if resource.cloud is None:
continue

requested_features = {clouds.CloudImplementationFeatures.ENVOY}
requested_features = {
clouds.CloudImplementationFeatures.HOST_ENVOY_LOAD_BALANCER
}
resource.cloud.check_features_are_supported(
resource, requested_features)

Expand Down

0 comments on commit 01dc978

Please sign in to comment.