[source] Revamp our source install script (part 1) #2198
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[[email protected]] To ease the release of these changes, this commit requires another commit that applies the changes to the source install script. Previsouly logging was done using supervisor & stdout, totally different from the package logging. There are options in the configuration to change the log location, so let's use them instead of doing this trick! In order to do this, this changes were made: * remove supervisor processes logging (except `supervisor` itself) * add a step to configure log location in the source install script In order to clean weird conf location & others: * move `supervisor.conf` from `supervisord/` to `agent/` with `datadog.conf` * update `bin/agent` to reflect this change * remove `bin/info` as it is handled in `bin/agent` now (was probably still here for _legacy_ reasons) * put launchd logs in the same directory as other logs (`logs/`)
[skip ci] Necessary so that it can be used in the source install script. + remove unused variable
This was referenced Jan 13, 2016
remh
pushed a commit
that referenced
this pull request
Jan 21, 2016
…mp-ext-files [source] Revamp our source install script (part 1)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Applies the changes of #1504 to the files other than
setup_agent.shShould be merged before the 5.7.0 release.