Add connection status always online

CHANGELOG.md

@@ -13,12 +13,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - System-wide default welcome message ([#3301])
 - Privacy setting to disable finding users by partial matches of their name or email address ([#2264], [#3316])
 - Hints in admin UI file uploads indicating supported file types and maximum allowed file size ([#3235])
+- Option to set connection status of servers to always online ([#3317], [#3373])
 
 ### Changed
 
 - Allow SVG and WebP images to be used as livestream pause images ([#3235])
 - Accessibility: aria-labels for filter and sort select elements in room tabs ([#3298])
 - Accessibility: Updated aria-label values for buttons and select controls to provide more descriptive context ([#3241], [#3242])
+- Prometheus metric label `pilos_servers_total{status="unhealthy"}` to `pilos_servers_total{status="faulty"}` ([#3373])
+- Connection status terminology in log messages (`unhealthy` to `faulty`; `healthy` to `online`; `old_health` to `old_connection_status`) ([#3373])
 
 ### Fixed
 
@@ -888,6 +891,8 @@ You can find the changelog for older versions there [here](https://github.com/TH
 [#3314]: https://github.com/THM-Health/PILOS/issues/3314
 [#3315]: https://github.com/THM-Health/PILOS/pull/3315
 [#3316]: https://github.com/THM-Health/PILOS/pull/3316
+[#3317]: https://github.com/THM-Health/PILOS/issues/3317
+[#3373]: https://github.com/THM-Health/PILOS/pull/3373
 [unreleased]: https://github.com/THM-Health/PILOS/compare/v4.17.0...develop
 [v3.0.0]: https://github.com/THM-Health/PILOS/releases/tag/v3.0.0
 [v3.0.1]: https://github.com/THM-Health/PILOS/releases/tag/v3.0.1

app/Enums/ServerHealth.php → app/Enums/ServerConnectionStatus.php

@@ -4,12 +4,9 @@
 
 namespace App\Enums;
 
-/**
- * Custom status response codes of the api
- */
-enum ServerHealth: int
+enum ServerConnectionStatus: int
 {
     case ONLINE = 1;
-    case UNHEALTHY = 0;
+    case FAULTY = 0;
     case OFFLINE = -1;
 }

app/Enums/ServerStatus.php

@@ -4,9 +4,6 @@
 
 namespace App\Enums;
 
-/**
- * Custom status response codes of the api
- */
 enum ServerStatus: int
 {
     case DISABLED = -1;

app/Http/Controllers/api/v1/ServerController.php

@@ -107,13 +107,16 @@ public function update(ServerRequest $request, Server $server)
         $server->secret = $request->secret;
         $server->strength = $request->strength;
         $server->status = $request->status;
+        $server->connection_status_always_online = $request->boolean('connection_status_always_online');
 
         $server->error_count = 0;
         $server->recover_count = config('bigbluebutton.server_online_threshold');
 
         // Check if server is online/offline and update usage data
-        $serverService = new ServerService($server);
-        $serverService->updateUsage();
+        if (! $server->connection_status_always_online) {
+            $serverService = new ServerService($server);
+            $serverService->updateUsage();
+        }
 
         $server->save();
 
@@ -134,13 +137,16 @@ public function store(ServerRequest $request)
         $server->secret = $request->secret;
         $server->strength = $request->strength;
         $server->status = $request->status;
+        $server->connection_status_always_online = $request->boolean('connection_status_always_online');
 
         $server->error_count = 0;
         $server->recover_count = config('bigbluebutton.server_online_threshold');
 
         // Check if server is online/offline and update usage data
-        $serverService = new ServerService($server);
-        $serverService->updateUsage();
+        if (! $server->connection_status_always_online) {
+            $serverService = new ServerService($server);
+            $serverService->updateUsage();
+        }
 
         $server->save();
 

app/Http/Requests/ServerRequest.php

@@ -19,6 +19,7 @@ public function rules()
             'secret' => ['required', 'string', 'max:255'],
             'strength' => ['required', 'integer', 'min:1', 'max:10'],
             'status' => ['required', Rule::enum(ServerStatus::class)],
+            'connection_status_always_online' => ['required', 'boolean'],
         ];
 
         if ($this->route('server')) {

app/Http/Resources/ServerResource.php

@@ -41,8 +41,9 @@ public function toArray($request)
             'name' => $this->name,
             'description' => $this->description,
             'strength' => $this->strength,
+            'connection_status_always_online' => $this->connection_status_always_online,
             'status' => $this->status,
-            'health' => $this->health,
+            'connection_status' => $this->connection_status,
             'participant_count' => $this->participant_count,
             'listener_count' => $this->listener_count,
             'voice_participant_count' => $this->voice_participant_count,

app/Models/Server.php

@@ -4,7 +4,7 @@
 
 namespace App\Models;
 
-use App\Enums\ServerHealth;
+use App\Enums\ServerConnectionStatus;
 use App\Enums\ServerStatus;
 use App\Observers\ServerObserver;
 use App\Traits\AddsModelNameTrait;
@@ -22,6 +22,7 @@ class Server extends Model
 
     protected $casts = [
         'strength' => 'integer',
+        'connection_status_always_online' => 'boolean',
         'status' => ServerStatus::class,
         'participant_count' => 'integer',
         'listener_count' => 'integer',
@@ -78,24 +79,24 @@ public function getLogLabel()
         return $this->name.' ('.$this->id.')';
     }
 
-    public function getHealthAttribute(): ?ServerHealth
+    public function getConnectionStatusAttribute(): ?ServerConnectionStatus
     {
         if ($this->status == ServerStatus::DISABLED) {
             return null;
         }
 
-        return self::calcHealth($this->recover_count, $this->error_count);
+        return self::calculateConnectionStatus($this->recover_count, $this->error_count);
     }
 
-    public static function calcHealth(int $recover_count, int $error_count): ServerHealth
+    public static function calculateConnectionStatus(int $recover_count, int $error_count): ServerConnectionStatus
     {
         if ($recover_count >= config('bigbluebutton.server_online_threshold')) {
-            return ServerHealth::ONLINE;
+            return ServerConnectionStatus::ONLINE;
         }
         if ($error_count >= config('bigbluebutton.server_offline_threshold')) {
-            return ServerHealth::OFFLINE;
+            return ServerConnectionStatus::OFFLINE;
         }
 
-        return ServerHealth::UNHEALTHY;
+        return ServerConnectionStatus::FAULTY;
     }
 }

app/Observers/ServerObserver.php

@@ -4,7 +4,7 @@
 
 namespace App\Observers;
 
-use App\Enums\ServerHealth;
+use App\Enums\ServerConnectionStatus;
 use App\Enums\ServerStatus;
 use App\Models\Server;
 use Illuminate\Support\Facades\Log;
@@ -54,28 +54,28 @@ public function updated(Server $server): void
             ]);
         }
 
-        // Check if server health changed
-        $newHealth = Server::calcHealth($server->recover_count, $server->error_count);
-        $previousHealth = Server::calcHealth($server->getOriginal('recover_count'), $server->getOriginal('error_count'));
-        if ($newHealth != $previousHealth) {
-            if ($newHealth == ServerHealth::OFFLINE) {
-                Log::error('Server {server} health changed to offline', [
+        // Check if server connection status changed
+        $newConnectionStatus = Server::calculateConnectionStatus($server->recover_count, $server->error_count);
+        $previousConnectionStatus = Server::calculateConnectionStatus($server->getOriginal('recover_count'), $server->getOriginal('error_count'));
+        if ($newConnectionStatus != $previousConnectionStatus) {
+            if ($newConnectionStatus == ServerConnectionStatus::OFFLINE) {
+                Log::error('Server {server} changed to offline', [
                     'server' => $server->getLogLabel(),
-                    'old_health' => $previousHealth->name,
+                    'old_connection_status' => $previousConnectionStatus->name,
                 ]);
             }
 
-            if ($newHealth == ServerHealth::UNHEALTHY) {
-                Log::warning('Server {server} health changed to unhealthy', [
+            if ($newConnectionStatus == ServerConnectionStatus::FAULTY) {
+                Log::warning('Server {server} changed to faulty', [
                     'server' => $server->getLogLabel(),
-                    'old_health' => $previousHealth->name,
+                    'old_connection_status' => $previousConnectionStatus->name,
                 ]);
             }
 
-            if ($newHealth == ServerHealth::ONLINE) {
-                Log::notice('Server {server} health changed to healthy', [
+            if ($newConnectionStatus == ServerConnectionStatus::ONLINE) {
+                Log::notice('Server {server} changed to online', [
                     'server' => $server->getLogLabel(),
-                    'old_health' => $previousHealth->name,
+                    'old_connection_status' => $previousConnectionStatus->name,
                 ]);
             }
         }

app/Prometheus/Collectors/ServerCollector.php

@@ -30,6 +30,6 @@ public function collect(): void
             ->set(Server::where('status', ServerStatus::ENABLED)
                 ->where('recover_count', '<', config('bigbluebutton.server_online_threshold'))
                 ->where('error_count', '<', config('bigbluebutton.server_offline_threshold'))
-                ->count(), ['unhealthy']);
+                ->count(), ['faulty']);
     }
 }

app/Services/ServerService.php

@@ -4,7 +4,7 @@
 
 namespace App\Services;
 
-use App\Enums\ServerHealth;
+use App\Enums\ServerConnectionStatus;
 use App\Enums\ServerStatus;
 use App\Models\Meeting;
 use App\Models\MeetingStat;
@@ -87,7 +87,11 @@ private function getBBBVersion(): ?string
      */
     public function handleApiCallFailed()
     {
-        if ($this->server->health != ServerHealth::OFFLINE) {
+        if ($this->server->connection_status_always_online) {
+            return;
+        }
+
+        if ($this->server->connection_status != ServerConnectionStatus::OFFLINE) {
             $this->server->error_count++;
         }
 
@@ -96,23 +100,25 @@ public function handleApiCallFailed()
         $this->server->timestamps = false;
         $this->server->save();
 
-        if ($this->server->health == ServerHealth::OFFLINE) {
+        if ($this->server->connection_status == ServerConnectionStatus::OFFLINE) {
             $this->setMeetingsDetached();
         }
     }
 
     public function handleApiCallSuccessful()
     {
-        if ($this->server->health != ServerHealth::ONLINE) {
-            $this->server->recover_count++;
-        }
+        if (! $this->server->connection_status_always_online) {
+            if ($this->server->connection_status != ServerConnectionStatus::ONLINE) {
+                $this->server->recover_count++;
+            }
 
-        if ($this->server->health == ServerHealth::ONLINE) {
-            $this->server->error_count = 0;
-        }
+            if ($this->server->connection_status == ServerConnectionStatus::ONLINE) {
+                $this->server->error_count = 0;
+            }
 
-        $this->server->timestamps = false;
-        $this->server->save();
+            $this->server->timestamps = false;
+            $this->server->save();
+        }
 
         $this->endDetachedMeetings();
 
@@ -206,7 +212,7 @@ public function updateUsage($updateServerStatistics = false, $updateMeetingStati
                 $this->server->stats()->save($serverStat);
             }
 
-            if ($this->server->health == ServerHealth::OFFLINE) {
+            if ($this->server->connection_status == ServerConnectionStatus::OFFLINE) {
                 // Clear current live server status
                 $this->server->participant_count = null;
                 $this->server->listener_count = null;

database/factories/ServerFactory.php

@@ -34,6 +34,7 @@ public function definition()
             'version' => '2.4.5',
             'strength' => 1,
             'load' => 0,
+            'connection_status_always_online' => false,
         ];
     }
 }

database/migrations/2026_07_17_000000_add_connection_status_always_online_to_servers_table.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types=1);
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     */
+    public function up(): void
+    {
+        Schema::table('servers', function (Blueprint $table) {
+            $table->boolean('connection_status_always_online')->default(false)->after('strength');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     */
+    public function down(): void
+    {
+        Schema::table('servers', function (Blueprint $table) {
+            $table->dropColumn('connection_status_always_online');
+        });
+    }
+};

database/migrations/migrate-to-v4/2024_03_06_152142_add_health_counters_to_servers_table.php

@@ -20,24 +20,21 @@ public function up(): void
             $table->integer('recover_count')->default(0);
         });
 
-        // Migrate the status column to health counters and status
+        // Migrate the status column to connection status counters and status
         foreach (Server::all() as $server) {
 
             switch ($server->getRawOriginal('status')) {
-                // Disabled
-                case -1:
+                case -1: // Disabled
                     $server->status = ServerStatus::DISABLED;
                     break;
-                    // Offline
-                case 0:
-                    // Server is unhealthy, but not offline yet
+                case 0: // Offline
+                    // Server is faulty, but not offline yet
                     $server->recover_count = 0;
                     $server->error_count = 0;
                     $server->status = ServerStatus::ENABLED;
                     break;
-                    // Online
-                case 1:
-                    // Server is healthy
+                case 1: // Online
+                    // Server is online
                     $server->recover_count = config('bigbluebutton.server_online_threshold');
                     $server->error_count = 0;
                     $server->status = ServerStatus::ENABLED;

docs/static/grafana-dashboard.json

@@ -604,7 +604,7 @@
           {
             "matcher": {
               "id": "byName",
-              "options": "pilos_servers_total{instance=\"app:80\", job=\"prometheus\", status=\"unhealthy\"}"
+              "options": "pilos_servers_total{instance=\"app:80\", job=\"prometheus\", status=\"faulty\"}"
             },
             "properties": [
               {
@@ -713,12 +713,12 @@
           },
           "disableTextWrap": false,
           "editorMode": "builder",
-          "expr": "pilos_servers_total{status=\"unhealthy\"}",
+          "expr": "pilos_servers_total{status=\"faulty\"}",
           "fullMetaSearch": false,
           "hide": false,
           "includeNullMetadata": true,
           "range": true,
-          "refId": "Unhealthy",
+          "refId": "Faulty",
           "useBackend": false
         }
       ],

lang/en/admin.php

@@ -226,6 +226,7 @@
         'draining' => 'Draining',
         'edit' => 'Edit server :name',
         'enabled' => 'Enabled',
+        'faulty' => 'Faulty',
         'flash' => [
             'panic' => [
                 'description_meetings_successful' => '{0} No meetings were successfully stopped. |{1} 1 meeting was successfully stopped. |[2,*] :count meetings were successfully stopped.',
@@ -259,9 +260,10 @@
         'status' => 'Status',
         'strength' => 'Server strength',
         'strength_description' => 'Load balancing factor; the higher the factor, the more participants and meetings the server can handle',
+        'connection_status_always_online' => 'Connection always online',
+        'connection_status_always_online_description' => 'The server is always considered online and available for new meetings; connection issues are ignored',
         'test_connection' => 'Test connection',
         'tile_description' => 'The servers provide the BBB infrastructure for the meetings.',
-        'unhealthy' => 'Faulty',
         'unknown' => 'Unknown',
         'usage_info' => 'The usage (meetings, participants, videos) also contains meetings that are managed by other systems.',
         'version' => 'Version',