Skip to content

Commit 87e5dae

Browse files
authored
fix: tests, upgrade ci module (#614)
1 parent 9526de7 commit 87e5dae

File tree

21 files changed

+156
-116
lines changed

21 files changed

+156
-116
lines changed

README.md

Lines changed: 31 additions & 30 deletions
Large diffs are not rendered by default.

examples/default/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
module "terraform_devsecops_alm" {
2-
source = "git::https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm?ref=v2.8.1"
2+
source = "../../"
33
ibmcloud_api_key = var.ibmcloud_api_key
44
toolchain_resource_group = var.toolchain_resource_group
55
toolchain_region = var.toolchain_region

examples/default/version.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ terraform {
44
# Pin to the lowest provider version of the range defined in the main module's version.tf to ensure lowest version still works
55
ibm = {
66
source = "IBM-Cloud/ibm"
7-
version = "= 1.79.1"
7+
version = ">= 1.79.2, < 2.0.0"
88
}
99
}
1010
}

examples/devsecops-ci-toolchain-bring-your-own-app/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
module "terraform_devsecops_alm" {
2-
source = "git::https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm?ref=v2.8.1"
2+
source = "../../"
33
ibmcloud_api_key = var.ibmcloud_api_key
44
toolchain_resource_group = var.toolchain_resource_group
55
toolchain_region = var.toolchain_region

examples/devsecops-ci-toolchain-bring-your-own-app/version.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ terraform {
44
# Pin to the lowest provider version of the range defined in the main module's version.tf to ensure lowest version still works
55
ibm = {
66
source = "IBM-Cloud/ibm"
7-
version = "= 1.79.1"
7+
version = ">= 1.79.2, < 2.0.0"
88
}
99
}
1010
}

examples/devsecops-ci-toolchain-with-key-protect/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
module "terraform_devsecops_alm" {
2-
source = "git::https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm?ref=v2.8.1"
2+
source = "../../"
33
ibmcloud_api_key = var.ibmcloud_api_key
44
toolchain_resource_group = var.toolchain_resource_group
55
toolchain_region = var.toolchain_region

examples/devsecops-ci-toolchain-with-key-protect/version.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ terraform {
44
# Pin to the lowest provider version of the range defined in the main module's version.tf to ensure lowest version still works
55
ibm = {
66
source = "IBM-Cloud/ibm"
7-
version = "= 1.79.1"
7+
version = ">= 1.79.2, < 2.0.0"
88
}
99
}
1010
}

ibm_catalog.json

Lines changed: 32 additions & 18 deletions
Large diffs are not rendered by default.

main.tf

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# workaround for Schematics automatically setting a null value to false for bool type
22
# Issue for the enable_key_protect and enable_secrets_manager inputs as the logic fails with regard to
3-
# determing whether to use the enable_secrets_manager or the toolchain specific counter parts ci/cd/cc_enable_secrets_manager
3+
# determining whether to use the enable_secrets_manager or the toolchain specific counter parts ci/cd/cc_enable_secrets_manager
44
# this could be addressed manually by ticking a setting in Schematics for each of the inputs
55
locals {
66

@@ -332,7 +332,7 @@ module "prereqs" {
332332
module "devsecops_ci_toolchain" {
333333
count = var.create_ci_toolchain ? 1 : 0
334334
depends_on = [ibm_resource_instance.cd_instance]
335-
source = "git::https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-ci-toolchain?ref=v2.7.0"
335+
source = "git::https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-ci-toolchain?ref=v2.7.1"
336336
ibmcloud_api_key = var.ibmcloud_api_key
337337
toolchain_name = (var.prefix == "") ? local.ci_toolchain_name : format("${var.prefix}-%s", local.ci_toolchain_name)
338338
toolchain_region = (var.ci_toolchain_region == "") ? var.toolchain_region : replace(replace(var.ci_toolchain_region, "ibm:yp:", ""), "ibm:ys1:", "")
@@ -460,6 +460,7 @@ module "devsecops_ci_toolchain" {
460460
compliance_pipeline_existing_repo_url = var.compliance_pipeline_existing_repo_url
461461
compliance_pipeline_source_repo_url = var.compliance_pipeline_source_repo_url
462462
compliance_pipelines_repo_name = var.compliance_pipeline_repo_name
463+
compliance_pipelines_repo_is_private_repo = var.compliance_pipelines_repo_is_private_repo
463464

464465
#PIPELINE CONFIG REPO
465466
pipeline_config_repo_existing_url = local.ci_pipeline_config_repo_existing_url
@@ -720,6 +721,7 @@ module "devsecops_cd_toolchain" {
720721
compliance_pipeline_existing_repo_url = var.compliance_pipeline_existing_repo_url
721722
compliance_pipeline_source_repo_url = var.compliance_pipeline_source_repo_url
722723
compliance_pipelines_repo_name = var.compliance_pipeline_repo_name
724+
compliance_pipelines_repo_is_private_repo = var.compliance_pipelines_repo_is_private_repo
723725

724726
#PIPELINE CONFIG REPO
725727
pipeline_config_repo_existing_url = local.cd_pipeline_config_repo_existing_url
@@ -996,6 +998,7 @@ module "devsecops_cc_toolchain" {
996998
compliance_pipeline_existing_repo_url = var.compliance_pipeline_existing_repo_url
997999
compliance_pipeline_source_repo_url = var.compliance_pipeline_source_repo_url
9981000
compliance_pipelines_repo_name = var.compliance_pipeline_repo_name
1001+
compliance_pipelines_repo_is_private_repo = var.compliance_pipelines_repo_is_private_repo
9991002

10001003
#PIPELINE CONFIG REPO
10011004
pipeline_config_repo_existing_url = local.cc_pipeline_config_repo_existing_url

prereqs/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ locals {
1212
# updated_at = "2024-07-05T22:05:57.000Z"
1313
# }
1414
# 2) Use index to find the index of the object containing the correct name
15-
# 3) retrive that object from the list and get the ID from it
15+
# 3) retrieve that object from the list and get the ID from it
1616
secret_group_list = (var.sm_exists) ? data.ibm_sm_secret_groups.secret_groups[0].secret_groups : []
1717
secret_group_id = try(local.secret_group_list[index(local.secret_group_list[*].name, var.sm_secret_group_name)].id, "")
1818

0 commit comments

Comments
 (0)