Skip to content

Commit 8ce98fe

Browse files
cache_name -> cache_image
Signed-off-by: David Korczynski <[email protected]>
1 parent c30bd8c commit 8ce98fe

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

infra/build/functions/build_lib.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ def get_docker_build_step(image_names,
388388
use_buildkit_cache=False,
389389
src_root='oss-fuzz',
390390
architecture='x86_64',
391-
cache_name=''):
391+
cache_image=''):
392392
"""Returns the docker build step."""
393393
assert len(image_names) >= 1
394394
directory = os.path.join(src_root, directory)
@@ -405,7 +405,7 @@ def get_docker_build_step(image_names,
405405
_make_image_name_architecture_specific(image_name, architecture)
406406
for image_name in image_names
407407
]
408-
if cache_name:
408+
if cache_image:
409409
args.extend(['--build-arg', f'CACHE_IMAGE={cache_name}'])
410410

411411
for image_name in image_names:
@@ -442,7 +442,7 @@ def get_project_image_steps( # pylint: disable=too-many-arguments
442442
config,
443443
architectures=None,
444444
experiment=False,
445-
cache_name=''):
445+
cache_image=''):
446446
"""Returns GCB steps to build OSS-Fuzz project image."""
447447
if architectures is None:
448448
architectures = []
@@ -462,7 +462,7 @@ def get_project_image_steps( # pylint: disable=too-many-arguments
462462
docker_build_step = get_docker_build_step([image],
463463
os.path.join('projects', name),
464464
src_root=src_root,
465-
cache_name=cache_name)
465+
cache_image=cache_image)
466466
steps.append(docker_build_step)
467467
srcmap_step_id = get_srcmap_step_id()
468468
steps.extend([{

infra/build/functions/build_project.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -358,18 +358,18 @@ def get_build_steps_for_project(project,
358358
# if we use caching, then we need to use the right name. We assume that
359359
# there is only a single sanitizer.
360360
if use_caching:
361-
cache_name = ('us-central1-docker.pkg.dev/oss-fuzz/oss-fuzz-gen/'
362-
f'{project.name}-ofg-cached-{project.sanitizers[0]}')
361+
cache_image = ('us-central1-docker.pkg.dev/oss-fuzz/oss-fuzz-gen/'
362+
f'{project.name}-ofg-cached-{project.sanitizers[0]}')
363363
else:
364-
cache_name = ''
364+
cache_image = ''
365365
build_steps = build_lib.get_project_image_steps(
366366
project.name,
367367
project.image,
368368
project.fuzzing_language,
369369
config=config,
370370
architectures=project.architectures,
371371
experiment=config.experiment,
372-
cache_name=cache_name)
372+
cache_image=cache_image)
373373

374374
# Sort engines to make AFL first to test if libFuzzer has an advantage in
375375
# finding bugs first since it is generally built first.

0 commit comments

Comments
 (0)