-
Notifications
You must be signed in to change notification settings - Fork 6
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
Cilium API Rate Limiting Test #414
Open
mushiboy
wants to merge
92
commits into
main
Choose a base branch
from
mugeshsp/api-rate-limiting-cilium
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
92 commits
Select commit
Hold shift + click to select a range
dad0003
Add API rate limiting performance evaluation pipeline
mushiboy 8eeca43
Update API rate-limiting pipeline configuration and create a new conf…
mushiboy aa5a43f
Add Cilium API Rate Limiter metric to existing cilium-metrics
mushiboy 5697495
Reduce the number of repeats in the API rate limiting performance eva…
mushiboy c7647d1
Fix metric queries and add cilium agent metrics
mushiboy 74bc855
Refactor API rate limiting pipeline configuration to streamline param…
mushiboy a4b7631
Merge API Rate Limiting test with load-config
mushiboy b5143d1
Update API rate limiting pipeline topology from service-churn to slo
mushiboy f8b1108
Add API rate limiting topology and related configuration files
mushiboy d51a937
Changed API rate limiting configuration
mushiboy b4fa439
Updated default -> DefaultParam at line 44
mushiboy 5df6b17
Fix YAML syntax in load-config.yaml by closing the conditional block
mushiboy 7c907f0
Fix YAML syntax in load-config.yaml by correcting the closing conditi…
mushiboy e96a074
Update default function calls to DefaultParam in reconcile-objects.yaml
mushiboy df79618
Fix YAML syntax by ensuring proper closure of conditional block in re…
mushiboy 96f8ac3
Remove unused burst parameter from BurstLoad configuration in load-co…
mushiboy b4bfd9b
Fix: Rename burstLoad to qpsLoad in BurstLoad tuning set in load-conf…
mushiboy 981e8d9
Fix: Update pods_per_node assignment for api_rate_limiting_test in sl…
mushiboy 277c89e
Fix: Add api_rate_limiting_test and pods argument to main function in…
mushiboy b91b8bb
Fix: Update argument parsing for api_rate_limiting_test and pods in s…
mushiboy bbd0b4b
Fix: Increase cpu_per_node from 1 to 4 in api-rate-limiting.yml for i…
mushiboy adaa583
Fix: Reorder API rate limiting test argument in execute.yml for impro…
mushiboy b007ed0
Revert "Fix: Update pods_per_node assignment for api_rate_limiting_te…
mushiboy 31183f8
Fix: Update pods_per_node assignment for api_rate_limiting_test in sl…
mushiboy c6549fa
Fix: Remove api_rate_limiting_test and pods arguments from argument p…
mushiboy 54b6b1d
Fix: Correct spacing in collect.yml and execute.yml
mushiboy 254bbf1
Fix: Add api_rate_limiting_test argument to configuration and update …
mushiboy 88468c1
Fix: Add api_rate_limiting_test argument to main function in slo.py f…
mushiboy 42b41f0
Fix: Reorder API_RATE_LIMITING_TEST argument in execute.yml for corre…
mushiboy 369eaea
Fix: Add API_RATE_LIMITING_TEST default value and conditional logic i…
mushiboy fd4be3c
Fix: Set default value for bigDeploymentsPerNamespace to 1 and remove…
mushiboy 7675891
Fix: Add smallDeploymentSize and smallDeploymentsPerNamespace to load…
mushiboy db3960b
Fix: Set CL2_SERVICE_TEST and CL2_API_RATE_LIMITING_TEST to true in s…
mushiboy 934b660
Fix: Set CL2_SERVICE_TEST to false in slo.py for accurate test config…
mushiboy 3d106b3
Testing with lower pod numbers
mushiboy 8b57d2f
Fix: Update API_RATE_LIMITING_PODS_PER_NODE to 250 and set default po…
mushiboy 884ec89
Fix: Update bigDeploymentSize variable to use pods parameter in load-…
mushiboy 0200416
Fix: Update pods parameter in load-config.yaml to default to 0 for AP…
mushiboy 7b4d0b9
Fix: Add pods argument to API Rate Limiting Test in slo.py and update…
mushiboy e597133
Fix: Update SCENARIO_VERSION to 'test' in api-rate-limiting.yml
mushiboy 55bc92f
Update pods parameter in api-rate-limiting.yml to 134 for testing par…
mushiboy 6c281f5
Fix: Remove default value for pods argument in slo.py and update rela…
mushiboy bac5d49
Fix: Set default value for pods argument in configure_clusterloader2 …
mushiboy c8b85d4
Fix: Remove api-rate-limiting-config.yaml and set default value for p…
mushiboy ad66384
Enhance Cilium API Rate Limiter metrics in cilium-measurements.yaml b…
mushiboy 242bd49
Testing with 50 pods
mushiboy 1933a46
Fix indentation in cilium-measurements.yaml
mushiboy 7c45374
Fix: Cilium Measurements
mushiboy 15027b9
Fix
mushiboy 54dcde3
Fix: CiliumAPIRateLimiterProcessedRequestsCount
mushiboy 6768722
Measurement fix
mushiboy 2b67a94
Fix: CiliumAPIRateLimiterRequestsInFlight
mushiboy 67a4188
Remove ByAction query from Cilium measurements configuration
mushiboy 0433f51
Update Cilium measurements to include time window in queries
mushiboy cff58ed
Update Cilium measurements to use a 15-minute time window for queries
mushiboy 3dd245c
Commented out requests in flight
mushiboy 034bf6b
Comment out NetworkProgrammingLatency configuration in measurements.yaml
mushiboy b62d83a
Remove API rate limiting test from clusterloader2 collect script
mushiboy d876166
Add pods argument for API Rate Limiting Test in slo.py and update col…
mushiboy 7a3267b
Fix
mushiboy 369e135
Add pods argument for API Rate Limiting Test in slo.py and update col…
mushiboy 41f7a87
Update API rate limiting scenario version and increase pod count to 200
mushiboy a3731f5
Testing the rate limit config
mushiboy a539959
Update scenario version to test 200 pods with low rate limit and adju…
mushiboy bf20a88
Increase CPU allocation per node to 4 for API rate limiting test
mushiboy c0e6155
Test-Run Changes
mushiboy 40cc708
Increase desired nodes to 7 in API rate limiting resource validation
mushiboy c26ccfc
200 pods test
mushiboy 5e3e4cd
Decrease desired nodes to 3 in API rate limiting resource validation
mushiboy 70f380d
Update Cilium measurements to use variable time intervals for rate qu…
mushiboy c784b85
Refactor Cilium measurements to remove unnecessary sum in rate queries
mushiboy faeeb6f
Add script to set a unique Run ID before publish in clusterloader2 co…
mushiboy d3f1381
Update scenario version to benchmark in API rate limiting configuration
mushiboy bc9f3b8
Remove unnecessary blank lines in load-config.yaml
mushiboy 2389b37
Add parameter for configurable number of pods in API rate limiting pi…
mushiboy d6e721f
Update scenario version to main in API rate limiting configuration
mushiboy d508e7b
Fix query in Cilium measurements to sum processed requests rate
mushiboy 9f36264
Refactor Cilium measurements queries to use sum for accurate metrics …
mushiboy 85a3a40
Refactor indentation
mushiboy f0d6935
Fix Cilium measurements query to use rate for accurate processed requ…
mushiboy 27b9259
CiliumAPIRateLimiterRequestsInFlight
mushiboy f607a60
Refactor Cilium measurements queries to use sum for accurate metrics …
mushiboy 0491879
Refactor Cilium measurements queries and update pod actions to use va…
mushiboy 6bdd1bd
Update scenario version to v1 in API rate limiting pipeline
mushiboy fef9aad
Update scenario version to v1.1 in API rate limiting pipeline
mushiboy 8ef35be
Refactor Cilium measurements queries
mushiboy aa93862
Testing API Process time metric
mushiboy e5f2b66
Comment out Cilium API Rate Limiter metrics in configuration
mushiboy 09982df
Update scenario version to 'test' in API rate limiting pipeline
mushiboy 2162965
Add 'Requests' metric to Cilium API process time measurements
mushiboy d5ffea7
Add 'TotalTime' metric to Cilium API process time measurements
mushiboy 1658670
Update 'Requests' metric to use 'increase' function for accurate API …
mushiboy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this parameter for?