Skip to content
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

Terraform module gcp compute engine #33

Merged
merged 8 commits into from
Sep 22, 2023

Conversation

angeleenasunny
Copy link
Collaborator

Terraform Plan :

module.compute_engine.google_compute_instance.compute_engine will be created

  • resource "google_compute_instance" "compute_engine" {
    • can_ip_forward = false

    • cpu_platform = (known after apply)

    • current_status = (known after apply)

    • deletion_protection = false

    • guest_accelerator = (known after apply)

    • id = (known after apply)

    • instance_id = (known after apply)

    • label_fingerprint = (known after apply)

    • machine_type = "e2-medium"

    • metadata_fingerprint = (known after apply)

    • min_cpu_platform = (known after apply)

    • name = "Qburst-DevOps"

    • project = (known after apply)

    • self_link = (known after apply)

    • tags = [

      • "QBurst",
        ]
    • tags_fingerprint = (known after apply)

    • zone = "us-central1-a"

    • boot_disk {

      • auto_delete = true

      • device_name = (known after apply)

      • disk_encryption_key_sha256 = (known after apply)

      • kms_key_self_link = (known after apply)

      • mode = "READ_WRITE"

      • source = (known after apply)

      • initialize_params {

        • image = "ubuntu-os-cloud/ubuntu-2004-lts"
        • labels = {
          • "my_label" = "QBurst"
            }
        • size = (known after apply)
        • type = (known after apply)
          }
          }
    • network_interface {

      • internal_ipv6_prefix_length = (known after apply)
      • ipv6_access_type = (known after apply)
      • ipv6_address = (known after apply)
      • name = (known after apply)
      • network = "10.1.1.0"
      • network_ip = (known after apply)
      • stack_type = (known after apply)
      • subnetwork = (known after apply)
      • subnetwork_project = (known after apply)
        }
    • service_account {

module.compute_engine.google_service_account.computeengine_sa will be created

  • resource "google_service_account" "computeengine_sa" {
    • account_id = "compute-engine-sa"
    • disabled = false
    • display_name = "Service Account"
    • email = (known after apply)
    • id = (known after apply)
    • member = (known after apply)
    • name = (known after apply)
    • project = (known after apply)
    • unique_id = (known after apply)
      }

Plan: 2 to add, 0 to change, 0 to destroy.

Changes to Outputs:

  • computeengine_module_outputs = {
    • instance_id = (known after apply)
      }

@@ -0,0 +1,13 @@
module "compute_engine" {
source = ".../.../gcp/modules/compute_engine"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls change to ../../modules/compute_engine

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qburst-praven This is done

@@ -0,0 +1,16 @@

variable "name"{
type=string
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add defaults for all variables, "" in this case. Please update the README with the same.

Also, please run terraform fmt --recrusive against the terraform folder.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qburst-praven Formatting is done and changes committed

@angeleenasunny angeleenasunny merged commit ef1643f into main Sep 22, 2023
@qburst-praven qburst-praven deleted the Terraform_Module_GCP_Compute_Engine branch September 22, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants