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

Add new tag to identify scans of tasks debugging from the IDE #311

Open
rolgalan opened this issue May 31, 2024 · 2 comments
Open

Add new tag to identify scans of tasks debugging from the IDE #311

rolgalan opened this issue May 31, 2024 · 2 comments

Comments

@rolgalan
Copy link

rolgalan commented May 31, 2024

Hi! 👋

When a task is executed from the IDE with the debugger enabled, the total build time reported will include all the debugging.

While this is expected, it might introduce some artificial outliers in the data. When looking into the p95, searching for long builds or aggregating results this might induce to a wrong analysis of the build health of a project.

I detected this by debugging some unit tests, which I believe is a pretty common use case when developing.

I noticed that IntellijIdea (checked with Android Studio) attaches 2 environment properties when debugging that are not present in normal runs:

idea.debugger.dispatch.port : 57328
idea.debugger.dispatch.addr : 127.0.0.1

Any of these properties can be used to add a new tag like IDE debugging to identify (and filter out when required) these scans.

I've added a draft PR #312 with a proposal to solve this. Let me know if you want to move it forward with this improvement :)

Thanks!

@erichaagdev
Copy link
Member

Hi, @rolgalan. Thanks for taking the time to create this issue and corresponding PR.

Having an indicator that a build was using a debugger would be a good addition to CCUD.

We need to ensure the solution will work in all cases. It needs to work for the sibling CCUD Maven extension and other IDEs. It's also possible to attach a debugger to the build itself.

After taking a quick look at your PR, I don't see any reason your changes couldn't be ported to the Maven extension, but they don't support debugging from Eclipse.

We will continue having discussions internally about the direction we want to take with these changes and report back.

@rolgalan
Copy link
Author

I don't work with any Maven project, so I was not even aware of this sibling extension.

Thanks for looking into it. Let me know if you want that I open the PR to review or keep it in draft/discard it.

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

No branches or pull requests

2 participants