Resolved offerings (#2369)

Signed-off-by: Grant Linville <grant@acorn.io>

Author: g-linville

docs/docs/40-admin/03-computeclasses.md

@@ -40,6 +40,7 @@ affinity: # The same affinity fields for Pods
             operator: In
             values:
             - bar
+supportedRegions: ["local"] # should always be set to ["local"]
 ```
 
 If `memory.min`, `memory.max`, `memory.values`, `affinity`, and `tolerations` are not given, then there are no scheduling rules for workloads using the compute class.

pkg/apis/api.acorn.io/v1/types.go

@@ -23,10 +23,7 @@ func (in *App) GetStopped() bool {
 }
 
 func (in *App) GetRegion() string {
-	if in.Spec.Region != "" {
-		return in.Spec.Region
-	}
-	return in.Status.Defaults.Region
+	return in.Status.ResolvedOfferings.Region
 }
 
 type Acornfile v1.AppSpec

pkg/apis/internal.acorn.io/v1/appinstance.go

@@ -14,7 +14,7 @@ type AppInstanceCondition string
 
 var (
 	AppInstanceConditionDefined             = "defined"
-	AppInstanceConditionDefaults            = "defaults"
+	AppInstanceConditionResolvedOfferings   = "resolved-offerings"
 	AppInstanceConditionScheduling          = "scheduling"
 	AppInstanceConditionNamespace           = "namespace"
 	AppInstanceConditionParsed              = "parsed"
@@ -52,23 +52,18 @@ type AppInstance struct {
 }
 
 func (in *AppInstance) HasRegion(region string) bool {
-	return in.Status.Defaults.Region == region || in.Spec.Region == region
+	return in.Status.ResolvedOfferings.Region == region
 }
 
 func (in *AppInstance) GetRegion() string {
-	if in.Spec.Region != "" {
-		return in.Spec.Region
-	}
-	return in.Status.Defaults.Region
+	return in.Status.ResolvedOfferings.Region
 }
 
 func (in *AppInstance) SetDefaultRegion(region string) {
-	if in.Spec.Region == "" {
-		if in.Status.Defaults.Region == "" {
-			in.Status.Defaults.Region = region
-		}
-	} else {
-		in.Status.Defaults.Region = ""
+	if in.Spec.Region != "" {
+		in.Status.ResolvedOfferings.Region = in.Spec.Region
+	} else if in.Status.ResolvedOfferings.Region == "" {
+		in.Status.ResolvedOfferings.Region = region
 	}
 }
 
@@ -209,7 +204,7 @@ type AppInstanceStatus struct {
 	AppStatus                 AppStatus               `json:"appStatus,omitempty"`
 	Scheduling                map[string]Scheduling   `json:"scheduling,omitempty"`
 	Conditions                []Condition             `json:"conditions,omitempty"`
-	Defaults                  Defaults                `json:"defaults,omitempty"`
+	ResolvedOfferings         ResolvedOfferings       `json:"resolvedOfferings,omitempty"`
 	Summary                   CommonSummary           `json:"summary,omitempty"`
 	Permissions               []Permissions           `json:"permissions,omitempty"`               // Permissions given to this appInstance (only containers within, not nested Acorns/Services)
 	DeniedConsumerPermissions []Permissions           `json:"deniedConsumerPermissions,omitempty"` // Permissions given to this appInstance by a consumed service, which it is not authorized to have
@@ -229,19 +224,25 @@ type AppStatusStaged struct {
 	ImageAllowed                  *bool         `json:"imageAllowed,omitempty"`
 }
 
-type Defaults struct {
-	VolumeSize *resource.Quantity       `json:"volumeSize,omitempty"`
-	Volumes    map[string]VolumeDefault `json:"volumes,omitempty"`
-	Memory     map[string]*int64        `json:"memory,omitempty"`
-	Region     string                   `json:"region,omitempty"`
+type ResolvedOfferings struct {
+	Volumes    map[string]VolumeResolvedOffering    `json:"volumes,omitempty"`
+	VolumeSize *resource.Quantity                   `json:"volumeSize,omitempty"`
+	Containers map[string]ContainerResolvedOffering `json:"containers,omitempty"`
+	Region     string                               `json:"region,omitempty"`
 }
 
-type VolumeDefault struct {
+type VolumeResolvedOffering struct {
 	Class       string      `json:"class,omitempty"`
 	Size        Quantity    `json:"size,omitempty"`
 	AccessModes AccessModes `json:"accessModes,omitempty"`
 }
 
+type ContainerResolvedOffering struct {
+	Class     string   `json:"class,omitempty"`
+	Memory    *int64   `json:"memory,omitempty"`
+	CPUScaler *float64 `json:"cpuScaler,omitempty"`
+}
+
 type Scheduling struct {
 	Requirements      corev1.ResourceRequirements `json:"requirements,omitempty"`
 	Affinity          *corev1.Affinity            `json:"affinity,omitempty"`

pkg/apis/internal.acorn.io/v1/zz_generated.deepcopy.go

@@ -31,7 +31,7 @@ ACORNS:
                 },
                 "appSpec": {},
                 "appStatus": {},
-                "defaults": {},
+                "resolvedOfferings": {},
                 "summary": {}
             }
         }

pkg/cli/testdata/TestAll/acorn_all_-o_json.golden

@@ -17,7 +17,7 @@ items:
     appSpec: {}
     appStatus: {}
     columns: {}
-    defaults: {}
+    resolvedOfferings: {}
     staged:
       appImage:
         buildContext: {}

pkg/cli/testdata/TestAll/acorn_all_-o_yaml.golden

@@ -597,12 +597,34 @@ func containerAnnotation(container v1.Container) string {
 	return string(json)
 }
 
+func resolvedOfferingsAnnotation(appInstance *v1.AppInstance, container v1.Container) (string, error) {
+	if resolved, exists := appInstance.Status.ResolvedOfferings.Containers[container.Name]; exists {
+		data, err := convert.EncodeToMap(resolved)
+		if err != nil {
+			return "", err
+		}
+
+		j, err := json.Marshal(data)
+		if err != nil {
+			return "", err
+		}
+
+		return string(j), nil
+	}
+	return "", nil
+}
+
 func podAnnotations(appInstance *v1.AppInstance, container v1.Container) map[string]string {
 	annotations := map[string]string{
 		labels.AcornContainerSpec: containerAnnotation(container),
 	}
 	addPrometheusAnnotations(annotations, container)
 
+	offerings, err := resolvedOfferingsAnnotation(appInstance, container)
+	if err == nil && offerings != "" {
+		annotations[labels.AcornContainerResolvedOfferings] = offerings
+	}
+
 	images := map[string]string{}
 	addImageAnnotations(images, appInstance, container)
 

pkg/controller/appdefinition/deploy.go

@@ -116,7 +116,7 @@ status:
   appSpec: {}
   appStatus: {}
   columns: {}
-  defaults: {}
+  resolvedOfferings: {}
   staged:
     appImage:
       buildContext: {}
@@ -157,7 +157,7 @@ status:
   appSpec: {}
   appStatus: {}
   columns: {}
-  defaults: {}
+  resolvedOfferings: {}
   staged:
     appImage:
       buildContext: {}
@@ -200,8 +200,8 @@ status:
     status: "True"
     success: true
     type: defined
-  defaults: {}
   namespace: app-created-namespace
+  resolvedOfferings: {}
   staged:
     appImage:
       buildContext: {}

pkg/controller/appdefinition/testdata/TestComputeMem.golden

@@ -132,7 +132,7 @@ status:
   appSpec: {}
   appStatus: {}
   columns: {}
-  defaults: {}
+  resolvedOfferings: {}
   staged:
     appImage:
       buildContext: {}
@@ -212,8 +212,8 @@ status:
     status: "True"
     success: true
     type: defined
-  defaults: {}
   namespace: app-created-namespace
+  resolvedOfferings: {}
   staged:
     appImage:
       buildContext: {}