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

udpate otel-collector workflow config #623

Merged
merged 3 commits into from
Jun 26, 2024
Merged

udpate otel-collector workflow config #623

merged 3 commits into from
Jun 26, 2024

Conversation

sallyom
Copy link
Collaborator

@sallyom sallyom commented Jun 25, 2024

updates workflow otel-collector config with mTLS and ROSA endpoint

@sallyom
Copy link
Collaborator Author

sallyom commented Jun 25, 2024

Traces are reaching OpenShift (ROSA) otel-collector, yay! - not sure how useful the trace data is, but now that we have the connection in place, we can update the telemetry collected.
Screenshot 2024-06-24 at 11 14 36 PM

Signed-off-by: sallyom <[email protected]>
@sallyom sallyom force-pushed the otel-trace-workflow branch 4 times, most recently from 37c1cf9 to 3f74c9e Compare June 25, 2024 05:03
@sallyom sallyom changed the title udpate otel-collector workflow config WIP udpate otel-collector workflow config Jun 25, 2024
@sallyom sallyom force-pushed the otel-trace-workflow branch 2 times, most recently from 306e1bb to 642aa80 Compare June 25, 2024 15:51
@sallyom sallyom changed the title WIP udpate otel-collector workflow config udpate otel-collector workflow config Jun 25, 2024
@sallyom
Copy link
Collaborator Author

sallyom commented Jun 25, 2024

I think this is a good start. Workflow jobs (only chatbot for now) create a single trace to track how long the job took to complete
Screenshot 2024-06-25 at 12 12 40 PM

@sallyom sallyom force-pushed the otel-trace-workflow branch from 642aa80 to d45439b Compare June 25, 2024 16:17
@rhatdan
Copy link
Member

rhatdan commented Jun 25, 2024

@Gregory-Pereira @lmilbaum PTAL

Copy link
Collaborator

@Gregory-Pereira Gregory-Pereira left a comment

Choose a reason for hiding this comment

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

/lgtm

@rhatdan rhatdan merged commit 75e03f3 into main Jun 26, 2024
3 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.

3 participants