Skip to content

Commit dd03ab6

Browse files
timrosterSean Sundberg
authored andcommitted
Tags vpc and security group, net acl
* Add optional "tags" variable * Add tag to base security group * Add tags to default security group and network acl * Only tag security group and network acl when provisioning the resources closes #58 Signed-off-by: Tim Robinson <[email protected]>
1 parent 3664b16 commit dd03ab6

File tree

5 files changed

+24
-11
lines changed

5 files changed

+24
-11
lines changed

.github/workflows/verify-pr.yaml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,3 @@ jobs:
1616

1717
verifyMetadata:
1818
uses: cloud-native-toolkit/action-workflows/.github/workflows/verify-module-metadata.yaml@v1
19-
20-
securityScan:
21-
uses: cloud-native-toolkit/action-workflows/.github/workflows/gitguardian-scan.yaml@v1
22-
secrets:
23-
GITGUARDIAN_API_KEY: ${{ secrets.GITGUARDIAN_API_KEY }}

.github/workflows/verify.yaml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,8 @@ jobs:
1717
verifyMetadata:
1818
uses: cloud-native-toolkit/action-workflows/.github/workflows/verify-module-metadata.yaml@v1
1919

20-
securityScan:
21-
uses: cloud-native-toolkit/action-workflows/.github/workflows/gitguardian-scan.yaml@v1
22-
secrets:
23-
GITGUARDIAN_API_KEY: ${{ secrets.GITGUARDIAN_API_KEY }}
24-
2520
release:
26-
needs: [verify, verifyMetadata, securityScan]
21+
needs: [verify, verifyMetadata]
2722
uses: cloud-native-toolkit/action-workflows/.github/workflows/release-module.yaml@v1
2823
secrets:
2924
TOKEN: ${{ secrets.TOKEN }}

main.tf

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ resource ibm_is_vpc vpc {
3939
default_security_group_name = "${local.vpc_name}-default"
4040
default_network_acl_name = "${local.vpc_name}-default"
4141
default_routing_table_name = "${local.vpc_name}-default"
42+
tags = var.tags
4243
}
4344

4445
data ibm_is_vpc vpc {
@@ -47,6 +48,20 @@ data ibm_is_vpc vpc {
4748
name = local.vpc_name
4849
}
4950

51+
resource ibm_resource_tag sg-tag {
52+
count = var.provision ? 1 : 0
53+
54+
resource_id = local.vpc.default_security_group_crn
55+
tags = var.tags
56+
}
57+
58+
resource ibm_resource_tag nacl-tag {
59+
count = var.provision ? 1 : 0
60+
61+
resource_id = local.vpc.default_network_acl_crn
62+
tags = var.tags
63+
}
64+
5065
resource ibm_is_vpc_address_prefix cidr_prefix {
5166
count = var.provision && local.provision_cidr ? var.address_prefix_count : 0
5267

@@ -133,6 +148,7 @@ resource ibm_is_security_group base {
133148
name = local.base_security_group_name
134149
vpc = lookup(local.vpc, "id", "")
135150
resource_group = local.resource_group_id
151+
tags = var.tags
136152
}
137153

138154
data ibm_is_security_group base {

test/stages/stage2-vpc.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,5 @@ module "dev_vpc" {
66
name_prefix = var.name_prefix
77
address_prefix_count = var.address_prefix_count
88
address_prefixes = jsondecode(var.address_prefixes)
9+
tags = ["test"]
910
}

variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,3 +50,9 @@ variable "internal_cidr" {
5050
description = "The cidr range of the internal network"
5151
default = "10.0.0.0/8"
5252
}
53+
54+
variable "tags" {
55+
type = list(string)
56+
default = []
57+
description = "Tags that should be added to the instance"
58+
}

0 commit comments

Comments
 (0)