Skip to content

Coralogix processor first PR #33458

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
Merged
merged 50 commits into from
Aug 9, 2024
Merged

Coralogix processor first PR #33458

merged 50 commits into from
Aug 9, 2024

Conversation

galrose
Copy link
Contributor

@galrose galrose commented Jun 10, 2024

Description:
Adding a feature - Adding a feature to create templates (blueprints) from sql queries.
Currently specifically for postgresql and mysql queries.

Link to tracking Issue: #33090

Testing:
currently no tests, will be added in next PR

Documentation:
Added documentation for possible configuration and the usecase of the processor

@galrose galrose requested review from a team and mx-psi June 10, 2024 15:02
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Jun 10, 2024 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@crobert-1
Copy link
Member

This PR should have a changelog entry. 👍

crobert-1
crobert-1 reviewed Jun 10, 2024
View reviewed changes
Copy link
Member

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

Looks good overall. I've left a lot of comments on the README, the goal is to be able to flesh out UI/UX up front, before getting too far into the implementation weeds.

There are also a number of CI/CD failures caused by this PR, so please take a look and resolve when you're able. Let me know if you run into any issues there, happy to help!

galrose and others added 2 commits June 11, 2024 10:54