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

Tail Logs From Bottom of File or Find First Error #933

Merged
merged 19 commits into from Apr 21, 2016

Conversation

@Calvinp
Copy link
Contributor

Calvinp commented Mar 3, 2016

Working on tailing the log from the bottom of the file instead of the top.

…ill cause the Mesos backend to send from the end of the file instead of the beginning
@ssalinas ssalinas added the hs_staging label Mar 4, 2016
@Calvinp Calvinp changed the title Tail Logs From Bottom of File (WIP) Tail Logs From Bottom of File Mar 15, 2016
@ssalinas ssalinas modified the milestone: 0.4.12 Mar 18, 2016
Calvinp added 11 commits Mar 29, 2016
…he file. In that case they tail from the first error.

However, the regex that is matched to determine what is and isn't an error is constant; forthcoming is the ability to specify it
…ne in the smtp config

Still forthcoming: Ability to override the default task log error and/or the one in smtp config with one in a request
… specified, this will override the deploy configuration and the global default regex
…the task log error regex is bad
…, defaults for new config items are now in SMTPConfiguration.java
@ssalinas ssalinas modified the milestones: 0.5.0, 0.6.0 Apr 5, 2016
Calvinp added 2 commits Apr 6, 2016
…sitive or not. Defaults to not case sensitive
…e sensitive regexes
@Calvinp Calvinp changed the title Tail Logs From Bottom of File Tail Logs From Bottom of File or Find First Error Apr 6, 2016
@Calvinp Calvinp added the hs_qa label Apr 6, 2016
@Calvinp Calvinp added the hs_stable label Apr 14, 2016
@ssalinas
Copy link
Member

ssalinas commented Apr 15, 2016

Hey @Calvinp seems this has a bunch of extra diff in there, can we merge master into here so the run now prompt diff isn't in this as well? Should be good to merge soon, just want to check the changes again

@Calvinp
Copy link
Contributor Author

Calvinp commented Apr 15, 2016

Done

@ssalinas
Copy link
Member

ssalinas commented Apr 21, 2016

This looks gtg, thanks @Calvinp

@ssalinas ssalinas merged commit 3bc86f3 into master Apr 21, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ssalinas ssalinas deleted the email-tail-logs-from-bottom branch Apr 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.