Skip to content

Commit

Permalink
Passes --public-ip-address to hailctl dataproc start (#14653)
Browse files Browse the repository at this point in the history
Closes #14652.

See populationgenomics#346. Thanks for the
contribution @illusional!

Gives Dataproc clusters started via `hailctl dataproc start` internet
access by default, since we need it to install some of our dependencies,
per the error message in the linked issue.

---------

Co-authored-by: Michael Franklin <[email protected]>
Co-authored-by: Michael Franklin <[email protected]>
  • Loading branch information
3 people authored Aug 19, 2024
1 parent 7d25779 commit 856d388
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions hail/python/hailtop/hailctl/dataproc/start.py
Original file line number Diff line number Diff line change
Expand Up @@ -396,6 +396,8 @@ def jvm_heap_size_gib(machine_type: str, memory_fraction: float) -> int:
if service_account:
cmd.append('--service-account={}'.format(service_account))

cmd.append('--public-ip-address')

cmd.extend(pass_through_args)

# print underlying gcloud command
Expand Down

0 comments on commit 856d388

Please sign in to comment.