BrightcoveOS/Diamond — pull request #146 by bclermont (2012-09-16 09:58:02+00:00)
Title: Fix sentry

While deploy this on some servers I found a small logic bug.

And I took the opportunity to create better report in Sentry, but my handler need to know the hostname.

Because, a single Sentry project can report the alerts from multiple hosts, so I need the hostname in the alert title.

So I move diamond.collector.Collector.get_hostname to diamond.collector.get_hostname

I can now reuse it in my handler.

maybe the base Handler class should also have the get_hostname method too?

maybe move get_hostname in diamond.utils and have both base classes method to call diamond.utils.get_hostname(self.config) ?