Support for aliases on Infrastructure Resources

.changes/unreleased/Bugfix-20231107-162955.yaml

@@ -0,0 +1,3 @@
+kind: Bugfix
+body: Fix index out of range when returning Service errors
+time: 2023-11-07T16:29:55.13703-05:00

.changes/unreleased/Feature-20231107-163024.yaml

@@ -0,0 +1,3 @@
+kind: Feature
+body: Add support for managing aliases on Infrastructure Resources
+time: 2023-11-07T16:30:24.334158-05:00

opslevel/resource_opslevel_infra.go

@@ -21,7 +21,8 @@
 			"aliases": {
 				Type:        schema.TypeList,
 				Description: "The aliases of the infrastructure resource.",
-				Computed:    true,
+				ForceNew:    false,
+				Optional:    true,
 				Elem:        &schema.Schema{Type: schema.TypeString},
 			},
 			"schema": {
@@ -80,6 +81,33 @@
 	}
 }
 
+func reconcileInfraAliases(d *schema.ResourceData, resource *opslevel.InfrastructureResource, client *opslevel.Client) error {
+	expectedAliases := getStringArray(d, "aliases")
+	existingAliases := resource.Aliases
+	for _, existingAlias := range existingAliases {
+		if stringInArray(existingAlias, expectedAliases) {
+			continue
+		}
+		// Delete
+		err := client.DeleteInfraAlias(existingAlias)
+		if err != nil {
+			return err
+		}
+	}
+	for _, expectedAlias := range expectedAliases {
+		if stringInArray(expectedAlias, existingAliases) {
+			continue
+		}
+		// Add
+		id := opslevel.NewID(resource.Id)
+		_, err := client.CreateAliases(*id, []string{expectedAlias})
+		if err != nil {
+			return err
+		}
+	}
+	return nil
+}
+
 func flattenInfraProviderData(resource *opslevel.InfrastructureResource) []map[string]any {
 	return []map[string]any{{
 		"account": resource.ProviderData.AccountName,
@@ -114,6 +142,12 @@
 		return err
 	}
 	d.SetId(resource.Id)
+
+	err = reconcileInfraAliases(d, resource, client)
+	if err != nil {
+		return err
+	}
+
 	return resourceInfrastructureRead(d, client)
 }
 
@@ -147,7 +181,7 @@
 func resourceInfrastructureUpdate(d *schema.ResourceData, client *opslevel.Client) error {
 	id := d.Id()
 
-	_, err := client.UpdateInfrastructure(id, opslevel.InfraInput{
+	resource, err := client.UpdateInfrastructure(id, opslevel.InfraInput{
 		Schema:   d.Get("schema").(string),
 		Owner:    opslevel.NewID(d.Get("owner").(string)),
 		Provider: expandInfraProviderData(d),
@@ -157,6 +191,13 @@
 		return err
 	}
 
+	if d.HasChange("aliases") {
+		err = reconcileInfraAliases(d, resource, client)
+		if err != nil {
+			return err
+		}
+	}
+
 	d.Set("last_updated", timeLastUpdated())
 	return resourceInfrastructureRead(d, client)
 }

opslevel/resource_opslevel_service.go

@@ -234,7 +234,7 @@
 		}
 		_, err := client.ServiceApiDocSettingsUpdate(string(resource.Id), docPath.(string), source)
 		if err != nil {
-			log.Error().Err(err).Msgf("failed to update service '%s' api doc settings", resource.ManagedAliases[0])
+			log.Error().Err(err).Msgf("failed to update service '%s' api doc settings", resource.Aliases[0])
 		}
 	}
 
@@ -338,9 +338,9 @@
 	}
 
 	if d.HasChange("aliases") {
-		tagsErr := reconcileServiceAliases(d, resource, client)
-		if tagsErr != nil {
-			return tagsErr
+		err = reconcileServiceAliases(d, resource, client)
+		if err != nil {
+			return err
 		}
 	}
 
@@ -367,7 +367,7 @@
 		}
 		_, err := client.ServiceApiDocSettingsUpdate(string(resource.Id), docPath, docSource)
 		if err != nil {
-			log.Error().Err(err).Msgf("failed to update service '%s' api doc settings", resource.ManagedAliases[0])
+			log.Error().Err(err).Msgf("failed to update service '%s' api doc settings", resource.Aliases[0])
 		}
 	}