Skip to content

reset loggly on every deploy#157

Merged
bkendall merged 5 commits intomasterfrom
add-rsyslog-restart
Sep 28, 2015
Merged

reset loggly on every deploy#157
bkendall merged 5 commits intomasterfrom
add-rsyslog-restart

Conversation

@anandkumarpatel
Copy link
Copy Markdown

  • since we are doing log truncation we need to restart rsyslog each deploy
  • also we save old log file before moving

Comment thread ansible/roles/loggly/tasks/main.yml Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no. bad. the point of log rotation is to keep them small, don't keep them around. also, IT'S IN LOGGLY, WE DON'T NEED TO KEEP IT lol

Comment thread ansible/roles/loggly/tasks/main.yml Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you || here. does stop return not 0 if it already exists?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if service is already stopped this errors.
and I wish service worked here but it does not, that also errors.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(bummer the service module doesn't work 😦)

bkendall added a commit that referenced this pull request Sep 28, 2015
@bkendall bkendall merged commit cd17cd1 into master Sep 28, 2015
@bkendall bkendall deleted the add-rsyslog-restart branch September 28, 2015 18:27
@bkendall bkendall removed their assignment Jul 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants