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

docs: Add manual metric load test #757

Merged
merged 16 commits into from
Feb 1, 2024
Merged

docs: Add manual metric load test #757

merged 16 commits into from
Feb 1, 2024

Conversation

hisarbalik
Copy link
Contributor

@hisarbalik hisarbalik commented Jan 30, 2024 •
edited
Loading

Description

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

  • Add metric gateway load test scripts
  • Add metric agent load test scripts
  • Move all load test in a single test script and document (Traces and Metrics)

Changes refer to particular issues, PRs or documents:

Traceability

  • The PR is linked to a GitHub issue.
  • New features have a milestone set.
  • New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
  • The corresponding GitHub issue has a respective area and kind label.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • Adjusted the documentation if the change is user-facing.
  • The feature is unit-tested
  • The feature is e2e-tested

@hisarbalik hisarbalik requested a review from a team as a code owner January 30, 2024 09:30
@kyma-bot kyma-bot added the cla: yes Indicates the PR's author has signed the CLA. label Jan 30, 2024
@hisarbalik hisarbalik 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 Jan 30, 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 Jan 30, 2024
@hisarbalik hisarbalik added area/documentation Documentation changes area/metrics MetricPipeline labels Jan 30, 2024
NHingerl
NHingerl reviewed Jan 31, 2024
View reviewed changes
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
chrkl
chrkl previously approved these changes Jan 31, 2024
View reviewed changes
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jan 31, 2024
NHingerl
NHingerl reviewed Jan 31, 2024
View reviewed changes
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Jan 31, 2024
NHingerl
NHingerl reviewed Jan 31, 2024
View reviewed changes
Copy link
Contributor

@NHingerl NHingerl left a comment

Choose a reason for hiding this comment

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

I missed this before

docs/contributor/telemetry-load-test/README.md Outdated Show resolved Hide resolved
@hisarbalik hisarbalik removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 1, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 1, 2024
@kyma-bot kyma-bot merged commit dfe10a4 into kyma-project:main Feb 1, 2024
7 checks passed
@a-thaler a-thaler added this to the 1.9.0 milestone Feb 2, 2024
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

@chrkl chrkl Awaiting requested review from chrkl

Assignees

@chrkl chrkl

@NHingerl NHingerl

Labels
area/documentation Documentation changes area/metrics MetricPipeline cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
1.9.0
Development

Successfully merging this pull request may close these issues.
5 participants
@hisarbalik @chrkl @NHingerl @a-thaler @kyma-bot