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

KM-19: Add module code #2

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Conversation

warrenbrown
Copy link
Collaborator

No description provided.

.github/workflows/pre-commit.yaml Outdated Show resolved Hide resolved
.github/workflows/pre-commit.yaml Outdated Show resolved Hide resolved
.github/workflows/terraform.yaml Outdated Show resolved Hide resolved
modules/workspace/main.tf Outdated Show resolved Hide resolved
@warrenbrown warrenbrown force-pushed the feature/km-19-add-module-code branch 2 times, most recently from 009da85 to 387c80c Compare July 12, 2020 16:04
@warrenbrown warrenbrown requested a review from johncblandii July 12, 2020 19:35
@warrenbrown warrenbrown force-pushed the feature/km-19-add-module-code branch 2 times, most recently from 0ef680e to 4b64ac3 Compare July 19, 2020 17:04
@github-actions github-actions bot added the ci label Jul 19, 2020
@github-actions
Copy link

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.


------------------------------------------------------------------------

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

2 similar comments
@github-actions
Copy link

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.


------------------------------------------------------------------------

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

@github-actions
Copy link

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.


------------------------------------------------------------------------

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

@warrenbrown warrenbrown force-pushed the feature/km-19-add-module-code branch from 763fb75 to ccb95b0 Compare July 21, 2020 20:53
@github-actions
Copy link

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

@github-actions
Copy link

✅ Terraform Initialization
✅ Terraform Validation
✅ Terraform Plan

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

The remote workspace is configured to work with configuration at
test relative to the target repository.

Terraform will upload the contents of the following directory,
excluding files or directories as defined by a .terraformignore file
at /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace/.terraformignore (if it is present),
in order to capture the filesystem context the remote workspace expects:
    /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace

To view this run in a browser, visit:
https://app.terraform.io/app/km-sandbox/terraform-tfc-workspace/runs/run-QKEZRgcF3jw1rzzD

Waiting for the plan to start...

Terraform v0.12.29
Configuring remote state backend...
Initializing Terraform configuration...
2020/08/10 18:11:39 [DEBUG] Using modified User-Agent: Terraform/0.12.29 TFC/32d6f8a83a
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

null_resource.test1: Refreshing state... [id=2940663747714446314]

------------------------------------------------------------------------

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create
  - destroy

Terraform will perform the following actions:

  # null_resource.test1 will be destroyed
  - resource "null_resource" "test1" {
      - id = "2940663747714446314" -> null
    }

  # module.core.tfe_workspace.workspace will be created
  + resource "tfe_workspace" "workspace" {
      + auto_apply            = false
      + external_id           = (known after apply)
      + file_triggers_enabled = true
      + id                    = (known after apply)
      + name                  = "terraform-tfc-workspace"
      + operations            = true
      + organization          = "organization-test"
      + queue_all_runs        = true
      + terraform_version     = "latest"
      + working_directory     = "/test"

      + vcs_repo {
          + branch             = "master"
          + identifier         = "katapultmedia/terraform-tfc-workspace"
          + ingress_submodules = false
          + oauth_token_id     = "alkasiopejk"
        }
    }

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

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

Copy link
Member

@johncblandii johncblandii left a comment

Choose a reason for hiding this comment

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

Couple tweaks.

main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
outputs.tf Show resolved Hide resolved
@github-actions
Copy link

github-actions bot commented Sep 8, 2020

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

The remote workspace is configured to work with configuration at
test relative to the target repository.

Terraform will upload the contents of the following directory,
excluding files or directories as defined by a .terraformignore file
at /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace/.terraformignore (if it is present),
in order to capture the filesystem context the remote workspace expects:
    /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace

To view this run in a browser, visit:
https://app.terraform.io/app/km-sandbox/terraform-tfc-workspace/runs/run-Xbqn1oJmZJsaH2XC

Waiting for the plan to start...

Terraform v0.13.2
Configuring remote state backend...
Initializing Terraform configuration...
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.


Error: Provider configuration not present

To work with null_resource.test1 its original provider configuration at
provider["registry.terraform.io/-/null"] is required, but it has been removed.
This occurs when a provider configuration is removed while objects created by
that provider still exist in the state. Re-add the provider configuration to
destroy null_resource.test1, after which you can remove the provider
configuration again.

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

@github-actions
Copy link

github-actions bot commented Sep 8, 2020

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

The remote workspace is configured to work with configuration at
test relative to the target repository.

Terraform will upload the contents of the following directory,
excluding files or directories as defined by a .terraformignore file
at /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace/.terraformignore (if it is present),
in order to capture the filesystem context the remote workspace expects:
    /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace

To view this run in a browser, visit:
https://app.terraform.io/app/km-sandbox/terraform-tfc-workspace/runs/run-RfrmjctYGMtUhLWR

Waiting for the plan to start...

Terraform v0.13.2
Configuring remote state backend...
Initializing Terraform configuration...

Error: Provider configuration not present

Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

To work with null_resource.test1 its original provider configuration at
provider["registry.terraform.io/-/null"] is required, but it has been removed.
This occurs when a provider configuration is removed while objects created by
that provider still exist in the state. Re-add the provider configuration to
destroy null_resource.test1, after which you can remove the provider
configuration again.

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

@github-actions
Copy link

github-actions bot commented Sep 8, 2020

✅ Terraform Initialization
✅ Terraform Validation
❌ Terraform Plan

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

The remote workspace is configured to work with configuration at
test relative to the target repository.

Terraform will upload the contents of the following directory,
excluding files or directories as defined by a .terraformignore file
at /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace/.terraformignore (if it is present),
in order to capture the filesystem context the remote workspace expects:
    /home/runner/work/terraform-tfc-workspace/terraform-tfc-workspace

To view this run in a browser, visit:
https://app.terraform.io/app/km-sandbox/terraform-tfc-workspace/runs/run-opFHoDtSy9CqUxrA

Waiting for the plan to start...

Terraform v0.13.2
Configuring remote state backend...
Initializing Terraform configuration...
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be

persisted to local or remote state storage.

Error: Provider configuration not present

To work with null_resource.test1 its original provider configuration at
provider["registry.terraform.io/-/null"] is required, but it has been removed.
This occurs when a provider configuration is removed while objects created by
that provider still exist in the state. Re-add the provider configuration to
destroy null_resource.test1, after which you can remove the provider
configuration again.

User: @warrenbrown
Action: pull_request
Working Directory: ./test
Workflow: terraform

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants