Apm: Add INCLUDE_PROCESS_ARGS config option to all agents

Created on 4 Mar 2019  路  3Comments  路  Source: elastic/apm

Description of the issue

The Java agent is currently the only agent that has the INCLUDE_PROCESS_ARGS config option. What this option does is toggle whether or not the process object is added to the metadata event. If disabled, this in turn means that the APM Server will not attach the process information to each transaction and in turn save disk space.

If this makes sense for the Java agent, I don't see why we shouldn't add it to the other agents as well.

What we are voting on

@elastic/apm-agent-devs please vote on if you like to add this config option to your agent. If voting No or N/A, please add a comment explaining 馃槂

Vote

| Agent | Yes | No | Indifferent | N/A | Link to agent issue
| --------|:----:|:---:|:-----------:|:----:|:-------------------:|
| .NET |

  • [x]
|
  • [ ]
|
  • [ ]
|
  • [ ]
|
| Go |
  • [ ]
|
  • [ ]
|
  • [x]
|
  • [ ]
|
| Java |
  • [x]
|
  • [ ]
|
  • [ ]
|
  • [ ]
|
| Node.js |
  • [x]
|
  • [ ]
|
  • [ ]
|
  • [ ]
|
| Python |
  • [x]
|
  • [ ]
|
  • [ ]
|
  • [ ]
|
| Ruby |
  • [ ]
|
  • [ ]
|
  • [x]
|
  • [ ]
|
| RUM |
  • [ ]
|
  • [ ]
|
  • [ ]
|
  • [x]
|

apm-agents poll

Most helpful comment

I seem to recall the reason for this is the Java agent's args list sometimes being hundreds of arguments? I don't immediately see why anyone would want this in Ruby.

Again, I'll bang my good old drum: Let's not add options _just because_. Ready to be voted down 馃槈

All 3 comments

I seem to recall the reason for this is the Java agent's args list sometimes being hundreds of arguments? I don't immediately see why anyone would want this in Ruby.

Again, I'll bang my good old drum: Let's not add options _just because_. Ready to be voted down 馃槈

I seem to recall the reason for this is the Java agent's args list sometimes being hundreds of arguments?

That's correct

We decided on Zoom that this wasn't required for agents to implement, but feel free to do so of course

Was this page helpful?
0 / 5 - 0 ratings

Related issues

alex-fedotyev picture alex-fedotyev  路  12Comments

jianzzz picture jianzzz  路  12Comments

axw picture axw  路  6Comments

bmorelli25 picture bmorelli25  路  5Comments

felixbarny picture felixbarny  路  9Comments