Adds constant_tags to ThreadStats

[johnistan]

Added the same constant_tags to the ThreadStats class as I had done to the blocking version.

[yannmh]

Looking great ! Again, thanks @jofusa !

We'll merge your PR shortly so your changes can be shipped with the next release 🚢

[yannmh]

constant_tags are not associated to events in dogstatsd. To be consistent, we should do the same with ThreadStats, what do you think ?

[johnistan]

Or I can add to dogstatsd. We have a use case for it.

On Fri, Oct 16, 2015, 9:12 AM Yann [email protected] wrote:

In datadog/threadstats/base.py
#90 (comment):

@@ -116,6 +122,13 @@ def event(self, title, text, alert_type=None, aggregation_key=None,
'The web server is up again', alert_type='success')
"""
if not self._disabled:

•        # Append all client level tags to every event
  

•        if self.constant_tags:
  

•            if tags:
  

•                tags += self.constant_tags
  

•            else:
  

•                tags = self.constant_tags
  

constant_tags are not associated to events in dogstatsd. To be
consistent, we should do the same with ThreadStats, what do you think ?

—
Reply to this email directly or view it on GitHub
https://github.com/DataDog/datadogpy/pull/90/files#r42253553.

[johnistan]

Added constant tags to base agent. let me know if you like it, or i can rip it out

[yannmh]

Thanks a lot @jofusa ❗
Changes will be released later today in a v0.10.0 version 🚢