Skip to content

Commit

Permalink
[Core][BugFix] Remove extra field _require_fuse in `Resources.to_ya…
Browse files Browse the repository at this point in the history
…ml_config()` (#3457)

fix
  • Loading branch information
cblmemo authored Apr 22, 2024
1 parent 118fc79 commit 1bdcd01
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions sky/resources.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,7 @@ def __init__(

self._docker_login_config = _docker_login_config

self._requires_fuse = (_requires_fuse
if _requires_fuse is not None else False)
self._requires_fuse = _requires_fuse

self._set_cpus(cpus)
self._set_memory(memory)
Expand Down Expand Up @@ -422,7 +421,9 @@ def is_image_managed(self) -> Optional[bool]:
return self._is_image_managed

@property
def requires_fuse(self) -> Optional[bool]:
def requires_fuse(self) -> bool:
if self._requires_fuse is None:
return False
return self._requires_fuse

@requires_fuse.setter
Expand Down

0 comments on commit 1bdcd01

Please sign in to comment.