Skip to content

Commit 34a03fc

Browse files
committed
Fix minor things
1 parent 08312b2 commit 34a03fc

File tree

1 file changed

+5
-4
lines changed
  • lib/src/main/java/com/scalar/admin/kubernetes

1 file changed

+5
-4
lines changed

lib/src/main/java/com/scalar/admin/kubernetes/Pauser.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -149,10 +149,10 @@ void unpauseWithRetry(RequestCoordinator coordinator, int maxRetryCount, TargetS
149149
return;
150150
} catch (Exception e) {
151151
if (++retryCounter >= maxRetryCount) {
152-
// If someone uses this library directly instead of using our CLI, users should handle the
153-
// exception properly. However, this case is a very critical issue. Therefore, we output
154-
// the error message here despite whether the exception is handled or not on the caller
155-
// side.
152+
// Users who directly utilize this library, bypassing our CLI, are responsible for proper
153+
// exception handling. However, this scenario represents a critical issue. Consequently,
154+
// we output the error message here regardless of whether the calling code handles the
155+
// exception.
156156
logger.error(
157157
"Failed to unpause Scalar product. They are still in paused. You must restart related"
158158
+ " pods by using the `kubectl rollout restart deployment {}`"
@@ -176,6 +176,7 @@ TargetSnapshot getTarget() throws PauserException {
176176
return targetSelector.select();
177177
}
178178

179+
@VisibleForTesting
179180
RequestCoordinator getRequestCoordinator(TargetSnapshot target) {
180181
return new RequestCoordinator(
181182
target.getPods().stream()

0 commit comments

Comments
 (0)