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

Include the translation time and configuration push time in performance test results #6876

Open
2 of 3 tasks
randmonkey opened this issue Dec 19, 2024 · 1 comment
Open
2 of 3 tasks

Comments

@randmonkey
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Does this enhancement require public documentation?

  • I have added an Acceptance Criteria item for adding and/or adjusting public documentation (if applicable)

Problem Statement

The current performance test result does not include the elapsed time for translating k8s resources and duration of sending configuration to Kong gateway(s) separately. Including the duration of the two steps could help us to know the bottleneck of making the k8s resource effective on Kong gateway and optimize the performance here.

Proposed Solution

Now we have two metrics to collect the duration of the two steps:

  • ingress_controller_translation_duration_milliseconds for translation
  • ingress_controller_configuration_push_duration_milliseconds for pushing config to Kong gateway(s)
    We can make it possible to output the result in the performance test.

Additional information

No response

Acceptance Criteria

  • We can see the duration of the translation and pushing configuration to Kong gateway(s) separately in the result of performance tests.
@tao12345666333
Copy link
Member

I will assign this to me. When I have bw, I will start dealing with it.

@tao12345666333 tao12345666333 self-assigned this Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants