terraform-google-modules / terraform-google-bastion-host

Generates a bastion host VM compatible with OS Login and IAP Tunneling that can be used to access internal VMs

https://registry.terraform.io/modules/terraform-google-modules/bastion-host/google

Apache License 2.0

128 stars 92 forks source link

Fix: Update Debian Image to 12 #179

Closed RoseSecurity closed 7 months ago

RoseSecurity commented 9 months ago

What and Why

Update default machine_type to debian-12
Debian 12.0 was initially released on June 10th, 2023

google-cla[bot] commented 9 months ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

RoseSecurity commented 7 months ago

@Djabx Any thoughts on this?

Djabx commented 7 months ago

@RoseSecurity Hi, No issue for me: I'm setting the variable on my side.

But I'm just a contributor, not a maintainer :P

apeabody commented 7 months ago

Thanks for the contribution @RoseSecurity!

It looks like one of the commits used an email address not covered by the CLA.

RoseSecurity commented 7 months ago

Thanks for the contribution @RoseSecurity!

It looks like one of the commits used an email address not covered by the CLA.

No worries! Just signed the CLA with that email address as well. @apeabody

apeabody commented 7 months ago

Thanks for the contribution @RoseSecurity! It looks like one of the commits used an email address not covered by the CLA.

No worries! Just signed the CLA with that email address as well. @apeabody

Thanks! - That check is now green.

apeabody commented 7 months ago

/gcbrun

apeabody commented 7 months ago

/gcbrun