[dev.icinga.com #9184] Add timestamp support for GelfWriter #2970
Comments
Updated by mfriedrich on 2015-04-27 09:22:22 +00:00
|
Updated by mfriedrich on 2015-04-27 09:22:37 +00:00
|
Updated by mfriedrich on 2015-10-27 20:55:23 +00:00
|
Updated by mfriedrich on 2016-04-07 10:45:26 +00:00 Any news? :) |
Updated by mariussturm on 2016-04-11 06:57:34 +00:00 @dnsmichi could you please review and merge branch `9184-gelf-timestamp`? |
Updated by mfriedrich on 2016-04-11 13:32:24 +00:00
Hi Marius, I've seen that you've also pushed that fix into the git master - please don't do that, we'll take care about that when reviewing your branch. It is now resetted/removed. Those commits should always contain a "refs #" in the commit message. The subject would start with a capital letter too. The rest looks good, merged, thanks. Kind regards, |
Updated by mariussturm on 2016-04-11 13:36:17 +00:00 Urgs, sorry that was not the intention. Tried to push to a new branch, guess something went wrong there. |
Updated by Anonymous on 2016-04-11 13:40:03 +00:00
Applied in changeset 15cb9c1. |
Updated by gbeutner on 2016-04-13 10:25:50 +00:00 Please remove your branches once they've been merged into the master branch. |
This issue has been migrated from Redmine: https://dev.icinga.com/issues/9184
Created by mfriedrich on 2015-04-27 09:22:17 +00:00
Assignee: mariussturm
Status: Resolved (closed on 2016-04-11 13:40:03 +00:00)
Target Version: 2.5.0
Last Update: 2016-04-13 10:25:50 +00:00 (in Redmine)
Similar to GraphiteWriter in #8374 supporting the actual timestamp provided by the check result when available.
Changesets
2016-04-11 12:48:16 +00:00 by (unknown) 15cb9c1
2016-04-15 19:17:11 +00:00 by gbeutner e673909
Relations:
The text was updated successfully, but these errors were encountered: