Skip to content

Commit e1a94cd

Browse files
author
akocbek
committed
updates rally
1 parent f55f789 commit e1a94cd

File tree

3 files changed

+39
-18
lines changed

3 files changed

+39
-18
lines changed

ibm_catalog.json

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1397,22 +1397,19 @@
13971397
},
13981398
{
13991399
"role_crns": [
1400-
"crn:v1:bluemix:public:iam::::serviceRole:Writer"
1401-
],
1402-
"service_name": "codeengine"
1403-
},
1404-
{
1405-
"role_crns": [
1406-
"crn:v1:bluemix:public:iam::::role:Editor"
1400+
"crn:v1:bluemix:public:iam::::serviceRole:Writer",
1401+
"crn:v1:bluemix:public:iam::::role:Editor"
14071402
],
1408-
"service_name": "codeengine"
1403+
"service_name": "codeengine",
1404+
"notes": "Required for creating code engine."
14091405
},
14101406
{
14111407
"role_crns": [
14121408
"crn:v1:bluemix:public:iam::::serviceRole:Writer",
14131409
"crn:v1:bluemix:public:iam::::role:Editor"
14141410
],
1415-
"service_name": "container-registry"
1411+
"service_name": "container-registry",
1412+
"notes": "Required for creating container registry namespaces."
14161413
},
14171414
{
14181415
"service_name": "is",

solutions/fleets/main.tf

Lines changed: 30 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -329,6 +329,7 @@ module "cloud_logs" {
329329
resource_group_id = module.resource_group.resource_group_id
330330
region = var.region
331331
instance_name = local.icl_name
332+
resource_tags = var.resource_tags
332333
}
333334

334335
resource "ibm_iam_service_id" "logs_service_id" {
@@ -355,8 +356,8 @@ resource "ibm_iam_service_policy" "logs_policy" {
355356
########################################################################################################################
356357
locals {
357358
enable_cloud_monitoring = var.cloud_monitoring_plan == "none" ? false : true
358-
monitoring_name = "${local.prefix}-sysdig"
359-
monitoring_key_name = "${local.prefix}-sysdig-key"
359+
monitoring_name = "${local.prefix}sysdig"
360+
monitoring_key_name = "${local.prefix}sysdig-key"
360361
}
361362

362363
module "cloud_monitoring" {
@@ -370,6 +371,7 @@ module "cloud_monitoring" {
370371
service_endpoints = "public-and-private"
371372
enable_platform_metrics = false
372373
manager_key_name = local.monitoring_key_name
374+
resource_tags = var.resource_tags
373375
}
374376

375377

@@ -395,7 +397,7 @@ locals {
395397
"data" = {
396398
password = var.ibmcloud_api_key,
397399
username = "iamapikey",
398-
server = "us.icr.io"
400+
server = local.container_registry
399401
}
400402
}
401403
}
@@ -460,3 +462,28 @@ module "secret" {
460462
# Issue with provider, service_access is not supported at the moment. https://github.com/IBM-Cloud/terraform-provider-ibm/issues/5232
461463
# service_access = each.value.service_access
462464
}
465+
466+
##############################################################################
467+
# Container Registry
468+
##############################################################################
469+
locals {
470+
registry_region_result = data.external.container_registry_region.result
471+
registry = lookup(local.registry_region_result, "registry", null)
472+
container_registry = local.registry != null ? "private.${local.registry}" : null
473+
registry_region_error = lookup(local.registry_region_result, "error", null)
474+
475+
# This will cause Terraform to fail if "error" is present in the external script output executed as a part of container_registry_region
476+
# tflint-ignore: terraform_unused_declarations
477+
fail_if_registry_region_error = local.registry_region_error != null ? tobool("Registry region script failed: ${local.registry_region_error}") : null
478+
}
479+
480+
# get the container registry endpoint according to the region
481+
data "external" "container_registry_region" {
482+
program = ["bash", "../../scripts/get-cr-region.sh"]
483+
484+
query = {
485+
RESOURCE_GROUP_ID = module.resource_group.resource_group_id
486+
REGION = var.region
487+
IBMCLOUD_API_KEY = var.ibmcloud_api_key
488+
}
489+
}

solutions/fleets/outputs.tf

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,19 +58,16 @@ output "vpc_name" {
5858
}
5959

6060
output "next_steps_text" {
61-
value = "Now, visit the Code Engine project URL to verify secrets, network configurations, and ensure readiness for workload deployment."
61+
value = "Now, visit the documentation to learn how to run a serverless fleet using Code Engine."
6262
description = "Next steps text"
6363
}
6464

6565
output "next_step_primary_label" {
66-
value = "Go to Code Engine Project"
66+
value = "Learn how to run a serverless fleet"
6767
description = "Primary label"
6868
}
6969

7070
output "next_step_primary_url" {
71-
value = "https://cloud.ibm.com/containers/serverless/project/${var.region}/${module.project.id}/overview"
71+
value = "https://github.com/IBM/CodeEngine/blob/main/beta/serverless-fleets/README.md#launch-a-fleet"
7272
description = "primary url"
7373
}
74-
75-
76-

0 commit comments

Comments
 (0)