Skip to content

Commit

Permalink
Fixes failing test related to TaskNodeOverrides (#2230)
Browse files Browse the repository at this point in the history
Signed-off-by: Thomas J. Fan <[email protected]>
Signed-off-by: Jan Fiedler <[email protected]>
  • Loading branch information
thomasjpfan authored and fiedlerNr9 committed Jul 25, 2024
1 parent 36c2ccc commit ce8cf78
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions flytekit/models/core/workflow.py
Original file line number Diff line number Diff line change
Expand Up @@ -598,7 +598,7 @@ def __init__(
self,
resources: typing.Optional[Resources],
extended_resources: typing.Optional[tasks_pb2.ExtendedResources],
container_image: typing.Optional[str],
container_image: typing.Optional[str] = None,
):
self._resources = resources
self._extended_resources = extended_resources
Expand All @@ -613,7 +613,7 @@ def extended_resources(self) -> tasks_pb2.ExtendedResources:
return self._extended_resources

@property
def container_image(self) -> str:
def container_image(self) -> typing.Optional[str]:
return self._container_image

def to_flyte_idl(self):
Expand Down

0 comments on commit ce8cf78

Please sign in to comment.