Skip to content

Commit

Permalink
pre-commit fixs
Browse files Browse the repository at this point in the history
  • Loading branch information
@diegolagospagopa
diegolagospagopa committed Jul 21, 2023
1 parent 67c779a commit b5091fc
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 7 deletions.
2 changes: 1 addition & 1 deletion src/aks-platform/scripts/terraform.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ function download_tool() {
return 1
else
chmod +x $tool
echo "${tool} downloaded! Please note this tool WON'T be copied in your **/bin folder for safety reasons.
echo "${tool} downloaded! Please note this tool WON'T be copied in your **/bin folder for safety reasons.
You need to do it yourself!"
read -p "Press enter to continue"

Expand Down
2 changes: 1 addition & 1 deletion src/domains/diego-app/02_namespace_domain.tf
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ module "domain_pod_identity" {
namespace = kubernetes_namespace.domain_namespace.metadata[0].name
key_vault_id = data.azurerm_key_vault.kv_domain.id

secret_permissions = ["Get"]
secret_permissions = ["Get"]
certificate_permissions = ["Get"]
}

Expand Down
1 change: 0 additions & 1 deletion src/domains/diego-app/80_middleware_tools.tf
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,3 @@ module "cert_mounter" {
kv_name = data.azurerm_key_vault.kv_domain.name
tenant_id = data.azurerm_subscription.current.tenant_id
}

11 changes: 8 additions & 3 deletions src/domains/diego-app/README.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,18 @@
|------|---------|
| <a name="requirement_azuread"></a> [azuread](#requirement\_azuread) | = 2.21.0 |
| <a name="requirement_azurerm"></a> [azurerm](#requirement\_azurerm) | <= 3.45.0 |
| <a name="requirement_helm"></a> [helm](#requirement\_helm) | = 2.8.0 |
| <a name="requirement_helm"></a> [helm](#requirement\_helm) | = 2.7.1 |
| <a name="requirement_kubernetes"></a> [kubernetes](#requirement\_kubernetes) | = 2.17.0 |
| <a name="requirement_null"></a> [null](#requirement\_null) | = 3.2.1 |

## Modules

| Name | Source | Version |
|------|--------|---------|
| <a name="module_cert_mounter"></a> [cert\_mounter](#module\_cert\_mounter) | git::https://github.com/pagopa/terraform-azurerm-v3.git//cert_mounter | v6.20.1 |
| <a name="module_domain_key_vault_secrets_query"></a> [domain\_key\_vault\_secrets\_query](#module\_domain\_key\_vault\_secrets\_query) | git::https://github.com/pagopa/terraform-azurerm-v3.git//key_vault_secrets_query | v6.20.1 |
| <a name="module_domain_pod_identity"></a> [domain\_pod\_identity](#module\_domain\_pod\_identity) | git::https://github.com/pagopa/terraform-azurerm-v3.git//kubernetes_pod_identity | v6.20.1 |
| <a name="module_domain_pod_identity"></a> [domain\_pod\_identity](#module\_domain\_pod\_identity) | git::https://github.com/pagopa/terraform-azurerm-v3.git//kubernetes_pod_identity | pod-identity-fix-permissions |
| <a name="module_tls_checker"></a> [tls\_checker](#module\_tls\_checker) | git::https://github.com/pagopa/terraform-azurerm-v3.git//tls_checker | v6.20.1 |

## Resources

Expand All @@ -27,8 +29,9 @@
| [azurerm_key_vault_secret.app_insights_connection_string](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/key_vault_secret) | resource |
| [azurerm_key_vault_secret.azure_devops_sa_cacrt](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/key_vault_secret) | resource |
| [azurerm_key_vault_secret.azure_devops_sa_token](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/key_vault_secret) | resource |
| [azurerm_private_dns_a_record.ingress](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/private_dns_a_record) | resource |
| [azurerm_role_assignment.aks_cluster_role](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/role_assignment) | resource |
| [helm_release.reloader](https://registry.terraform.io/providers/hashicorp/helm/2.8.0/docs/resources/release) | resource |
| [helm_release.reloader](https://registry.terraform.io/providers/hashicorp/helm/2.7.1/docs/resources/release) | resource |
| [kubernetes_config_map.added](https://registry.terraform.io/providers/hashicorp/kubernetes/2.17.0/docs/resources/config_map) | resource |
| [kubernetes_config_map.changed](https://registry.terraform.io/providers/hashicorp/kubernetes/2.17.0/docs/resources/config_map) | resource |
| [kubernetes_config_map.replaced](https://registry.terraform.io/providers/hashicorp/kubernetes/2.17.0/docs/resources/config_map) | resource |
Expand All @@ -53,6 +56,7 @@
| [azurerm_log_analytics_workspace.log_analytics](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/log_analytics_workspace) | data source |
| [azurerm_monitor_action_group.email](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/monitor_action_group) | data source |
| [azurerm_monitor_action_group.slack](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/monitor_action_group) | data source |
| [azurerm_private_dns_zone.internal](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/private_dns_zone) | data source |
| [azurerm_resource_group.monitor_rg](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/resource_group) | data source |
| [azurerm_resource_group.rg_vnet_core](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/resource_group) | data source |
| [azurerm_subscription.current](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/data-sources/subscription) | data source |
Expand Down Expand Up @@ -87,6 +91,7 @@
| <a name="input_prefix"></a> [prefix](#input\_prefix) | n/a | `string` | n/a | yes |
| <a name="input_tags"></a> [tags](#input\_tags) | n/a | `map(any)` | <pre>{<br> "CreatedBy": "Terraform"<br>}</pre> | no |
| <a name="input_terraform_remote_state_core"></a> [terraform\_remote\_state\_core](#input\_terraform\_remote\_state\_core) | n/a | <pre>object({<br> resource_group_name = string,<br> storage_account_name = string,<br> container_name = string,<br> key = string<br> })</pre> | n/a | yes |
| <a name="input_tls_cert_check_helm"></a> [tls\_cert\_check\_helm](#input\_tls\_cert\_check\_helm) | tls cert helm chart configuration | <pre>object({<br> chart_version = string,<br> image_name = string,<br> image_tag = string<br> })</pre> | n/a | yes |

## Outputs

Expand Down
2 changes: 1 addition & 1 deletion src/domains/diego-app/env/dev/terraform.tfvars
View file
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ log_analytics_workspace_resource_group_name = "dvopla-d-monitor-rg"
aks_name = "dvopla-d-neu-dev01-aks"
aks_resource_group_name = "dvopla-d-neu-dev01-aks-rg"

ingress_load_balancer_ip = "10.11.100.250"
ingress_load_balancer_ip = "10.11.100.250"
ingress_load_balancer_hostname = "dev01.diego.internal.devopslab.pagopa.it"

#
Expand Down

0 comments on commit b5091fc

Please sign in to comment.