Commit Graph

5 Commits

Author SHA1 Message Date
cschantz 083d0c5b8b Fix memory capacity calculation to iterate through domains not just users
Problem:
- calculate_server_memory_capacity() showed '0MB required'
- Only iterated through users, called find_fpm_pool_config() with username only
- cPanel uses domain-based pool configs (domain.conf not username.conf)
- Result: No pools found, 0MB calculated

Fix:
- Added nested loop: users → domains
- Pass both username AND domain to find_fpm_pool_config()
- Extract pool name from config file to get actual process memory
- Use get_fpm_memory_usage(pool_name) directly instead of calculate_memory_per_process()
- Added domain to details output format

Changes:
- Lines 745-800: Rewrote user iteration to include domain loop
- Now correctly finds pools like pickledperil.com.conf
- Calculates actual memory usage per pool

Result:
- Memory capacity analysis now shows real data
- Proper OOM risk assessment
2025-12-03 01:23:34 -05:00
cschantz 0238eadf43 Fix integer expression errors in php-analyzer.sh
Problem:
- Lines 435, 447, 457: integer expression expected errors
- convert_to_bytes() returns empty string when input is empty
- Bash arithmetic fails on empty strings: [ "" -lt 128 ]

Fix:
- Added empty checks before all numeric comparisons
- Pattern: [ -n "$var" ] && [ "$var" -lt value ]
- Applied to lines 435, 447, 457

Lines fixed:
- 435: post_bytes vs upload_bytes comparison
- 447: memory_bytes vs 128MB comparison
- 457: error_count > 0 comparison

Result:
- No more integer expression errors
- Script completes domain analysis successfully
2025-12-03 01:16:33 -05:00
cschantz 9ebf06a62e Fix SCRIPT_DIR variable collision in PHP libraries
CRITICAL BUG FIX:

Problem: php-detector.sh and php-analyzer.sh were setting SCRIPT_DIR
which collided with parent script's SCRIPT_DIR variable causing
/lib/lib/ double path bug when sourcing libraries.

Solution:
- Changed SCRIPT_DIR to _LIB_DIR in both php-detector.sh and php-analyzer.sh
- Changed exit 1 to return 1 in sourced libraries (exit kills parent script)

Files modified:
- lib/php-detector.sh: Use _LIB_DIR instead of SCRIPT_DIR
- lib/php-analyzer.sh: Use _LIB_DIR instead of SCRIPT_DIR, return instead of exit

This prevents variable collision when libraries are sourced by modules.
2025-12-03 00:52:44 -05:00
cschantz 257e846685 Add server-wide memory capacity check (Option 9) - Critical OOM prevention
NEW FEATURES:
- Menu Option 9: Check Server Memory Capacity (OOM Risk)
- Calculates total memory if ALL PHP-FPM pools hit max_children
- Identifies servers at risk of Out-Of-Memory (OOM) kills
- Provides balanced memory allocation recommendations

TWO NEW ANALYZER FUNCTIONS:

1. calculate_server_memory_capacity()
   - Iterates through all users/PHP-FPM pools
   - Calculates: max_children × avg_memory_per_process
   - Sums total across all pools
   - Compares to total RAM
   - Returns: total_required|total_ram|percentage|status

   Status Levels:
   - HEALTHY:  <60% RAM (safe)
   - CAUTION:  60-75% RAM (watch)
   - WARNING:  75-90% RAM (risky)
   - CRITICAL: >90% RAM (OOM likely!)

2. calculate_balanced_memory_allocation()
   - Analyzes traffic for each user (requests/minute)
   - Calculates proportional memory allocation
   - Reserves 20% of RAM for system (min 2GB)
   - Distributes remaining RAM based on traffic
   - Returns recommendations: REDUCE / INCREASE / OPTIMAL

   Example output:
   USER     CURRENT_MAX  AVG_MB  TRAFFIC_RPM  RECOMMENDED_MAX  REASON
   user1    50          45MB     120          75              INCREASE (traffic demands)
   user2    100         60MB     10           15              REDUCE (prevent OOM)

MENU OPTION 9 FEATURES:
- Shows total RAM vs required memory
- Displays percentage and color-coded status
- Optional per-user breakdown table
- Optional balanced recommendations
- Interactive: ask user what details to show

USE CASE:
Server has 16GB RAM. 10 users each with max_children=50, avg 50MB/process.
Total required: 10 × 50 × 50MB = 25GB
Percentage: 156% of RAM → CRITICAL!
Result: Server WILL run out of memory and kill processes!

This feature addresses user's request:
"calculating max children and memory allocation and then combining all the
 accounts to see if the memory will hit over the memory cap if at capacity"

CRITICAL for preventing OOM kills on shared hosting servers!
2025-12-02 20:39:20 -05:00
cschantz f74cf1d9ea Phase 2: Add comprehensive PHP analysis engine (lib/php-analyzer.sh)
ANALYSIS CAPABILITIES (12 functions):
- Error log analysis (memory exhausted, max_children, timeouts, slow requests)
- Resource usage calculations (memory per process, optimal max_children)
- Traffic analysis (peak concurrent requests, avg requests/minute)
- OPcache effectiveness analysis (hit rate, memory usage, recommendations)
- Configuration issue detection (security, performance, capacity issues)
- Complete domain analysis reporting

ERROR LOG ANALYSIS:
- analyze_memory_exhausted_errors: Track "Allowed memory size exhausted"
- analyze_max_children_errors: Detect "server reached pm.max_children" (CRITICAL!)
- analyze_slow_requests: Parse slow request logs, track slowest scripts
- analyze_execution_timeout_errors: Find "Maximum execution time exceeded"

RESOURCE CALCULATIONS:
- calculate_memory_per_process: Average KB per PHP-FPM process
- calculate_optimal_max_children: Intelligent calculation based on:
  * Available system memory (total - reserved)
  * Average memory per process
  * 20% safety buffer
  * Minimum sanity checks

TRAFFIC ANALYSIS:
- calculate_peak_concurrent_requests: Peak concurrent from access logs
- calculate_avg_requests_per_minute: Average load over time period

OPCACHE ANALYSIS:
- analyze_opcache_effectiveness: Status, hit rate, memory usage, recommendations
  * Detects if disabled (40-70% perf loss!)
  * Calculates hit rate (should be >90%)
  * Checks wasted memory and cache capacity

ISSUE DETECTION (7 critical checks):
- detect_php_config_issues: Comprehensive configuration validation
  1. post_max_size < upload_max_filesize (CRITICAL - uploads fail)
  2. display_errors = On (HIGH - security risk)
  3. memory_limit too low (MEDIUM - performance issue)
  4. pm.max_children errors (CRITICAL - capacity issue)
  5. Memory exhausted errors (HIGH - need more RAM or optimization)
  6. OPcache disabled or low hit rate (HIGH/MEDIUM - performance)
  7. pm.max_requests = 0 (MEDIUM - memory leaks accumulate)
  8. pm = static on low traffic (LOW - wastes memory)

COMPREHENSIVE REPORTING:
- analyze_domain_php: Complete analysis report including:
  * PHP version detection
  * Configuration hierarchy (4 priority levels)
  * Effective settings (memory, execution, uploads)
  * PHP-FPM pool configuration
  * Resource usage (processes, memory)
  * OPcache status and hit rates
  * Traffic analysis (24h)
  * Error analysis (7 days)
  * Issues detected with severity levels
  * Optimization recommendations with reasoning

HELPER FUNCTIONS:
- convert_to_bytes: Parse human-readable sizes (128M → bytes)

INTEGRATION:
- Uses lib/php-detector.sh for all detection
- Uses lib/system-detect.sh for system info
- All functions exported for use by main optimizer

NEXT PHASE: modules/performance/php-optimizer.sh (interactive menu + apply changes)
2025-12-02 20:28:27 -05:00