Skip to content

Add JDWP debug configuration for alfresco-repository#543

Open
korutx wants to merge 2 commits intoAlfresco:mainfrom
microboxlabs:feature/jdwp-debug-configuration
Open

Add JDWP debug configuration for alfresco-repository#543
korutx wants to merge 2 commits intoAlfresco:mainfrom
microboxlabs:feature/jdwp-debug-configuration

Conversation

@korutx
Copy link

@korutx korutx commented Jan 31, 2026

Summary

  • Add configuration options to enable remote debugging via JDWP agent for the alfresco-repository chart
  • When enabled, this exposes a debug port and automatically configures the -agentlib:jdwp JVM argument in CATALINA_OPTS
  • Follows the same pattern as existing SMTP and IMAP configuration

Configuration

configuration:
  debug:
    enabled: true      # Enable/disable debug mode (default: false)
    port: 8888         # Debug port (default: 8888)
    suspend: "n"       # Suspend JVM on startup waiting for debugger (default: "n")

Changes

  • values.yaml: Added configuration.debug section with enabled, port, and suspend options
  • configmap-repository.yaml: Added conditional JDWP agentlib configuration to CATALINA_OPTS
  • deployment.yaml: Added conditional debug port exposure in container ports

Test plan

  • Deploy with configuration.debug.enabled: false (default) - verify no debug port or agentlib config
  • Deploy with configuration.debug.enabled: true - verify debug port is exposed and agentlib is in CATALINA_OPTS
  • Test remote debugger connection to the exposed port
  • Verify suspend: "y" pauses JVM startup until debugger connects

Made with Cursor

korutx and others added 2 commits January 31, 2026 16:09
Add configuration options to enable remote debugging via JDWP agent.
When enabled, this exposes a debug port and automatically configures
the -agentlib:jdwp JVM argument in CATALINA_OPTS.

Configuration options:
- configuration.debug.enabled: Enable/disable debug mode
- configuration.debug.port: Debug port (default: 8888)
- configuration.debug.suspend: Suspend JVM on startup (default: "n")

Co-authored-by: Cursor <cursoragent@cursor.com>
@gionn
Copy link
Member

gionn commented Feb 4, 2026

Hello @korutx, thanks for raising this!

Would you mind adding an unit test to cover this specific feature, bump the chart version to next minor and update readme (by running pre-commit)?

@gionn gionn self-assigned this Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants