Skip to content

Commit 55d46fe

Browse files
authored
Fix build breakage caused by #14140. (#14155)
The return type of `build_lib.run_build` was changed, without changing all the callers. In particular, `build_project.run_build` was expected to still return an ID. ``` > git grep build_project.run_build infra/build/functions/project_experiment.py: return build_project.run_build(project_name, infra/build/functions/request_build.py: build_id = build_project.run_build(oss_fuzz_project, build_steps, credentials, infra/build/functions/target_experiment.py: build_id = build_project.run_build( infra/build/functions/trial_build.py: build_ids[project_name] = (build_project.run_build( infra/build/functions/trial_build_test.py: @mock.patch('build_project.run_build') ``` Fixes #14153
1 parent 3b047a7 commit 55d46fe

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

infra/build/functions/build_project.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -775,14 +775,15 @@ def run_build(oss_fuzz_project,
775775
'logsBucket': bucket,
776776
'queueTtl': str(QUEUE_TTL_SECONDS) + 's',
777777
}
778-
return build_lib.run_build(oss_fuzz_project,
779-
build_steps,
780-
credentials,
781-
cloud_project,
782-
timeout,
783-
body_overrides=body_overrides,
784-
tags=tags,
785-
experiment=experiment)
778+
build = build_lib.run_build(oss_fuzz_project,
779+
build_steps,
780+
credentials,
781+
cloud_project,
782+
timeout,
783+
body_overrides=body_overrides,
784+
tags=tags,
785+
experiment=experiment)
786+
return build['id']
786787

787788

788789
def parse_args(description, args):

0 commit comments

Comments
 (0)