Skip to content

Commit

Permalink
Remove specific reservations
Browse files Browse the repository at this point in the history
  • Loading branch information
Michaelvll committed Aug 20, 2024
1 parent e854233 commit 0271b78
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 20 deletions.
11 changes: 0 additions & 11 deletions docs/source/reference/config.rst
Original file line number Diff line number Diff line change
Expand Up @@ -207,17 +207,6 @@ Available fields and semantics:
#
# Default: false.
prioritize_reservations: false
#
# The targeted capacity reservations to be considered when provisioning
# clusters on AWS. SkyPilot will automatically prioritize this reserved
# capacity (considered as zero cost) if the requested resources matches the
# reservation.
#
# Ref: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/capacity-reservations-launch.html
specific_reservations:
- cr-a1234567
- cr-b2345678
# Identity to use for AWS instances (optional).
#
Expand Down
4 changes: 1 addition & 3 deletions sky/clouds/utils/aws_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,7 @@ class AWSReservation:
def use_reservations() -> bool:
prioritize_reservations = skypilot_config.get_nested(
('aws', 'prioritize_reservations'), False)
specific_reservations = skypilot_config.get_nested(
('aws', 'specific_reservations'), set())
return prioritize_reservations or specific_reservations
return prioritize_reservations


@cachetools.cached(cache=cachetools.TTLCache(maxsize=10,
Expand Down
6 changes: 0 additions & 6 deletions sky/utils/schemas.py
Original file line number Diff line number Diff line change
Expand Up @@ -709,12 +709,6 @@ def get_config_schema():
'prioritize_reservations': {
'type': 'boolean',
},
'specific_reservations': {
'type': 'array',
'items': {
'type': 'string',
},
},
'disk_encrypted': {
'type': 'boolean',
},
Expand Down

0 comments on commit 0271b78

Please sign in to comment.