Skip to content

Commit ce0c21e

Browse files
authored
fix: bypassing scc refresh after deprecation (#6401)
* fix: bypassing scc refresh after deprecation * fix: bypassing scc refresh after deprecation * added end of life comments on checks
1 parent 5d9ee65 commit ce0c21e

File tree

2 files changed

+41
-18
lines changed

2 files changed

+41
-18
lines changed

ibm/service/resourcecontroller/data_source_ibm_resource_instance.go

Lines changed: 22 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ func DataSourceIBMResourceInstanceRead(d *schema.ResourceData, meta interface{})
190190
}
191191
if _, ok := d.GetOk("name"); ok {
192192
name := d.Get("name").(string)
193+
193194
resourceInstanceListOptions := rc.ListResourceInstancesOptions{
194195
Name: &name,
195196
}
@@ -302,12 +303,17 @@ func DataSourceIBMResourceInstanceRead(d *schema.ResourceData, meta interface{})
302303

303304
ID: instance.ResourceID,
304305
}
305-
service, _, err := globalClient.GetCatalogEntry(&options)
306-
if err != nil {
307-
return fmt.Errorf("[ERROR] Error retrieving service offering: %s", err)
308-
}
309306

310-
d.Set("service", service.Name)
307+
// Note: Once the Compliance service (SCC) reaches its end of life, this conditional check can be revisited or safely removed.
308+
if *instance.ResourceID == "compliance" {
309+
d.Set("service", "compliance")
310+
} else {
311+
service, _, err := globalClient.GetCatalogEntry(&options)
312+
if err != nil {
313+
return fmt.Errorf("[ERROR] Error retrieving service offering: %s", err)
314+
}
315+
d.Set("service", service.Name)
316+
}
311317

312318
d.Set(flex.ResourceName, instance.Name)
313319
d.Set(flex.ResourceCRN, instance.CRN)
@@ -354,11 +360,18 @@ func DataSourceIBMResourceInstanceRead(d *schema.ResourceData, meta interface{})
354360

355361
ID: instance.ResourcePlanID,
356362
}
357-
plan, _, err := globalClient.GetCatalogEntry(&planOptions)
358-
if err != nil {
359-
return fmt.Errorf("[ERROR] Error retrieving plan: %s", err)
363+
364+
// Note: Once the Compliance service (SCC) reaches its end of life, this conditional check can be revisited or safely removed.
365+
if *instance.ResourceID == "compliance" {
366+
d.Set("plan", "security-compliance-center-standard-plan")
367+
} else {
368+
plan, _, err := globalClient.GetCatalogEntry(&planOptions)
369+
if err != nil {
370+
return fmt.Errorf("[ERROR] Error retrieving plan: %s", err)
371+
}
372+
d.Set("plan", plan.Name)
360373
}
361-
d.Set("plan", plan.Name)
374+
362375
d.Set("crn", instance.CRN)
363376
tags, err := flex.GetTagsUsingCRN(meta, *instance.CRN)
364377
if err != nil {

ibm/service/resourcecontroller/resource_ibm_resource_instance.go

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -581,13 +581,17 @@ func ResourceIBMResourceInstanceRead(d *schema.ResourceData, meta interface{}) e
581581
}
582582
rsCatRepo := rsCatClient.ResourceCatalog()
583583

584-
serviceOff, err := rsCatRepo.GetServiceName(*instance.ResourceID)
585-
if err != nil {
586-
return fmt.Errorf("[ERROR] Error retrieving service offering: %s", err)
584+
// Note: Once the Compliance service (SCC) reaches its end of life, this conditional check can be revisited or safely removed.
585+
if *instance.ResourceID == "compliance" {
586+
d.Set("service", *instance.ResourceID)
587+
} else {
588+
serviceOff, err := rsCatRepo.GetServiceName(*instance.ResourceID)
589+
if err != nil {
590+
return fmt.Errorf("[ERROR] Error retrieving service offering: %s", err)
591+
}
592+
d.Set("service", serviceOff)
587593
}
588594

589-
d.Set("service", serviceOff)
590-
591595
d.Set(flex.ResourceName, instance.Name)
592596
d.Set(flex.ResourceCRN, instance.CRN)
593597
d.Set(flex.ResourceStatus, instance.State)
@@ -599,11 +603,17 @@ func ResourceIBMResourceInstanceRead(d *schema.ResourceData, meta interface{}) e
599603
}
600604
d.Set(flex.ResourceControllerURL, rcontroller+"/services/")
601605

602-
servicePlan, err := rsCatRepo.GetServicePlanName(*instance.ResourcePlanID)
603-
if err != nil {
604-
return fmt.Errorf("[ERROR] Error retrieving plan: %s", err)
606+
// Note: Once the Compliance service (SCC) reaches its end of life, this conditional check can be revisited or safely removed.
607+
if *instance.ResourceID == "compliance" {
608+
d.Set("plan", "security-compliance-center-standard-plan")
609+
} else {
610+
servicePlan, err := rsCatRepo.GetServicePlanName(*instance.ResourcePlanID)
611+
if err != nil {
612+
return fmt.Errorf("[ERROR] Error retrieving plan: %s", err)
613+
}
614+
d.Set("plan", servicePlan)
605615
}
606-
d.Set("plan", servicePlan)
616+
607617
d.Set("guid", instance.GUID)
608618
// ### Modificataion : Setting "onetime_credentials"
609619
d.Set("onetime_credentials", instance.OnetimeCredentials)

0 commit comments

Comments
 (0)