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

Create OpenShift dashboard to visualize Collector metrics #2996

Merged
merged 15 commits into from
Jul 18, 2024

Conversation

iblancasa
Copy link
Contributor

Description: this PR adds the creation of OpenShift dashboards to monitor OpenTelemetry Collector instances.

@iblancasa iblancasa requested a review from a team as a code owner May 29, 2024 13:46
main.go Outdated Show resolved Hide resolved
internal/openshift/dashboards/openshift.go Outdated Show resolved Hide resolved
internal/openshift/dashboards/openshift_test.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
@iblancasa
Copy link
Contributor Author

@pavolloffay anything else?

Copy link
Member

@pavolloffay pavolloffay left a comment

Choose a reason for hiding this comment

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

LGTM, I would just document how to maintain the dashboard JSON, How it is created, how it can be modified.

internal/openshift/dashboards/dashboards.go Show resolved Hide resolved
"k8s.io/client-go/kubernetes"
)

// The dashboard is created manually following the syntax from Grafana 5. For development purposes, this dashboard can be created just loading the JSON file
Copy link
Member

Choose a reason for hiding this comment

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

just by

Signed-off-by: Israel Blancas <[email protected]>
main.go Outdated Show resolved Hide resolved
internal/openshift/dashboards/metrics-dashboard.json Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
internal/openshift/dashboards/dashboards.go Outdated Show resolved Hide resolved
internal/openshift/dashboards/dashboards.go Show resolved Hide resolved
@pavolloffay
Copy link
Member

@swiatekm could you please review?

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext: To enable this feature, you need to specify the `--create-openshift-dashboard` argument to the operator
Copy link
Member

Choose a reason for hiding this comment

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

Does it make sense to start openshift specific flags with an --openshift-* prefix?

Copy link
Member

Choose a reason for hiding this comment

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

I think it's a nice idea :)

@frzifus frzifus requested a review from swiatekm July 17, 2024 21:13
@iblancasa
Copy link
Contributor Author

@swiatekm please, can you review?

@jaronoff97 jaronoff97 enabled auto-merge (squash) July 18, 2024 14:01
@jaronoff97 jaronoff97 merged commit d3f105b into open-telemetry:main Jul 18, 2024
33 checks passed
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.

Create OpenShift metrics dashboards from the operator
5 participants