Included stress test for Bulk API #1458
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@msvinaykumar Can you please review this PR |
|
@chandrams Can you please rebase this PR with latest mvp demo |
msvinaykumar
requested changes
Feb 4, 2025
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
chandrams
force-pushed
the
stress-test
branch
from
19e08a5 to
1898178
Compare
Done |
Yes, I have made changes as per the new format. But the tests fail with issues already raised. |
tests/scripts/helpers/kruize.py
Outdated
| if include: | ||
| getJobIDURL = url_include | ||
|
|
||
| if logger: |
There was a problem hiding this comment.
Since the logger condition is repeated multiple times, the logging and printing logic can be moved to a common function.
There was a problem hiding this comment.
Thanks Shreya, I have made the changes.
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
…g recommendations Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
| days_of_res = 15 | ||
| results_dir = "." | ||
| initial_end_time = "" | ||
| interval_hours = 6 |
There was a problem hiding this comment.
Please update interval_hours to 2 hr
Signed-off-by: Chandrakala Subramanyam <csubrama@redhat.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Included stress test for Bulk API
Fixes # (issue)
Type of change
How has this been tested?
Please describe the tests that were run to verify your changes and steps to reproduce. Please specify any test configuration required.
Test Configuration
Checklist 🎯
Additional information
Include any additional information such as links, test results, screenshots here