Skip to content

Disable embedded Git metadata extraction - #5242

Merged
nikita-tkachenko-datadog merged 1 commit into
masterfrom
nikita-tkachenko/disable-embedded-git-metadata
May 22, 2023
Merged

Disable embedded Git metadata extraction#5242
nikita-tkachenko-datadog merged 1 commit into
masterfrom
nikita-tkachenko/disable-embedded-git-metadata

Conversation

@nikita-tkachenko-datadog

@nikita-tkachenko-datadognikita-tkachenko-datadog commented May 22, 2023

Copy link
Copy Markdown
Contributor

What Does This Do

Disables embedded Git metadata extraction.

Motivation

Component that extracts embedded Git metadata looks for it in git.properties files that are available on the classpath.
The idea is that customers should configure build system plugins that would add this file to their release artifacts.

The problem is that these files can be found inside 3rd party libraries too, and there is no telling whether a file comes from the customer's jar or from a dependency.

Additional Notes

The feature is disabled temporarily until a long-term solution is agreed upon (most likely the filename will be changed to a non-standard one).

@pr-commenter

Copy link
Copy Markdown

Benchmarks

Parameters

BaselineCandidate
commit1.15.0-SNAPSHOT~16f2761b131.15.0-SNAPSHOT~f57c78e6f4
configbaselinecandidate
See matching parameters
BaselineCandidate
moduleAgentAgent
parentNoneNone

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 22 cases.

See unchanged results
scenarioΔ mean execution_time
scenario:Startup-base-Agentsame
scenario:Startup-base-Agent.startsame
scenario:Startup-base-BytebuddyAgentsame
scenario:Startup-base-GlobalTracersame
scenario:Startup-base-AppSecsame
scenario:Startup-base-Remote Configsame
scenario:Startup-base-Telemetrysame
scenario:Startup-iast-Agentsame
scenario:Startup-iast-Agent.startsame
scenario:Startup-iast-BytebuddyAgentsame
scenario:Startup-iast-GlobalTracersame
scenario:Startup-iast-AppSecsame
scenario:Startup-iast-IASTsame
scenario:Startup-iast-Remote Configsame
scenario:Startup-iast-Telemetrysame
scenario:Startup-waf-Agentsame
scenario:Startup-waf-Agent.startsame
scenario:Startup-waf-BytebuddyAgentsame
scenario:Startup-waf-GlobalTracersame
scenario:Startup-waf-AppSecsame
scenario:Startup-waf-Remote Configsame
scenario:Startup-waf-Telemetrysame

@nikita-tkachenko-datadog
nikita-tkachenko-datadog marked this pull request as ready for review May 22, 2023 10:06
@nikita-tkachenko-datadog
nikita-tkachenko-datadog requested a review from a team as a code ownerMay 22, 2023 10:06
@nikita-tkachenko-datadog
nikita-tkachenko-datadog deleted the nikita-tkachenko/disable-embedded-git-metadata branch May 22, 2023 11:00
@github-actionsgithub-actionsBot added this to the 1.15.0 milestone May 22, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@nikita-tkachenko-datadog@richardstartin