Add process check (#23)
This patch adds a check for running processes along with tests.
This commit is contained in:
		@@ -113,6 +113,7 @@ def process_result(result, output_format, verbose):
 | 
			
		||||
        method_name
 | 
			
		||||
    )
 | 
			
		||||
    output_formatter(result)
 | 
			
		||||
    sys.exit(result['exit_code'])
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
if __name__ == '__main__':
 | 
			
		||||
 
 | 
			
		||||
							
								
								
									
										79
									
								
								monitorstack/plugins/process.py
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										79
									
								
								monitorstack/plugins/process.py
									
									
									
									
									
										Normal file
									
								
							@@ -0,0 +1,79 @@
 | 
			
		||||
# Copyright 2017, Major Hayden <major@mhtx.net>
 | 
			
		||||
#
 | 
			
		||||
# Licensed under the Apache License, Version 2.0 (the "License");
 | 
			
		||||
# you may not use this file except in compliance with the License.
 | 
			
		||||
# You may obtain a copy of the License at
 | 
			
		||||
#
 | 
			
		||||
#     http://www.apache.org/licenses/LICENSE-2.0
 | 
			
		||||
#
 | 
			
		||||
# Unless required by applicable law or agreed to in writing, software
 | 
			
		||||
# distributed under the License is distributed on an "AS IS" BASIS,
 | 
			
		||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | 
			
		||||
# See the License for the specific language governing permissions and
 | 
			
		||||
# limitations under the License.
 | 
			
		||||
"""Get system uptime."""
 | 
			
		||||
import os
 | 
			
		||||
import platform
 | 
			
		||||
 | 
			
		||||
import click
 | 
			
		||||
 | 
			
		||||
from monitorstack.cli import pass_context
 | 
			
		||||
 | 
			
		||||
import psutil
 | 
			
		||||
 | 
			
		||||
DOC = """Check if a process is running."""
 | 
			
		||||
COMMAND_NAME = 'process'
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
@click.command(COMMAND_NAME, short_help=DOC)
 | 
			
		||||
@click.argument('process_name', nargs=1, type=str, required=True)
 | 
			
		||||
@pass_context
 | 
			
		||||
def cli(ctx, process_name):
 | 
			
		||||
    """Check if a process is running."""
 | 
			
		||||
    setattr(cli, '__doc__', DOC)
 | 
			
		||||
 | 
			
		||||
    output = {
 | 
			
		||||
        'exit_code': 0,
 | 
			
		||||
        'message': 'process check is ok',
 | 
			
		||||
        'measurement_name': 'process',
 | 
			
		||||
        'meta': {
 | 
			
		||||
            'platform': platform.platform(),
 | 
			
		||||
        },
 | 
			
		||||
        'variables': {}
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if check_process(process_name):
 | 
			
		||||
        output['variables'] = {process_name: 1}
 | 
			
		||||
    else:
 | 
			
		||||
        output['exit_code'] = 1
 | 
			
		||||
        output['message'] = '{} failed -- Process {} not found'.format(
 | 
			
		||||
            COMMAND_NAME,
 | 
			
		||||
            process_name
 | 
			
		||||
        )
 | 
			
		||||
        output['variables'] = {process_name: 0}
 | 
			
		||||
 | 
			
		||||
    return output
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
def check_process(process_name):
 | 
			
		||||
    """Check if a process is in the list of cmdlines."""
 | 
			
		||||
    matches = [x for x in get_cmdlines() if process_name in x]
 | 
			
		||||
    return True if len(matches) > 0 else False
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
def get_cmdlines():
 | 
			
		||||
    """Retrieve the cmdline of each process running on the system."""
 | 
			
		||||
    processes = []
 | 
			
		||||
 | 
			
		||||
    # Get our current PID as well as the parent so we can exclude them.
 | 
			
		||||
    current_pid = os.getpid()
 | 
			
		||||
    parent_pid = os.getppid()
 | 
			
		||||
 | 
			
		||||
    for proc in psutil.process_iter():
 | 
			
		||||
        try:
 | 
			
		||||
            if proc.pid not in [current_pid, parent_pid]:
 | 
			
		||||
                processes.append(' '.join(proc.cmdline()))
 | 
			
		||||
        except psutil.NoSuchProcess:
 | 
			
		||||
            pass
 | 
			
		||||
 | 
			
		||||
    return processes
 | 
			
		||||
@@ -1,4 +1,5 @@
 | 
			
		||||
click
 | 
			
		||||
openstacksdk>=0.9.14
 | 
			
		||||
psutil>=5.2.0
 | 
			
		||||
six
 | 
			
		||||
stevedore
 | 
			
		||||
 
 | 
			
		||||
@@ -48,7 +48,6 @@ def mock_get_consumer_usage(self):
 | 
			
		||||
            'id': 1,
 | 
			
		||||
            'name': 'flavor_one',
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
    }]
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
							
								
								
									
										61
									
								
								tests/test_plugin_process.py
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										61
									
								
								tests/test_plugin_process.py
									
									
									
									
									
										Normal file
									
								
							@@ -0,0 +1,61 @@
 | 
			
		||||
# Copyright 2017, Major Hayden <major@mhtx.net>
 | 
			
		||||
#
 | 
			
		||||
# Licensed under the Apache License, Version 2.0 (the "License");
 | 
			
		||||
# you may not use this file except in compliance with the License.
 | 
			
		||||
# You may obtain a copy of the License at
 | 
			
		||||
#
 | 
			
		||||
#     http://www.apache.org/licenses/LICENSE-2.0
 | 
			
		||||
#
 | 
			
		||||
# Unless required by applicable law or agreed to in writing, software
 | 
			
		||||
# distributed under the License is distributed on an "AS IS" BASIS,
 | 
			
		||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | 
			
		||||
# See the License for the specific language governing permissions and
 | 
			
		||||
# limitations under the License.
 | 
			
		||||
"""Tests for the process plugin."""
 | 
			
		||||
 | 
			
		||||
import json
 | 
			
		||||
 | 
			
		||||
from click.testing import CliRunner
 | 
			
		||||
 | 
			
		||||
from monitorstack.cli import cli
 | 
			
		||||
from monitorstack.plugins import process
 | 
			
		||||
 | 
			
		||||
 | 
			
		||||
class TestUptime(object):
 | 
			
		||||
    """Tests for the uptime monitor class."""
 | 
			
		||||
 | 
			
		||||
    def test_run_failure(self):
 | 
			
		||||
        """Ensure the run() method works."""
 | 
			
		||||
        runner = CliRunner()
 | 
			
		||||
        process_name = 'dont-go-chasing-waterfalls'
 | 
			
		||||
        result = runner.invoke(cli, [
 | 
			
		||||
            '-f', 'json',
 | 
			
		||||
            'process', process_name])
 | 
			
		||||
        result_json = json.loads(result.output)
 | 
			
		||||
        assert result_json['variables'] == {process_name: 0}
 | 
			
		||||
        assert result.exit_code == 1
 | 
			
		||||
 | 
			
		||||
    def test_run_success(self):
 | 
			
		||||
        """Ensure the run() method works."""
 | 
			
		||||
        runner = CliRunner()
 | 
			
		||||
        process_name = '/'
 | 
			
		||||
        result = runner.invoke(cli, [
 | 
			
		||||
            '-f', 'json',
 | 
			
		||||
            'process', process_name])
 | 
			
		||||
        result_json = json.loads(result.output)
 | 
			
		||||
        assert result_json['variables'] == {process_name: 1}
 | 
			
		||||
        assert result.exit_code == 0
 | 
			
		||||
 | 
			
		||||
    def test_check_process_success(self, monkeypatch):
 | 
			
		||||
        """Ensure the check_process() method works."""
 | 
			
		||||
        def mock_get_cmdlines():
 | 
			
		||||
            return ['process1', 'process2']
 | 
			
		||||
 | 
			
		||||
        monkeypatch.setattr(process, 'get_cmdlines', mock_get_cmdlines)
 | 
			
		||||
        result = process.check_process('process2')
 | 
			
		||||
        assert result
 | 
			
		||||
 | 
			
		||||
    def test_get_cmdlines(self):
 | 
			
		||||
        """Ensure the get_cmdlines() method works."""
 | 
			
		||||
        cmdlines = process.get_cmdlines()
 | 
			
		||||
        assert isinstance(cmdlines, list)
 | 
			
		||||
		Reference in New Issue
	
	Block a user