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

chore: Perf test for otel logging #1318

Merged
merged 39 commits into from
Aug 13, 2024
Merged

chore: Perf test for otel logging #1318

merged 39 commits into from
Aug 13, 2024

Conversation

k15r
Copy link
Contributor

@k15r k15r commented Aug 2, 2024 •
edited
Loading

Description

Changes proposed in this pull request (what was done and why):

  • initial config
  • initial loadtest-setup
  • fix script
  • update config to use batch processor
  • refactor loadtest script to be a bit more generic

Traceability

  • The PR is linked to a GitHub issue.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • If the change is user-facing, the documentation has been adjusted.
  • The feature is unit-tested.
  • The feature is e2e-tested.

@k15r k15r requested a review from a team as a code owner August 2, 2024 09:19
@kyma-bot kyma-bot added the cla: yes Indicates the PR's author has signed the CLA. label Aug 2, 2024
@k15r k15r added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed cla: yes Indicates the PR's author has signed the CLA. labels Aug 2, 2024
@kyma-bot kyma-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cla: yes Indicates the PR's author has signed the CLA. labels Aug 2, 2024
@k15r k15r added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cla: yes Indicates the PR's author has signed the CLA. labels Aug 2, 2024
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Aug 2, 2024
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 9, 2024
@k15r k15r added this to the 1.22.0 milestone Aug 9, 2024
@k15r k15r added kind/chore Categorizes issue or PR as related to a chore. area/logs LogPipeline area/documentation Documentation changes labels Aug 9, 2024
@k15r k15r mentioned this pull request Aug 12, 2024
Closed
@k15r k15r linked an issue Aug 12, 2024 that may be closed by this pull request
[OTLP Log] Gateway performance evaluation #1289
Closed
hisarbalik
hisarbalik previously approved these changes Aug 13, 2024
View reviewed changes
@kyma-bot kyma-bot added the lgtm Looks good to me! label Aug 13, 2024
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Aug 13, 2024
NHingerl
NHingerl reviewed Aug 13, 2024
View reviewed changes
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
docs/contributor/benchmarks/load-test-logs.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot added the lgtm Looks good to me! label Aug 13, 2024
@k15r k15r removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 13, 2024
@kyma-bot kyma-bot merged commit 794ca3a into main Aug 13, 2024
47 checks passed
@a-thaler a-thaler deleted the perf-test-otel-logs branch August 27, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NHingerl NHingerl NHingerl approved these changes

@hisarbalik hisarbalik hisarbalik approved these changes

Assignees

@hisarbalik hisarbalik

@NHingerl NHingerl

Labels
area/documentation Documentation changes area/logs LogPipeline cla: yes Indicates the PR's author has signed the CLA. kind/chore Categorizes issue or PR as related to a chore. lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
1.22.0
Development

Successfully merging this pull request may close these issues.

[OTLP Log] Gateway performance evaluation
4 participants
@k15r @hisarbalik @NHingerl @kyma-bot