chore: added review comments

query/components.go

@@ -12,6 +12,7 @@ var (
 		"owner",
 		"topology_type",
 		"topology_id",
+		"parent_id",
 		"type", // Deprecated. Use resource_selector.types instead
 	}
 )

query/resource_selector.go

@@ -12,6 +12,7 @@ import (
 	"github.com/flanksource/duty/types"
 	"github.com/google/uuid"
 	"github.com/patrickmn/go-cache"
+	"github.com/samber/lo"
 )
 
 // queryResourceSelector runs the given resourceSelector and returns the resource ids
@@ -64,14 +65,6 @@ func queryResourceSelector(ctx context.Context, resourceSelector types.ResourceS
 		query = query.Where("agent_id = ?", agent.ID)
 	}
 
-	if resourceSelector.ParentID != "" {
-		if resourceSelector.ParentID == "nil" {
-			query = query.Where("parent_id IS NULL")
-		} else {
-			query = query.Where("parent_id = ?", resourceSelector.ParentID)
-		}
-	}
-
 	if len(resourceSelector.LabelSelector) > 0 {
 		labels := collections.SelectorToMap(resourceSelector.LabelSelector)
 		var onlyKeys []string
@@ -90,11 +83,11 @@ func queryResourceSelector(ctx context.Context, resourceSelector types.ResourceS
 
 	if len(resourceSelector.FieldSelector) > 0 {
 		fields := collections.SelectorToMap(resourceSelector.FieldSelector)
-		columnWhereClauses := map[string]string{}
+		columnWhereClauses := map[string]any{}
 		var props models.Properties
 		for k, v := range fields {
 			if collections.Contains(allowedColumnsAsFields, k) {
-				columnWhereClauses[k] = v
+				columnWhereClauses[k] = lo.Ternary[any](v == "nil", nil, v)
 			} else {
 				props = append(props, &models.Property{Name: k, Text: v})
 			}

types/resource_selector.go

@@ -18,9 +18,6 @@ type ResourceSelector struct {
 	//  Additionally, the special "self" value can be used to select resources without an agent.
 	Agent string `yaml:"agent,omitempty" json:"agent,omitempty"`
 
-	// Set this to "nil" to query it as NULL or give a uuid
-	ParentID string `yaml:"parent_id,omitempty" json:"parent_id,omitempty"`
-
 	// Cache directives
 	//  'no-cache' (should not fetch from cache but can be cached)
 	//  'no-store' (should not cache)