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

Fixes #36 | Add systemd config #39

Merged
merged 1 commit into from
Jan 28, 2016
Merged

Fixes #36 | Add systemd config #39

merged 1 commit into from
Jan 28, 2016

Conversation

janisz
Copy link
Contributor

@janisz janisz commented Jan 27, 2016

systemd doesn't forward output to file so I added config option to write logs to file.

@@ -73,6 +73,7 @@ consul-tag | `marathon` | Common tag name added to every
listen | `:4000` | Accept connections at this address
log-level | `info` | Log level: panic, fatal, error, warn, info, or debug
log-format | `text` | Log format: JSON, text
log-format | | Save logs to file (e.g.: `/var/log/marathon-consul.log`). If empty logs are published to STDERR
Copy link
Contributor

Choose a reason for hiding this comment

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

log-format -> log-file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@dankraw
Copy link
Contributor

dankraw commented Jan 27, 2016

👍

@@ -37,6 +37,7 @@
},
"Log": {
"Level": "info",
"Format": "text"
"Format": "text",
"File": ""
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe we should set something like /var/log/marathon-consul.log as default?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

By default it's stderr, it could not work with systemd but works perfect with upstart

janisz added a commit that referenced this pull request Jan 28, 2016
@janisz janisz merged commit afb2e4f into master Jan 28, 2016
@janisz janisz deleted the add_systemd_service branch January 28, 2016 09:52
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.

3 participants