Skip to content

Commit

Permalink
Fix Dockerfile arg and formatted strings
Browse files Browse the repository at this point in the history
  • Loading branch information
Shane Brown committed Aug 9, 2023
1 parent 018cfa9 commit af41baa
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions buildrunner/docker/multiplatform_image_builder.py
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ def _build_single_image(self,

assert os.path.isdir(path) and os.path.exists(f"{file}"), \
f"Either path {path}({os.path.isdir(path)}) or file " \
"'{file}'({os.path.exists(f'{file}')}) does not exist!"
f"'{file}'({os.path.exists(f'{file}')}) does not exist!"

tagged_names = [f"{name}:{tag}" for tag in tags]
LOGGER.debug(f"Building {tagged_names} for {platform}")
Expand Down Expand Up @@ -335,7 +335,7 @@ def push(self, name: str, dest_names: List[str] = None) -> None:
curr_process.kill()
if retries == 0:
raise TimeoutError(f"Timeout pushing {dest_names} after {retries} retries"
" and {timeout_seconds} seconds each try")
f" and {timeout_seconds} seconds each try")
else:
# Process finished within timeout
LOGGER.info(f"Successfully created multiplatform images {dest_names}")
Expand Down
2 changes: 1 addition & 1 deletion buildrunner/steprunner/tasks/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ def run(self, context):
built_images = self.step_runner.multi_platform.build_multiple_images(
platforms=self.platform,
path=self.path,
file=self.config.get('dockerfile'),
file=self.dockerfile,
name=self.get_unique_build_name(),
)

Expand Down

0 comments on commit af41baa

Please sign in to comment.