Skip to content

Commit

Permalink
[#5246] Platform: Update cluster_health script to screen out non-loca…
Browse files Browse the repository at this point in the history
…l volumes

Summary:
  - Added key `-l` to the `df` command.

Test Plan:
Test scenario:

1. Create a universe;
2. Create an NFS mount point on one of nodes (see instructions in https://phabricator.dev.yugabyte.com/D10787);
3. Wait for the `health-check` is completed;
4. Check that "Disk utilization" section inside the email report doesn't have the mentioned NFS.

Reviewers: daniel

Reviewed By: daniel

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D10863
  • Loading branch information
SergeyPotachev committed Mar 12, 2021
1 parent 347fd54 commit ccfe166
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions managed/devops/bin/cluster_health.py
Expand Up @@ -252,7 +252,7 @@ def _remote_check_output(self, command):
return output

def get_disk_utilization(self):
remote_cmd = 'df -h 2>/dev/null'
remote_cmd = 'df -hl 2>/dev/null'
return self._remote_check_output(remote_cmd)

def check_disk_utilization(self):
Expand Down Expand Up @@ -406,7 +406,7 @@ def check_cqlsh(self):

errors = []
if not ('Connected to local cluster at {}:{}'
.format(self.node, self.ycql_port) in output or
.format(self.node, self.ycql_port) in output or
"AuthenticationFailed('Remote end requires authentication.'" in output):
errors = [output]
return e.fill_and_return_entry(errors, len(errors) > 0)
Expand Down

0 comments on commit ccfe166

Please sign in to comment.