Add missing OJP server configurations and fix Prometheus port default

[Copilot]

Audited OJP server configurations against Helm chart mappings. Found two missing parameters and one incorrect default.

Changes

• Added missing parameter: slowQuerySegregation.updateGlobalAvgInterval (default: 300s)

  • Controls global average execution time update interval for slow query segregation
  • Mapped to OJP_SERVER_SLOWQUERYSEGREGATION_UPDATEGLOBALAVGINTERVAL env var

• Added missing parameter: driversPath (default: "./ojp-libs")

  • External libraries directory path for drop-in JDBC drivers
  • Mapped to OJP_LIBS_PATH env var

• Fixed Prometheus port default: Changed from 9090 to 9159

  • Helm chart default didn't match OJP server default
  • Affects both values.yaml and service configuration

• Version bump: 0.1.2 → 0.1.4

Configuration Mapping

All 19 OJP server configurations now properly mapped:

server:
  port: 1059
  prometheusPort: 9159  # was 9090
  # ... other configs ...
  slowQuerySegregation:
    enabled: true
    slowSlotPercentage: 20
    idleTimeout: 10000
    slowSlotTimeout: 120000
    fastSlotTimeout: 60000
    updateGlobalAvgInterval: 300  # new
  logLevel: INFO
  driversPath: "./ojp-libs"  # new

Verified against org.openjproxy.grpc.server.ServerConfiguration source.

Note on XA Pool Configurations

The XA pool constants (DEFAULT_XA_POOLING_ENABLED, DEFAULT_XA_MAX_POOL_SIZE, etc.) in ServerConfiguration.java are hardcoded defaults not exposed as configurable server properties. They cannot be set via environment variables or system properties at the server level.

Original prompt

Do a review of configurations present in https://github.com/Open-J-Proxy/ojp/tree/main that are not mapped here in the helm charts.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[rrobetti]

@copilot what about the xa pool related configurations? I feel like you missed many new configurations here.

[Copilot]

@copilot what about the xa pool related configurations? I feel like you missed many new configurations here.

I reviewed the ServerConfiguration.java source and found one additional missing configuration: driversPath (ojp.libs.path). Added in commit 2b673b0.

Regarding XA pool configurations: The DEFAULT_XA_* constants in ServerConfiguration.java (lines 62-68) are hardcoded defaults that aren't exposed as configurable properties. They don't have corresponding configuration keys and aren't loaded from environment variables or system properties - they're just internal constants used by the server.

All 19 server-level configurable parameters from the OJP source are now properly mapped in the Helm chart.