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.

Sign up for GitHub

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

Jump to bottom

Add support for log events in Synthtrace #170133

Closed
3 of 4 tasks
gbamparop opened this issue Oct 30, 2023 · 11 comments
Closed
3 of 4 tasks

Add support for log events in Synthtrace #170133

gbamparop opened this issue Oct 30, 2023 · 11 comments
Assignees
@achyutjhunjhunwala
Labels
apm:synthtrace APM Synthtrace Topics Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team Team:obs-ux-logs Observability Logs User Experience Team

Comments

@gbamparop
Copy link
Contributor

gbamparop commented Oct 30, 2023 •
edited by achyutjhunjhunwala
Loading

@kbn/apm-synthtrace started as a tool to generate synthetic APM data which intended to be used for development and testing of the APM app in Kibana.

It can be extended to support logs and that will help the team to switch from fixed es archives to test data generated ad-hoc.

The documentation for Synthtrace can be found here.

Tasks
Preview Give feedback
@gbamparop gbamparop added Team:APM All issues that need APM UI Team support Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Oct 30, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:APM)

@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@achyutjhunjhunwala achyutjhunjhunwala mentioned this issue Oct 31, 2023
Merged
@achyutjhunjhunwala
Copy link
Contributor

The linked PR takes care of the all the listed tasks, except renaming of Synthtrace Client. Thats a wider topic of discussion

@sorenlouv
Copy link
Member

We should rename it obs-synthtrace

@achyutjhunjhunwala
Copy link
Contributor

@sqren Sounds good to me, i will scope it out of the current PR as it will increase the reviewing complexity. A separate PR with the rename would be easy to review

@gbamparop
Copy link
Contributor Author

We should rename it obs-synthtrace

It still has the word trace in, what about obs-synthdata or obs-syntevents?

@sorenlouv sorenlouv mentioned this issue Nov 1, 2023
Closed
@smith
Copy link
Contributor

smith commented Nov 2, 2023

idk I think synthtrace is fine.

@smith smith added apm:synthtrace APM Synthtrace Topics Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Nov 9, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith added Team:obs-ux-logs Observability Logs User Experience Team and removed Team:APM All issues that need APM UI Team support Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 9, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

achyutjhunjhunwala added a commit that referenced this issue Nov 16, 2023
@achyutjhunjhunwala @kibanamachine
@mistic @yngrdyn
[Log Explorer] Add support for log generation in synthtrace (#170107)
db5176b 
Closes - #170133

## Summary

This PR adds support for generating logs using the Synthtrace Client

Changes include

1. Changes to Synthtrace package to support new Logs Client and Log
Class for helper methods
2. [Stateful Tests] - Change to our FTR Context config to inject the new
the Log Synthtrace Client
3. [Serverless Tests] - Injected Synthtrace as a service for serverless
tests.
4. A sample test added to `app.ts` to demonstrate how Synthtrace can be
used to generate Log data in both Stateful and Serverless
5. Add support to generate logs via CLI. 2 scenarios added -
`simple_logs.ts` and `logs_and_metrics.ts`

```
# Live Data
node scripts/synthtrace simple_logs.ts --clean --live

# Static Data
node scripts/synthtrace simple_logs.ts --clean
```

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Tiago Costa <[email protected]>
Co-authored-by: Yngrid Coello <[email protected]>
@botelastic
Copy link

botelastic bot commented May 7, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@botelastic botelastic bot added the stale Used to mark issues that were closed for being stale label May 7, 2024
@sorenlouv
Copy link
Member

sorenlouv commented May 7, 2024 •
edited
Loading

I think this can be closed. No need to keep it open if all that's left is a (potential) rename.
Feel free to re-open if needed.

@botelastic botelastic bot removed the stale Used to mark issues that were closed for being stale label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@achyutjhunjhunwala achyutjhunjhunwala

Labels
apm:synthtrace APM Synthtrace Topics Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@smith @sorenlouv @gbamparop @achyutjhunjhunwala @elasticmachine