emilymye commented on a change in pull request #13399:
URL: https://github.com/apache/beam/pull/13399#discussion_r532993637
##########
File path: sdks/python/apache_beam/runners/portability/sdk_container_builder.py
##########
@@ -236,10 +235,21 @@ def invoke_docker_build_and_push(self,
container_image_name):
build.timeout = Duration().FromSeconds(seconds=1800)
now = time.time()
- _LOGGER.info('Building sdk container, this may take a few minutes...')
+ _LOGGER.info(
+ 'Building sdk container with google cloud build, this may '
+ 'take a few minutes...')
operation = client.create_build(project_id=project_id, build=build)
# if build fails exception will be raised and stops the job submission.
- result = operation.result()
+ try:
+ result = operation.result()
+ except Exception as e:
+ build_lists = client.list_builds(
Review comment:
Why list_ _builds vs get_build using the operation build ID here?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
