Improve comments

internal/inventory/asset.go

@@ -23,21 +23,37 @@ import (
 	"fmt"
 )
 
+// assetCategory is used to build the document index. Use only numbers, letters and dashes (-)
 type assetCategory string
-type assetSubCategory string
-type assetType string
-type assetSubType string
-type assetCloudProvider string
 
 const (
 	CategoryInfrastructure assetCategory = "infrastructure"
+)
 
+// assetSubCategory is used to build the document index. Use only numbers, letters and dashes (-)
+type assetSubCategory string
+
+const (
 	SubCategoryCompute assetSubCategory = "compute"
+)
+
+// assetType is used to build the document index. Use only numbers, letters and dashes (-)
+type assetType string
 
+const (
 	TypeVirtualMachine assetType = "virtual-machine"
+)
+
+// assetSubType is used to build the document index. Use only numbers, letters and dashes (-)
+type assetSubType string
 
+const (
 	SubTypeEC2 assetSubType = "ec2"
+)
 
+type assetCloudProvider string
+
+const (
 	AwsCloudProvider assetCloudProvider = "aws"
 )
 

internal/inventory/aws/fetcher_ec2.go

@@ -57,7 +57,7 @@ func newEc2Fetcher(logger *logp.Logger, identity *cloud.Identity, cfg aws.Config
 func (e *Ec2Fetcher) Fetch(ctx context.Context, assetChannel chan<- inventory.AssetEvent) {
 	instances, err := e.provider.DescribeInstances(ctx)
 	if err != nil {
-		e.logger.Errorf("Could not list ec2 instances (%v)", err)
+		e.logger.Errorf("Could not list ec2 instances: %v", err)
 		return
 	}