Refactor WorkloadResourceQuantity to use pointers for CPU and Memory fields

Author: rbjornstad

internal/graph/gengql/generated.go

@@ -181,12 +181,12 @@ type WorkloadResourceQuantity {
 	"""
 	The number of CPU cores.
 	"""
-	cpu: Float!
+	cpu: Float
 
 	"""
 	The amount of memory in bytes.
 	"""
-	memory: Int!
+	memory: Int
 }
 
 """

internal/graph/schema/workloads.graphqls

@@ -230,21 +230,21 @@ func (a *Application) Resources() *ApplicationResources {
 	if resources := a.Spec.Resources; resources != nil {
 		if resources.Limits != nil {
 			if q, err := resource.ParseQuantity(resources.Limits.Cpu); err == nil {
-				ret.Limits.CPU = q.AsApproximateFloat64()
+				ret.Limits.CPU = ptr.To(q.AsApproximateFloat64())
 			}
 
 			if m, err := resource.ParseQuantity(resources.Limits.Memory); err == nil {
-				ret.Limits.Memory = m.Value()
+				ret.Limits.Memory = ptr.To(m.Value())
 			}
 		}
 
 		if resources.Requests != nil {
 			if q, err := resource.ParseQuantity(resources.Requests.Cpu); err == nil {
-				ret.Requests.CPU = q.AsApproximateFloat64()
+				ret.Requests.CPU = ptr.To(q.AsApproximateFloat64())
 			}
 
 			if m, err := resource.ParseQuantity(resources.Requests.Memory); err == nil {
-				ret.Requests.Memory = m.Value()
+				ret.Requests.Memory = ptr.To(m.Value())
 			}
 		}
 

internal/workload/application/models.go

@@ -268,21 +268,21 @@ func (j *Job) Resources() *JobResources {
 
 	if resources.Limits != nil {
 		if q, err := resource.ParseQuantity(resources.Limits.Cpu); err == nil {
-			ret.Limits.CPU = q.AsApproximateFloat64()
+			ret.Limits.CPU = ptr.To(q.AsApproximateFloat64())
 		}
 
 		if m, err := resource.ParseQuantity(resources.Limits.Memory); err == nil {
-			ret.Limits.Memory = m.Value()
+			ret.Limits.Memory = ptr.To(m.Value())
 		}
 	}
 
 	if resources.Requests != nil {
 		if q, err := resource.ParseQuantity(resources.Requests.Cpu); err == nil {
-			ret.Requests.CPU = q.AsApproximateFloat64()
+			ret.Requests.CPU = ptr.To(q.AsApproximateFloat64())
 		}
 
 		if m, err := resource.ParseQuantity(resources.Requests.Memory); err == nil {
-			ret.Requests.Memory = m.Value()
+			ret.Requests.Memory = ptr.To(m.Value())
 		}
 	}
 

internal/workload/job/models.go

@@ -91,8 +91,8 @@ type WorkloadResources interface {
 }
 
 type WorkloadResourceQuantity struct {
-	CPU    float64 `json:"cpu"`
-	Memory int64   `json:"memory"`
+	CPU    *float64 `json:"cpu"`
+	Memory *int64   `json:"memory"`
 }
 
 type AuthIntegration interface {