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

[testing-on-gke part 1.2] Support generic workloads in gke load test-run scripts #2268

Merged
merged 4 commits into from
Aug 20, 2024

Conversation

gargnitingoogle
Copy link
Collaborator

@gargnitingoogle gargnitingoogle commented Aug 2, 2024

Description

This adds on top of #2247, #2284 and #2264 .

This is followed up in #2270 .

Link to the issue in case of a bug fix.

NA

Testing details

  1. Manual - NA
  2. Unit tests - NA
  3. Integration tests - NA

Copy link

codecov bot commented Aug 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.70%. Comparing base (14e7813) to head (ed2ee46).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2268      +/-   ##
==========================================
- Coverage   79.10%   78.70%   -0.40%     
==========================================
  Files         105      105              
  Lines       11620    11690      +70     
==========================================
+ Hits         9192     9201       +9     
- Misses       1968     2029      +61     
  Partials      460      460              
Flag Coverage Δ
unittests 78.70% <ø> (-0.40%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 467a393 to 96a1712 Compare August 2, 2024 12:16
@gargnitingoogle gargnitingoogle changed the title Add gke load testing/add new code/v2 [testing-on-gke part 2] Add gke load testing/add new code/v2 Aug 2, 2024
@gargnitingoogle gargnitingoogle changed the title [testing-on-gke part 2] Add gke load testing/add new code/v2 [testing-on-gke part 2] Support generic workloads in gke load test-run scripts Aug 2, 2024
@gargnitingoogle gargnitingoogle marked this pull request as ready for review August 5, 2024 05:24
@gargnitingoogle gargnitingoogle requested a review from a team as a code owner August 5, 2024 05:24
@gargnitingoogle gargnitingoogle requested review from raj-prince and removed request for a team August 5, 2024 05:24
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from 6eb4b67 to a8b17c3 Compare August 5, 2024 11:51
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from 20bf1f5 to aff7324 Compare August 5, 2024 12:10
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from 5c9cd16 to 394d34f Compare August 6, 2024 08:59
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from d9d3ee0 to e4f6c08 Compare August 6, 2024 09:03
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 394d34f to 6469044 Compare August 6, 2024 09:04
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from e4f6c08 to ab78513 Compare August 6, 2024 10:30
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 6469044 to 8360569 Compare August 6, 2024 10:30
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from ab78513 to 9f971fd Compare August 7, 2024 11:29
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from 2f15628 to 3d2b66f Compare August 7, 2024 15:18
@gargnitingoogle gargnitingoogle changed the title [testing-on-gke part 2] Support generic workloads in gke load test-run scripts [testing-on-gke part 1.2] Support generic workloads in gke load test-run scripts Aug 8, 2024
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from 2e21e96 to 0e991c4 Compare August 8, 2024 09:26
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from ff68c9e to 6d7db42 Compare August 8, 2024 09:31
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v1 branch from 0e991c4 to 3b98dcf Compare August 8, 2024 10:05
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 6d7db42 to a430524 Compare August 8, 2024 10:05
Base automatically changed from garnitin/add-gke-load-testing/add-new-code/v1 to master August 8, 2024 13:04
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from adb7a41 to 71570f3 Compare August 8, 2024 15:38
@gargnitingoogle gargnitingoogle removed the request for review from raj-prince August 9, 2024 05:30
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from 97cd4a8 to 65de360 Compare August 12, 2024 06:36
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 65de360 to f6c47eb Compare August 14, 2024 10:48
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch 2 times, most recently from 8fcfb01 to 522cde5 Compare August 19, 2024 08:59
@gargnitingoogle gargnitingoogle force-pushed the garnitin/add-gke-load-testing/add-new-code/v2 branch from 522cde5 to ed2ee46 Compare August 19, 2024 15:14
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.

2 participants