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 support to jmeter-influxdb2-listener-plugin #236

Open
3amrSaad opened this issue Oct 30, 2023 · 4 comments
Open

Add support to jmeter-influxdb2-listener-plugin #236

3amrSaad opened this issue Oct 30, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@3amrSaad
Copy link

Add support to jmeter-influxdb2-listener-plugin (https://github.com/mderevyankoaqa/jmeter-influxdb2-listener-plugin) as it generates very useful metrics than the one already existing as it throws the requests raw data in the database so I think this would be a very useful feature to be added

@rabelenda
Copy link
Contributor

Hello, thank you for bringing this into attention. I didn't know about this plugin, is very interesting.

What particular metrics you think are useful or are you using/or wish to use in your tests that the existing influxdb listener does not provide?

I reviewed very briefly the plugin, and it seems that adds latency, connection & processing time and response body (only for errors, and only for certain amount of errors). It also seems that imposes some limitations, eg: java 11+, jmeter 5.6.2, influxdb 2. Most of them look ok, but would require some special comment in user guide to avoid for example people having issues with java 8 or influxdb previous to 2.

Regards

@3amrSaad
Copy link
Author

3amrSaad commented Nov 1, 2023

Hello,

Thank you for your prompt response and for considering the integration of this InfluxDB Plugin into JMeter DSL. I'm delighted to see your interest in this plugin, and I believe it would be a valuable addition.

Here are some specific metrics and use cases where I use this plugin:

  • As you have stated the plugin captures detailed response data, which is essential for diagnosing performance issues. It enables us to track not only basic latency but also the breakdown of latency components (connection time, processing time) for deeper analysis.
  • The ability to store response bodies for error responses allows us to investigate issues more effectively. This is especially useful when we need to understand the root cause of an error and make necessary adjustments to our tests.

So I was hoping to be able to use it through JMeter DSL as I am currently in the process of migrating from JMX Files to it and I have been heavily relying on this plugin and all of our Grafana Dashboards are built based on it, so your support in this would be highly appreciated.

Best Regards

@rabelenda rabelenda added the enhancement New feature or request label Nov 1, 2023
@3amrSaad
Copy link
Author

Hello, Any updates or timeline regarding this feature?

@rabelenda
Copy link
Contributor

Hello, we don't have any timeline or updates regarding this feature. For the time being you can try using the wrapper element, you can build your own dsl element to support this listener, or you can propose some design or even a pr for the new feature.

I am a little hesitant to add something that requires java 11 and only have that feature requiring that right now. I have seen other listeners with similar features that we need to review farther to identify which is the best course of action in the long term. Finally, jmeter team is planning the release of jmeter 6, that I don't know how much it will take to be released, but is planned to drop java 8 support in favor of java 17, which would require JMeter DSL to move to that java version (if we want to stick with updated version of JMeter) and would avoid having one particular feature in the DSL requiring a particular java version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants