Skip to content

Conversation

@max3163
Copy link
Contributor

@max3163 max3163 commented Jan 26, 2017

This PR has 3 new changes for the Backend Listener :

  • Start the scheduler for the InfluxdbListener immediately to send start annotation even if the test take less than 5 sec.
  • Add a new parameter in the Influxdb Backend to put tags. ( i.e Tags are separated by spaces in grafana )
    Tags is put as Influxdb Tag for better query performance on it
  • Add new option in jmeter.properties to ignore BackendListener when validating the thread group of plan

@pmouawad
Copy link
Contributor

Hello,
Thanks for PR.
I'll look into merging it but what exactly is the 3rd point ? Is it to avoid sending to INflux the Validation results ?
Usually it's better to split PR in this case.

Thanks

@max3163
Copy link
Contributor Author

max3163 commented Jan 26, 2017

Hello,

Yes it's exactly to avoid sending data to InfluxDB or Graphite when you just validate one thread group.
Sorry for merging this commits in the PR, next time I will split.

@pmouawad
Copy link
Contributor

I tried the patch and I don't see annotations displayed.
I tried with:
eventTags = TestTag
And
eventTags = Test tag for release version=1.2.3

[
screen shot 2017-01-26 at 15 13 04
screen shot 2017-01-26 at 15 12 45

](url)

@max3163
Copy link
Contributor Author

max3163 commented Jan 26, 2017

In my opinion, your events measurement in Influxdb is "corrupted" by old test and looks like this

image

You have the same entry name as Field and as Tag.
Pls could you test with a clean events measurement ? => " drop measurement events "

@pmouawad
Copy link
Contributor

Hi,
You were right.
I have a second possible issue.
Here is what the meanAT shows. It does not look ok to me .
screen shot 2017-01-26 at 15 51 03

@max3163
Copy link
Contributor Author

max3163 commented Jan 26, 2017

Strange ... could you try with a Group By function in your query like this :

image

You have to put a time equal or above the polling time scheduler of JMeter.

@pmouawad
Copy link
Contributor

This is explained by:

@asfgit asfgit closed this in dcd9fa4 Jan 26, 2017
asfgit pushed a commit that referenced this pull request Jan 26, 2017
Based on #258 from Maxime Chassagneux
Bugzilla Id: 60654

git-svn-id: https://svn.apache.org/repos/asf/jmeter/trunk@1780469 13f79535-47bb-0310-9956-ffa450edef68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants