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 an Ansible role for OpenTelemetry Collector #138

Merged
merged 24 commits into from
Mar 12, 2024
Merged

Conversation

ishanjainn
Copy link
Member

No description provided.

roles/otel_collector/handlers/main.yml Outdated Show resolved Hide resolved
roles/otel_collector/handlers/main.yml Outdated Show resolved Hide resolved
roles/otel_collector/tasks/configure.yml Outdated Show resolved Hide resolved
roles/otel_collector/tasks/service.yml Outdated Show resolved Hide resolved
roles/otel_collector/tasks/service.yml Outdated Show resolved Hide resolved
roles/otel_collector/defaults/main.yml Outdated Show resolved Hide resolved
roles/otel_collector/defaults/main.yml Outdated Show resolved Hide resolved
@ishanjainn
Copy link
Member Author

@gardar also added you as a CODEOWNER for the Grafana Agent role as I've seen you have been reviewing a few PRs and its great!

Copy link
Collaborator

@gardar gardar left a comment

Choose a reason for hiding this comment

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

How about adding molecule scenarios for testing the role?
You can take a look at the grafana server role for inspiration.

@ishanjainn
Copy link
Member Author

How about adding molecule scenarios for testing the role?
You can take a look at the grafana server role for inspiration.

yeah, Wanna add that but was gonna tackle that in a seperate PR so that this can still be released for user testing

@ishanjainn ishanjainn merged commit 962ff9a into main Mar 12, 2024
9 checks passed
@ishanjainn ishanjainn deleted the otel-collector branch March 12, 2024 06:22
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