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

Dockerfile for Metricbeat to test cgroups #2492

Merged
merged 1 commit into from
Sep 13, 2016

Conversation

monicasarbu
Copy link
Contributor

@monicasarbu monicasarbu commented Sep 8, 2016

Add a Dockerfile for Metricbeat to test cgroups functionality.

You need to run the following commands to start a container with Kibana, one with Elasticsearch and one with Metricbeat:

$ make up ENV=5.0.0-cgroups.yml 

To stop all the containers, you can use:

$ make stop ENV=5.0.0-cgroups.yml

@monicasarbu monicasarbu added review in progress Pull request is currently in progress. labels Sep 8, 2016
#-------------------------- Elasticsearch output ------------------------------
output.elasticsearch:
# Array of hosts to connect to.
hosts: ["localhost:9200"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

That is probably not going to work as elasticsearch is not running on localhost but elasticsearch. The environment variables should be used ehre.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The elasticsearch is set in the command for running metricbeat in the Dockerfile-5.0.0-cgroups.

@monicasarbu monicasarbu removed the in progress Pull request is currently in progress. label Sep 9, 2016
@monicasarbu monicasarbu force-pushed the testing_containers_for_cgroups branch from 015ea61 to 641c8c3 Compare September 12, 2016 17:40
enabled: true
period: 10s
processes: ['.*']
cgroups: true
Copy link
Member

Choose a reason for hiding this comment

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

You should be able to use the default configuration file and specify this option on the CLI using -E metricbeat.modules.0.cgroups=true. This was recently fixed by #2521.

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. Thanks @urso for such a fast fix.

@monicasarbu monicasarbu force-pushed the testing_containers_for_cgroups branch 2 times, most recently from 0f2f64d to 2e01905 Compare September 12, 2016 18:06
@@ -10,6 +10,10 @@ stop:
${BASE_COMMAND} stop
${BASE_COMMAND} rm -f


up:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I recommend also to run the build command before running up, to make sure you always have the most recent environment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, you are right. I am adding build before.

run make up ENV=5.0.0-cgroups.yml
@monicasarbu monicasarbu force-pushed the testing_containers_for_cgroups branch from 4e839a5 to ddfcc1a Compare September 13, 2016 09:28
@monicasarbu
Copy link
Contributor Author

I addressed all comments, and it's ready for another review.

@ruflin ruflin merged commit 848ccf7 into elastic:master Sep 13, 2016
@monicasarbu monicasarbu deleted the testing_containers_for_cgroups branch September 13, 2016 10:38
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.

3 participants