Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding debug commands for vertx & spring-boot stacks #5938

Merged
merged 1 commit into from
Aug 10, 2017

Conversation

ibuziuk
Copy link
Member

@ibuziuk ibuziuk commented Aug 8, 2017

Signed-off-by: Ilya Buziuk [email protected]

What does this PR do?

Adds debug commands for 'vert.x' and 'spring-boot' stacks

What issues does this PR fix or reference?

redhat-developer/rh-che#238

Changelog

Adding debug commands for 'vert.x' and 'spring-boot' stacks

Release Notes

Debug commands for 'vert.x' and 'spring-boot' stacks

Docs PR

N / A

@ibuziuk ibuziuk requested a review from ashumilova as a code owner August 8, 2017 10:16
@ibuziuk ibuziuk requested review from a user, davidfestal and sunix August 8, 2017 10:17
@codenvy-ci
Copy link

Can one of the admins verify this patch?

1 similar comment
@codenvy-ci
Copy link

Can one of the admins verify this patch?

@ibuziuk ibuziuk requested a review from benoitf August 8, 2017 10:27
@benoitf
Copy link
Contributor

benoitf commented Aug 8, 2017

commands are ok but it looks like the port for debugging (5005) is not exposed on the docker images.

@ghost
Copy link

ghost commented Aug 8, 2017

@benoitf there's no need to expose it unless you need to remotely debug the app. Otherwise, Che debugger connects to localhost:5005

@benoitf
Copy link
Contributor

benoitf commented Aug 8, 2017

@eivantsov yes I was thinking on remote debugging as well but it's not the primary case.

Copy link
Contributor

@sunix sunix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ibuziuk ibuziuk merged commit 424f8ed into eclipse-che:master Aug 10, 2017
@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Sep 28, 2017
@slemeur slemeur added this to the 5.17.0 milestone Sep 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants