Revert "feat: PC-13893 Deprecate usage of objective's value field for…

… Composite SLOs (#529)" This reverts commit 0350ad4.
nobl9 · Sep 20, 2024 · 5aa05ba · 5aa05ba
1 parent afe5f88
commit 5aa05ba
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 1 deletion.
diff --git a/internal/manifest/v1alpha/examples/slo_composite.go b/internal/manifest/v1alpha/examples/slo_composite.go
@@ -74,6 +74,7 @@ func (s sloCompositeExample) SLO() v1alphaSLO.SLO {
 				{
 					ObjectiveBase: v1alphaSLO.ObjectiveBase{
 						DisplayName: "User experience",
+						Value:       ptr(0.0),
 						Name:        "user-experience",
 					},
 					BudgetTarget:    ptr(0.95),

diff --git a/manifest/v1alpha/slo/examples/composite-slo.yaml b/manifest/v1alpha/slo/examples/composite-slo.yaml
@@ -29,6 +29,7 @@
     budgetingMethod: Occurrences
     objectives:
     - displayName: User experience
+      value: 0.0
       name: user-experience
       target: 0.95
       composite:
@@ -95,6 +96,7 @@
     budgetingMethod: Occurrences
     objectives:
     - displayName: User experience
+      value: 0.0
       name: user-experience
       target: 0.95
       composite:
@@ -158,6 +160,7 @@
     budgetingMethod: Timeslices
     objectives:
     - displayName: User experience
+      value: 0.0
       name: user-experience
       target: 0.95
       timeSliceTarget: 0.9
@@ -225,6 +228,7 @@
     budgetingMethod: Timeslices
     objectives:
     - displayName: User experience
+      value: 0.0
       name: user-experience
       target: 0.95
       timeSliceTarget: 0.9

diff --git a/manifest/v1alpha/slo/slo.go b/manifest/v1alpha/slo/slo.go
@@ -68,7 +68,7 @@ type Attachment struct {
 // ObjectiveBase base structure representing an objective.
 type ObjectiveBase struct {
 	DisplayName string   `json:"displayName"`
-	Value       *float64 `json:"value,omitempty"`
+	Value       *float64 `json:"value"`
 	Name        string   `json:"name"`
 	NameChanged bool     `json:"-"`
 }

diff --git a/sdk/test_data/reader/expected/composite_v2_slo.tpl.json b/sdk/test_data/reader/expected/composite_v2_slo.tpl.json
@@ -12,6 +12,7 @@
     "objectives": [
       {
         "displayName": "composite-obj",
+        "value": null,
         "name": "composite-obj",
         "target": 0.99,
         "composite": {