Skip to content

Commit

Permalink
fix(stop_pods): harden logic
Browse files Browse the repository at this point in the history
  • Loading branch information
desaintmartin committed Oct 9, 2021
1 parent c1c4db8 commit 4a8c798
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 10 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# wiremind-kubernetes

## v6.3.5 (2021-10-06)
### Fixes
- stop_pods: harden logic.

## v6.3.4 (2021-10-06)
### Fixes
- Add python 3.10 CI.
Expand Down
2 changes: 1 addition & 1 deletion VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
6.3.4
6.3.5
13 changes: 4 additions & 9 deletions src/wiremind_kubernetes/kubernetes_helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ def is_deployment_stopped(self, deployment_name: str, statefulset: bool = False)

if current_scale > 0:
kind = statefulset and "StatefulSet" or "Deployment"
logger.info("%s %s still has %s living replicas", deployment_name, kind, current_scale)
logger.info("%s %s has %s living replicas", deployment_name, kind, current_scale)
return False
return True

Expand Down Expand Up @@ -317,17 +317,12 @@ def _stop_deployments(self, deployment_dict: Dict[str, int]):
"""
Scale down a dict (deployment_name, expected_scale) of Deployments.
"""
for deployment_name in deployment_dict:
self.scale_down_deployment(deployment_name)
stopped = False
for _ in range(self.SCALE_DOWN_MAX_WAIT_TIME):
for deployment_name in deployment_dict:
stopped = self._are_deployments_stopped(deployment_dict)
if stopped:
break
time.sleep(1)
if stopped:
self.scale_down_deployment(deployment_name)
if self._are_deployments_stopped(deployment_dict):
break
time.sleep(1)
else:
raise Exception("Timed out waiting for pods to be deleted: aborting.")

Expand Down

0 comments on commit 4a8c798

Please sign in to comment.