Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.5.3 utc #184

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

derandreasberger
Copy link

Hello! Please let me know how if you want to make a 5.5.3 build or if I should take care of a proper build myself (we need 5.5.3 due to ES SaaS constraints).

@derandreasberger
Copy link
Author

Please wait with review - it was working in debug-environment, but with the final build on our stage-environment the order seems to be wrong again; I'm currently trying to figure out what's wrong...

@sivasamyk
Copy link
Owner

sivasamyk commented Sep 20, 2017 via email

@derandreasberger
Copy link
Author

@sivasamyk the branch is ok (besides the merge-conflict)! I just verified the live-tail and search works on our stage with enabled utc-flag (it was just that kibana is optimizing/bundling plugins which i was not aware of, so i neede to manually purge the cached bundle-js files for logtrail).
Using one timestamp-field only sounds like a good idea!

Here is the actual configuration I used for testing:
{ "index_patterns" : [ { "es": { "default_index": "nms-logs_*", "allow_url_parameter": false }, "tail_interval_in_seconds": 1, "es_index_time_offset_in_seconds": 0, "display_timezone": "local", "display_timestamp_format": "YYYY-MM-DD HH:mm:ss,SSS", "display_timestamp_force_sort": true, "utc_based_timestamp": true, "max_buckets": 500, "default_time_range_in_days" : 0, "max_hosts": 100, "max_events_to_keep_in_viewer": 5000, "fields" : { "mapping" : { "timestamp" : "log_date", "display_timestamp" : "log_date", "hostname" : "host", "program": "appname", "message": "message" }, "message_format": "{{{env}}} {{{message}}}" } } ] }

As long as the newly introduced flags display_timestamp_force_sort are not active the default-behaviour should not be affected, but my testing was focused on UTC due to our use-case.

@sivasamyk
Copy link
Owner

@derandreasberger Thanks for pull request. Will review and let you know my comments!

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.

None yet

2 participants