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

Dev mode debug #802

Merged
merged 1 commit into from
Mar 14, 2023
Merged

Dev mode debug #802

merged 1 commit into from
Mar 14, 2023

Conversation

scottkurz
Copy link
Member

@scottkurz scottkurz commented Mar 11, 2023

Fixes #801

@scottkurz scottkurz force-pushed the dev-mode-debug branch 2 times, most recently from d9e184a to a039c53 Compare March 11, 2023 12:51
@scottkurz scottkurz marked this pull request as ready for review March 11, 2023 13:16
@scottkurz
Copy link
Member Author

One thing I noticed in working on this is that sometimes the "port in-use detection" fails, i.e. the ci.common findAvailablePort thinks I can use a port which another JVM is already using as the debug port.

I'm mostly running on Windows, which is the only place I've seen the problem (and haven't seen it on Linux).

@turkeylurkey not sure if you have any insight here. I see there was a later round of work specifically on Windows: OpenLiberty/ci.common#230

Thanks

@turkeylurkey
Copy link
Member

I think @TrevCraw could help with this.

@scottkurz
Copy link
Member Author

Just to be clear... I don't think this PR is making whatever shortcomings may exist in 'findAvailablePort' any worse.. Or, to the extent it is worse, well, it's on a path (dev mode gradle debug) that was completely unusable before.

So this PR could be merged without tackling that question at all if we want.

@turkeylurkey
Copy link
Member

The reason I wanted Trevor to look is that I thought dev mode has a dependency on the preceding steps.

@scottkurz scottkurz merged commit a4b90ad into OpenLiberty:main Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants