Add perfsage SLO reporting listener to the plugin registry#817
Add perfsage SLO reporting listener to the plugin registry#817undera merged 7 commits intoundera:masterfrom
Conversation
|
This plugin ships nothing that is not available in existing JMeter and plugins. Consider joining efforts into existing projects than increasing the fragmentation. That said, your plugin has good instruction on how to add it into JMeter if anyone wants exactly that. |
|
@undera I am a performance engineer and I was actually blocked while developing such scenario using jmeter. AFAIK, there is no native functionality or plugin in jmeter that provides comparison and analysis of results for a specific SLO. There are tools like Taurus (wrapper to jmeter), which actually fulfil this requirement but no native functionality as I said. If you are aware about any such plugin, please do let me know. It will be really helpful. |
|
Jmeter has Assertion concept that addresses what your plugin does. |
|
@undera No, it doesn't. JMeter assertion is specific to 'sampler' or a specific request. Although we can achieve similar behaviour - it won't be an efficient way of doing it. JMeter assertions need to run after each request, it will impact jmeter performance itself. I already tried to use samplers, but didn't look like a good pluggable solution for any performance pipeline. Taurus is another solution which is like my plugin, but it adds another layer of execution. Even before starting on this, I checked all available jmeter plugins and functionality. I wanted to develop something which is jmeter specific, and fills this specific gap. |
|
Fair enough, let's give it a try. Please resolve the conflicts with base file. And also |
SLOReporter is a helper class, not a JMeter TestElement/GUI component. componentClasses should list only discoverable JMeter components; the listener SLOAnalysisListener remains as markerClass and sole component entry. Made-with: Cursor
|
@undera Done. Please check. |
|
It has failed CI - take a look |
Apply json.dumps(indent=2) style so "Check repo JSON formatting" passes. Made-with: Cursor
undera
left a comment
undera
left a comment
There was a problem hiding this comment.
Be modest. You descriptor should be last in the list, not the first.
And please pull from master, I fixed the CI issue there.
|
Move it to the end and I'll merge |
Made-with: Cursor
…ins into add-perfsage-plugin
2 participants
Adds community plugin perfsage-slo-reporter to the catalog.
downloadUrlpoints at the GitHub Release asset;screenshotUrlusesraw.githubusercontent.comfor a stable image URL.