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

Revert "fix(terraform): turn on manual integration" #2163

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

thekaveman
Copy link
Member

This reverts commit ff98a62.

Follow up to #2160 (comment)

@thekaveman thekaveman requested a review from a team as a code owner June 13, 2024 17:44
@thekaveman thekaveman linked an issue Jun 13, 2024 that may be closed by this pull request
@github-actions github-actions bot added the infrastructure Terraform, Azure, etc. label Jun 13, 2024
@thekaveman thekaveman self-assigned this Jun 13, 2024
Copy link

Coverage report

This PR does not seem to contain any modification to coverable code.

@thekaveman
Copy link
Member Author

thekaveman commented Jun 13, 2024

OK I reverted the state back to before I merged this, and the new plan with this revert:

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

Terraform will perform the following actions:

  # azurerm_app_service_source_control.main must be replaced
-/+ resource "azurerm_app_service_source_control" "main" {
      ~ id                     = "/subscriptions/xxx/resourceGroups/RG-CDT-PUB-VIP-CALITP-D-001/providers/Microsoft.Web/sites/AS-CDT-PUB-VIP-CALITP-D-001" -> (known after apply)
      ~ scm_type               = "GitHub" -> (known after apply)
      ~ uses_github_action     = false -> (known after apply)
        # (7 unchanged attributes hidden)

      + github_action_configuration { # forces replacement
          + generate_workflow_file = false # forces replacement
          + linux_action           = (known after apply)

          + container_configuration {
              + image_name   = "cal-itp/benefits" # forces replacement
              + registry_url = "https://ghcr.io/" # forces replacement
            }
        }
    }

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

@thekaveman thekaveman merged commit 76783a3 into dev Jun 13, 2024
12 checks passed
@thekaveman thekaveman deleted the fix/terraform-source-control branch June 13, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Terraform, Azure, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Terraform config for new deployment setup
2 participants