remove system metrics

apps/app/src/features/opentelemetry/server/custom-metrics/index.ts

@@ -1,2 +1 @@
-export { addSystemMetrics } from './system-metrics';
 export { addApplicationMetrics } from './application-metrics';

apps/app/src/features/opentelemetry/server/custom-metrics/system-metrics.ts

@@ -1,129 +0,0 @@
-import * as os from 'os';
-import * as process from 'process';
-
-import { diag, metrics } from '@opentelemetry/api';
-
-import loggerFactory from '~/utils/logger';
-
-const logger = loggerFactory('growi:opentelemetry:custom-metrics:system-metrics');
-const loggerDiag = diag.createComponentLogger({ namespace: 'growi:custom-metrics:system' });
-
-export interface SystemMetricsConfig {
-  collectionInterval: number;
-}
-
-let lastCpuUsage: NodeJS.CpuUsage | null = null;
-
-function calculateCpuUsage(intervalMs: number): number | null {
-  try {
-    const currentUsage = process.cpuUsage();
-
-    if (lastCpuUsage === null) {
-      lastCpuUsage = currentUsage;
-      return null; // Cannot calculate usage rate on first measurement
-    }
-
-
-    const userDiff = currentUsage.user - lastCpuUsage.user;
-    const systemDiff = currentUsage.system - lastCpuUsage.system;
-    const totalDiff = userDiff + systemDiff;
-
-    // Convert microseconds to seconds and divide by collection interval to calculate usage rate
-    const intervalUs = intervalMs * 1000; // Convert to microseconds
-    const usage = (totalDiff / intervalUs) * 100;
-
-    lastCpuUsage = currentUsage;
-
-    // Clamp to 0-100% range
-    return Math.min(Math.max(usage, 0), 100);
-  }
-  catch (error) {
-    logger.error('Failed to calculate CPU usage', { error });
-    return null;
-  }
-}
-
-function getMemoryInfo(): { used: number; total: number; usagePercent: number } {
-  const totalMem = os.totalmem();
-  const freeMem = os.freemem();
-  const used = totalMem - freeMem;
-  const usagePercent = (used / totalMem) * 100;
-
-  return {
-    used,
-    total: totalMem,
-    usagePercent,
-  };
-}
-
-export function addSystemMetrics(config: SystemMetricsConfig): void {
-  logger.info('Starting system metrics collection');
-
-  const meter = metrics.getMeter('growi-system-metrics', '1.0.0');
-
-  // CPU usage
-  const cpuUsageGauge = meter.createObservableGauge('growi.system.cpu.usage', {
-    description: 'CPU usage percentage',
-    unit: '%',
-  });
-
-  // System memory usage
-  const memoryUsageGauge = meter.createObservableGauge('growi.system.memory.usage', {
-    description: 'System memory usage in bytes',
-    unit: 'bytes',
-  });
-
-  const memoryUsagePercentGauge = meter.createObservableGauge('growi.system.memory.usage_percent', {
-    description: 'System memory usage percentage',
-    unit: '%',
-  });
-  // Process memory usage
-  const processMemoryHeapGauge = meter.createObservableGauge('growi.process.memory.heap.used', {
-    description: 'Process heap memory usage in bytes',
-    unit: 'bytes',
-  });
-
-  const processMemoryRssGauge = meter.createObservableGauge('growi.process.memory.rss', {
-    description: 'Process resident set size in bytes',
-    unit: 'bytes',
-  });
-
-  // Metrics collection callback
-  meter.addBatchObservableCallback(
-    (result) => {
-      try {
-        // CPU usage
-        const cpuUsage = calculateCpuUsage(config.collectionInterval);
-        if (cpuUsage !== null) {
-          result.observe(cpuUsageGauge, cpuUsage, {
-            'host.name': os.hostname(),
-          });
-        }
-
-        // System memory
-        const memoryInfo = getMemoryInfo();
-        result.observe(memoryUsageGauge, memoryInfo.used, {
-          'host.name': os.hostname(),
-        });
-        result.observe(memoryUsagePercentGauge, memoryInfo.usagePercent, {
-          'host.name': os.hostname(),
-        });
-
-        // Process memory
-        const processMemory = process.memoryUsage();
-        result.observe(processMemoryHeapGauge, processMemory.heapUsed, {
-          'process.pid': process.pid.toString(),
-        });
-        result.observe(processMemoryRssGauge, processMemory.rss, {
-          'process.pid': process.pid.toString(),
-        });
-      }
-      catch (error) {
-        loggerDiag.error('Failed to collect system metrics', { error });
-      }
-    },
-    [cpuUsageGauge, memoryUsageGauge, memoryUsagePercentGauge, processMemoryHeapGauge, processMemoryRssGauge],
-  );
-
-  logger.info('System metrics collection started successfully');
-}

apps/app/src/features/opentelemetry/server/node-sdk-configuration.ts

@@ -11,7 +11,7 @@ import { ATTR_SERVICE_INSTANCE_ID } from '@opentelemetry/semantic-conventions/in
 import { getGrowiVersion } from '~/utils/growi-version';
 
 import { httpInstrumentationConfig as httpInstrumentationConfigForAnonymize } from './anonymization';
-import { addApplicationMetrics, addSystemMetrics } from './custom-metrics';
+import { addApplicationMetrics } from './custom-metrics';
 
 type Configuration = Partial<NodeSDKConfiguration> & {
   resource: Resource;
@@ -57,9 +57,6 @@ export const generateNodeSDKConfiguration = (serviceInstanceId?: string, enableA
 
     // add custom metrics
     addApplicationMetrics();
-    addSystemMetrics({
-      collectionInterval: 15000, // 15sec
-    });
   }
 
   if (serviceInstanceId != null) {