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

Make syslog optional #10

Closed
gforcada opened this Issue Mar 24, 2015 · 6 comments

Comments

Projects
None yet
2 participants
@gforcada
Owner

gforcada commented Mar 24, 2015

Referring to #9 a raw haproxy log not redirected to syslog will not contain the syslog slug at the beginning.

Make haproxy_log_analysis be able to handle this.

/cc @jason-ryan

@gforcada gforcada self-assigned this Mar 24, 2015

@jason-ryan

This comment has been minimized.

Show comment
Hide comment
@jason-ryan

jason-ryan Mar 30, 2015

Just wondered whether you had a chance to take a look at this?

Jason

jason-ryan commented Mar 30, 2015

Just wondered whether you had a chance to take a look at this?

Jason

@gforcada

This comment has been minimized.

Show comment
Hide comment
@gforcada

gforcada Apr 5, 2015

Owner

@jason-ryan sorry still not, lots of things to do at life and work... I accept patches though :)

Owner

gforcada commented Apr 5, 2015

@jason-ryan sorry still not, lots of things to do at life and work... I accept patches though :)

@jason-ryan

This comment has been minimized.

Show comment
Hide comment
@jason-ryan

jason-ryan Apr 8, 2015

No problem - afraid Python isn't my forte!

I'll take a quick peek at it though out of curiosity, I assume its just some pattern matching/parsing stuff on the line?

jason-ryan commented Apr 8, 2015

No problem - afraid Python isn't my forte!

I'll take a quick peek at it though out of curiosity, I assume its just some pattern matching/parsing stuff on the line?

@gforcada

This comment has been minimized.

Show comment
Hide comment
@gforcada

gforcada Apr 17, 2015

Owner

@jason-ryan sorry busy days lately... Yes, the regular expression is this one:

https://github.com/gforcada/haproxy_log_analysis/blob/master/haproxy/haproxy_logline.py#L14

The easiest would be to remove the syslog slug at the beginning of HaproxyLogLine._parse_line if there is such slug and if not just proceed.

I will try to work on it on the following days or weeks, but can't promise on it sadly.

Owner

gforcada commented Apr 17, 2015

@jason-ryan sorry busy days lately... Yes, the regular expression is this one:

https://github.com/gforcada/haproxy_log_analysis/blob/master/haproxy/haproxy_logline.py#L14

The easiest would be to remove the syslog slug at the beginning of HaproxyLogLine._parse_line if there is such slug and if not just proceed.

I will try to work on it on the following days or weeks, but can't promise on it sadly.

@gforcada gforcada closed this in 48d3a12 Apr 19, 2015

@gforcada

This comment has been minimized.

Show comment
Hide comment
@gforcada

gforcada Apr 19, 2015

Owner

@jason-ryan I just released version 1.1, can you give it a try? It should work without syslog.

Owner

gforcada commented Apr 19, 2015

@jason-ryan I just released version 1.1, can you give it a try? It should work without syslog.

@jason-ryan

This comment has been minimized.

Show comment
Hide comment
@jason-ryan

jason-ryan Apr 20, 2015

Bingo - tried a few commands and they work - thanks, great work! :-)

[image: photo]
Jason Ryan
VP Engineering

Trustev
Real Time, Online Identity Verification

email: jason.ryan@trustev.com
skype: jason_j_ryan
web: www.trustev.com

Trustev Ltd, 2100 Cork Airport Business Park, Cork, Ireland.

On 19 April 2015 at 20:53, Gil Forcada Codinachs notifications@github.com
wrote:

@jason-ryan https://github.com/jason-ryan I just released version 1.1,
can you give it a try? It should work without syslog.


Reply to this email directly or view it on GitHub
#10 (comment)
.

This message is for the named person's use only. If you received this
message in error, please immediately delete it and all copies and notify
the sender. You must not, directly or indirectly, use, disclose,
distribute, print, or copy any part of this message if you are not the
intended recipient. Any views expressed in this message are those of the
individual sender and not Trustev Ltd. Trustev is registered in Ireland No.
516425 and trades from 2100 Cork Airport Business Park, Cork, Ireland.

jason-ryan commented Apr 20, 2015

Bingo - tried a few commands and they work - thanks, great work! :-)

[image: photo]
Jason Ryan
VP Engineering

Trustev
Real Time, Online Identity Verification

email: jason.ryan@trustev.com
skype: jason_j_ryan
web: www.trustev.com

Trustev Ltd, 2100 Cork Airport Business Park, Cork, Ireland.

On 19 April 2015 at 20:53, Gil Forcada Codinachs notifications@github.com
wrote:

@jason-ryan https://github.com/jason-ryan I just released version 1.1,
can you give it a try? It should work without syslog.


Reply to this email directly or view it on GitHub
#10 (comment)
.

This message is for the named person's use only. If you received this
message in error, please immediately delete it and all copies and notify
the sender. You must not, directly or indirectly, use, disclose,
distribute, print, or copy any part of this message if you are not the
intended recipient. Any views expressed in this message are those of the
individual sender and not Trustev Ltd. Trustev is registered in Ireland No.
516425 and trades from 2100 Cork Airport Business Park, Cork, Ireland.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment