[receiver/jmx] mark receiver as deprecated #9063
Conversation
After many discussions it seems the community is leaning towards removing the components that execute subprocesses. As such, marking the JMX receiver as deprecated. Fixes open-telemetry#6750
|
While I generally agree with the concerns expressed by the community about running subprocesses, I did not think we had reached consensus to universally prohibit them. This receiver provides valuable functionality equivalent to a dozen scrapers and is not easily replaced. Technically, we still have guidelines for running subprocesses as well as documentation describing this type of use case. I don't want to drag on what has already been an intractable issue, but before this component is deprecated, I think we should establish whether we are prohibiting subprocesses in general vs defining stricter standards that allow for subprocesses under certain circumstances. If we are not prohibiting subprocesses in general, then I think #6750 is still a relevant conversation. This comment seems to have articulated a strategy that would go a long way toward securing the component. Certainly there would be more to it as well. I'm happy to push on that thread if we are still open to keeping the component. |
There was a problem hiding this comment.
@djaglowski I've added this to the agenda for the SIG meeting tomorrow. I chatted with some of the maintainers/approvers about this last week and the general sense I got from that thread is that folks preferred moving in the direction of removing the components.
I'll wait until the discussion tomorrow before moving this PR, #9062, and #9058 forward.
|
Closing this PR as per the comment: #6750 (comment) |
After many discussions it seems the community is leaning towards removing the components that execute subprocesses. As such, marking the JMX receiver as deprecated.
Related: #6750
If folks are ok with this path moving forward, I can follow this up with a PR to remove the component from the manifest in the releases repository in the next version.