Skip to content

Update default config to ensure logs go to stdout #67

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

Merged
merged 1 commit into from
Nov 8, 2016

Conversation

tianon
Copy link
Member

@tianon tianon commented Nov 3, 2016

No description provided.

@tianon
Copy link
Member Author

tianon commented Nov 3, 2016

As noted in #63 (comment) 👍

@tianon
Copy link
Member Author

tianon commented Nov 3, 2016

Arg, this doesn't actually do it properly either; this just redirects the logs to /usr/share/logstash/logs instead.

@tianon
Copy link
Member Author

tianon commented Nov 3, 2016

I'm not actually convinced this ever worked "properly".

@tianon
Copy link
Member Author

tianon commented Nov 3, 2016

Alright, this does the trick:

Sending Logstash logs to /var/log/logstash which is now configured via log4j2.properties.
The stdin plugin is now waiting for input:
17:19:01.908 [[main]-pipeline-manager] INFO  logstash.pipeline - Starting pipeline {"id"=>"main", "pipeline.workers"=>8, "pipeline.batch.size"=>125, "pipeline.batch.delay"=>5, "pipeline.max_inflight"=>1000}
17:19:01.926 [[main]-pipeline-manager] INFO  logstash.pipeline - Pipeline main started
17:19:02.019 [Api Webserver] INFO  logstash.agent - Successfully started Logstash API endpoint {:port=>9600}
hi
{
    "@timestamp" => 2016-11-03T17:19:15.246Z,
      "@version" => "1",
          "host" => "c787ae30890b",
       "message" => "hi",
          "type" => "stdin"
}

@tianon
Copy link
Member Author

tianon commented Nov 3, 2016

Updated just to have it make a backup of the original log4j2.properties file. 👍

@yosifkit yosifkit merged commit 5c756d0 into docker-library:master Nov 8, 2016
@yosifkit yosifkit deleted the logs branch November 8, 2016 00:42
tianon added a commit to infosiftr/stackbrew that referenced this pull request Nov 14, 2016
- `bash`: 4.4, patch level 5
- `cassandra`: add `jemalloc` (docker-library/cassandra#87)
- `java`: debian 9~b144-1
- `logstash`: update config for stdout logs (docker-library/logstash#67)
- `mongo`: 3.4.0-rc3, reduce `windowsservercore` image size (docker-library/mongo#120)
- `openjdk`: debian 9~b144-1
- `owncloud`: 8.0.16, 8.1.11, 8.2.9, 9.0.6, 9.1.2
- `php`: 5.6.28, 7.0.13, 7.1.0RC6
- `piwik`: 2.17.1, add `opcache` (matomo-org/docker#44)
- `postgres`: add `alpine` variants (docker-library/postgres#216)
- `pypy`: 5.6.0
- `tomcat`: switch from `unstable` to `stretch` for `openssl` (docker-library/tomcat#56)
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.

2 participants