Skip to content

Commit

Permalink
Merge pull request #4388 from s-vipin/fix-remove-custom-host-function
Browse files Browse the repository at this point in the history
fix status_cmd in remove_custom_host function
  • Loading branch information
mergify[bot] authored Jan 24, 2025
2 parents fbc6b49 + affa243 commit dde8ee9
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions ceph/rados/serviceability_workflows.py
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ def remove_custom_host(self, host_node_name: str):
Returns:
None | raises exception in case of failure
"""
status_cmd = ""
status_cmd = "ceph orch osd rm status -f json"
try:

def wait_osd_operation_status(status_cmd):
Expand Down Expand Up @@ -286,7 +286,6 @@ def wait_osd_operation_status(status_cmd):
if osd_id in osd_out_list:
osd_utils.osd_remove(self.cluster, osd_id=osd_id, zap=True)
time.sleep(10)
status_cmd = "ceph orch osd rm status -f json"
if wait_osd_operation_status(status_cmd):
log.info("The OSD successfully removed")
else:
Expand Down

0 comments on commit dde8ee9

Please sign in to comment.