Skip to content

Commit

Permalink
do not wipe out existing connection resource requirements on update (#…
Browse files Browse the repository at this point in the history
…10291)

* do not wipe out existing connection resource requirements on update

* format

* do not pull any values from worker configs here

* remove logger
  • Loading branch information
lmossman authored Feb 11, 2022
1 parent df5170a commit 820a9ff
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ public ConnectionRead updateConnection(final ConnectionUpdate connectionUpdate)
.withMemoryRequest(connectionUpdate.getResourceRequirements().getMemoryRequest())
.withMemoryLimit(connectionUpdate.getResourceRequirements().getMemoryLimit()));
} else {
newConnection.withResourceRequirements(workerConfigs.getResourceRequirements());
newConnection.withResourceRequirements(persistedSync.getResourceRequirements());
}

// update sync schedule
Expand Down

0 comments on commit 820a9ff

Please sign in to comment.