-
Notifications
You must be signed in to change notification settings - Fork 982
feat: Need to make compute-vm service_account configuration "optional" #1691
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
It's already possible AFAIK, just pass |
Hello, tested, but unfortunally didn't work, passed var.service_account to null and false for the creation and it created the instance with the default compute SA and with the following: service_account {
email : Known after apply
scopes : [
- "https://www.googleapis.com/auth/devstorage.read_only"
- "https://www.googleapis.com/auth/logging.write"
- "https://www.googleapis.com/auth/monitoring.write"
]
} The call: module "vm-test" {
source = "localterraform.com/catalog/module-compute-vm/google"
version = "26.0.0"
project_id = var.project_id
zone = "europe-west1-b"
name = var.hostname
hostname = var.hostname
instance_type = "custom-2-4096"
boot_disk = {
image = "debian-cloud/debian-11"
size = 65
}
network_interfaces = [
{
network = var.network
subnetwork = var.subnetwork
addresses = {
internal =var.ip
external = null
}
}
]
service_account_create = false
service_account = null
}
|
Ok, let me fix this :) Thanks for checking. |
Great, tested and working as expected, thank you Ludo! |
Fantastic, thanks for confirming, and of course raising the issue in the first place! |
Hello Team,
we need to have the capabilit of disable service_account configuration when creating a compute-vm as our security policies recognize it as a threat. Previously we used an ad-hoc module with a dynamic block for the service_account, can you implement this in your fabric module?
Thank you.
The text was updated successfully, but these errors were encountered: