self-review changes

Signed-off-by: Oleg Vasilev <[email protected]>
neondatabase · Jul 8, 2024 · 63c9086 · 63c9086
1 parent 3b379c2
commit 63c9086
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 9 deletions.
diff --git a/pkg/agent/core/state.go b/pkg/agent/core/state.go
@@ -463,7 +463,7 @@ func (s *state) calculateNeonVMAction(
 	desiredTime := logicalTime
 
 	if desiredResources.HasFieldLessThan(s.VM.Using()) {
-		// We are downscaling, so we needed a permit from monitor
+		// We are downscaling, so we needed a permit from the¡ monitor
 		desiredTime = desiredTime.Earliest(s.Monitor.CurrentLogicalTime)
 	}
 

diff --git a/pkg/agent/runner.go b/pkg/agent/runner.go
@@ -659,8 +659,6 @@ func (r *Runner) doNeonVMRequest(
 		Value: desiredLogicalTime,
 	}}
 
-	fmt.Printf("Desired Logical Time: %v\n", desiredLogicalTime)
-
 	patchPayload, err := json.Marshal(patches)
 	if err != nil {
 		panic(fmt.Errorf("Error marshalling JSON patch: %w", err))

diff --git a/pkg/plugin/run.go b/pkg/plugin/run.go
@@ -309,12 +309,7 @@ func (e *AutoscaleEnforcer) handleResources(
 		}),
 	)
 
-	result := api.Resources{
-		VCPU: pod.cpu.Reserved,
-		Mem:  pod.mem.Reserved,
-	}
-
-	return result, 200, nil
+	return api.Resources{VCPU: pod.cpu.Reserved, Mem: pod.mem.Reserved}, 200, nil
 }
 
 func (e *AutoscaleEnforcer) updateMetricsAndCheckMustMigrate(