Skip to content

Commit

Permalink
remove duplicate kwargs to docker build command
Browse files Browse the repository at this point in the history
  • Loading branch information
shubham3121 committed Jan 4, 2024
1 parent bcf6583 commit 98cfd35
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 0 additions & 1 deletion packages/syft/src/syft/custom_worker/builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@ def _build_image(self, tag: str, **build_opts) -> Tuple[Image, Iterable]:
image = client.images.build(
tag=tag,
pull=True,
rm=True,
timeout=self.BUILD_MAX_WAIT,
**build_opts,
)
Expand Down
5 changes: 4 additions & 1 deletion packages/syft/src/syft/service/worker/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,10 @@ def create_default_image(
"""
worker_config = DockerWorkerConfig(dockerfile=default_cpu_dockerfile)

result = image_stash.get_by_docker_config(credentials, worker_config)
result = image_stash.get_by_docker_config(
credentials=credentials,
config=worker_config,
)

if result.ok() is None:
default_syft_image = SyftWorkerImage(
Expand Down

0 comments on commit 98cfd35

Please sign in to comment.