Skip to content

Commit

Permalink
cleanup: remove unused ACI resources for trusted.ci and cert.ci (#532)
Browse files Browse the repository at this point in the history
While working on the Terraform module split in
jenkins-infra/helpdesk#3818, it appeared that
some unneeded (and unused) resources were creating by the "former
module".

The migration to the "3 distinct modules" kept the resources, but this
PR removes them as we won't need to use ACI with trusted and cert.

---------

Signed-off-by: Damien Duportal <[email protected]>
  • Loading branch information
dduportal authored Nov 29, 2023
1 parent 6bb4581 commit f3b400c
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 18 deletions.
8 changes: 0 additions & 8 deletions cert.ci.jenkins.io.tf
Original file line number Diff line number Diff line change
Expand Up @@ -71,14 +71,6 @@ module "cert_ci_jenkins_io_azurevm_agents" {
}
}

module "cert_ci_jenkins_io_aci_agents" {
source = "./.shared-tools/terraform/modules/azure-jenkinsinfra-aci-agents"

service_short_stripped_name = module.cert_ci_jenkins_io.service_short_stripped_name
aci_agents_resource_group_name = module.cert_ci_jenkins_io_azurevm_agents.ephemeral_agents_resource_group_name
controller_service_principal_id = module.cert_ci_jenkins_io.controler_service_principal_id
}

## Service DNS records
resource "azurerm_dns_a_record" "cert_ci_jenkins_io_controller" {
name = "controller"
Expand Down
2 changes: 1 addition & 1 deletion contributors.jenkins.io.tf
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ data "azurerm_storage_account_sas" "contributors_jenkins_io" {
}

output "contributors_jenkins_io_share_url" {
value = azurerm_storage_share.contributors_jenkins_io.url
value = azurerm_storage_share.contributors_jenkins_io.url
}

output "contributors_jenkins_io_sas_query_string" {
Expand Down
8 changes: 0 additions & 8 deletions trusted.ci.jenkins.io.tf
Original file line number Diff line number Diff line change
Expand Up @@ -89,14 +89,6 @@ module "trusted_ci_jenkins_io_azurevm_agents" {
}
}

module "trusted_ci_jenkins_io_aci_agents" {
source = "./.shared-tools/terraform/modules/azure-jenkinsinfra-aci-agents"

service_short_stripped_name = module.trusted_ci_jenkins_io.service_short_stripped_name
aci_agents_resource_group_name = module.trusted_ci_jenkins_io_azurevm_agents.ephemeral_agents_resource_group_name
controller_service_principal_id = module.trusted_ci_jenkins_io.controler_service_principal_id
}

resource "azurerm_private_dns_a_record" "trusted_ci_controller" {
name = "@"
zone_name = azurerm_private_dns_zone.trusted.name
Expand Down
2 changes: 1 addition & 1 deletion updates.jenkins.io.tf
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ data "azurerm_storage_account_sas" "updates_jenkins_io" {
}

output "updates_jenkins_io_share_url" {
value = azurerm_storage_share.updates_jenkins_io.url
value = azurerm_storage_share.updates_jenkins_io.url
}

output "updates_jenkins_io_sas_query_string" {
Expand Down

0 comments on commit f3b400c

Please sign in to comment.