MohsenAbrishami / stethoscope

src/Commands/MonitorCommand.php

<?php

namespace MohsenAbrishami\Stethoscope\Commands;

use Illuminate\Console\Command;
use MohsenAbrishami\Stethoscope\Events\TroubleOccurred;
use MohsenAbrishami\Stethoscope\LogRecord\Facades\Record;
use MohsenAbrishami\Stethoscope\Services\Cpu;
use MohsenAbrishami\Stethoscope\Services\storage;
use MohsenAbrishami\Stethoscope\Services\Memory;
use MohsenAbrishami\Stethoscope\Services\Network;
use MohsenAbrishami\Stethoscope\Services\WebServer;

class MonitorCommand extends Command
{
    public function __construct(Cpu $cpu, Memory $memory, Network $network, WebServer $webServer, storage $storage)
    {
        parent::__construct();

        $this->cpu = $cpu;

The property cpu does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->memory = $memory;

The property memory does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->network = $network;

The property network does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->webServer = $webServer;

The property webServer does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->storage = $storage;

The property storage does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'stethoscope:monitor';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'monitor memory usage, cpu usage, network connection and nginx status';

    /**
     * Execute the console command.
     *
     * @return int
     */
    public function handle()
    {
        $logs = [];

        $cpuUsage = $this->cpu->check();
        $memoryUsage = $this->memory->check();
        $networkStatus = $this->network->check();
        $storageFreeSpace = $this->storage->check();
        $webServerStatuses = $this->webServer->check();

        if (config('stethoscope.monitorable_resources.cpu') && $cpuUsage > config(('stethoscope.thresholds.cpu'))) {
            $logs['cpu'] = $cpuUsage;
        }

        if ($memoryUsage > config(('stethoscope.thresholds.memory')) && config('stethoscope.monitorable_resources.memory')) {
            $logs['memory'] = $memoryUsage;
        }

        if ($networkStatus == 'disconnected' && config('stethoscope.monitorable_resources.network')) {
            $logs['network'] = $networkStatus;
        }

        if ($storageFreeSpace < config(('stethoscope.thresholds.storage')) && config('stethoscope.monitorable_resources.storage')) {
            $logs['storage'] = $storageFreeSpace;
        }

        if ($webServerStatuses != 'active' && config('stethoscope.monitorable_resources.web_server')) {
            $logs['webServer'] = $webServerStatuses;
        }

        Record::record($logs);

The call to MohsenAbrishami\Stethoscope\LogRecord\Facades\Record::record() has too few arguments starting with memoryUsage.

$logs of type array is incompatible with the type integer expected by parameter $cpuUsage of MohsenAbrishami\Stethoscope\LogRecord\Facades\Record::record().

        if (! empty($logs)) {
            $logs['signature'] = $this->signature;
            TroubleOccurred::dispatch($logs);
        }
    }
}