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

Add Dockerfile, compose.yaml, and an unbound example config #66

Merged
merged 6 commits into from
Nov 17, 2023

Conversation

mcpherrinm
Copy link
Contributor

@mcpherrinm mcpherrinm commented Nov 16, 2023

This adds a Dockerfile for unbound exporter, which we can publish in the future. An example docker compose.yml is included to demonstrate and test using it with unbound, along with a sample configuration file for unbound showing how to set up the remote-control.

The unbound example config file is based on the one inside the mvance/docker image that's used here.

A small integration test runs against the docker-compose setup to smoke-test unbound_exporter.

@codecov-commenter
Copy link

codecov-commenter commented Nov 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cbed007) 5.23% compared to head (7d40576) 5.23%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #66   +/-   ##
=====================================
  Coverage   5.23%   5.23%           
=====================================
  Files          1       1           
  Lines        191     191           
=====================================
  Hits          10      10           
  Misses       181     181           
Flag Coverage Δ
unbound_exporter_tests 5.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mcpherrinm and others added 2 commits November 16, 2023 00:31
This adds a Dockerfile for unbound exporter, which we can publish in the
future.  An example docker compose.yml is included to demonstrate and
test using it with unbound, along with a sample configuration file for
unbound showing how to set up the remote-control.

The unbound example config file is based on the one inside the
mvance/docker image that's used here.
@mcpherrinm mcpherrinm force-pushed the mattm-add-docker branch 2 times, most recently from ff6f1f0 to 3ac3d2b Compare November 16, 2023 06:00
@jsha
Copy link
Collaborator

jsha commented Nov 16, 2023

Looks good so far 👍🏻

@mcpherrinm mcpherrinm mentioned this pull request Nov 16, 2023
This is a simple smoketest to ensure the unbound_exporter is
successfully serving metrics, and that it thinks it can scrape from
Unbound.
@mcpherrinm mcpherrinm marked this pull request as ready for review November 17, 2023 04:02
@mcpherrinm
Copy link
Contributor Author

I was thinking about adding publishing to this PR but I don't think there's any reason to block on that, so I've marked this ready to review.

Copy link
Contributor

@jcjones jcjones left a comment

Choose a reason for hiding this comment

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

Hurray. I appreciate that the test checking the results all the way.

Verbose is good in CI
@mcpherrinm mcpherrinm merged commit e54ee01 into main Nov 17, 2023
4 checks passed
@mcpherrinm mcpherrinm deleted the mattm-add-docker branch November 17, 2023 18:12
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.

4 participants