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

added integration of hunter, refactored code #6

Closed
wants to merge 0 commits into from
Closed

added integration of hunter, refactored code #6

wants to merge 0 commits into from

Conversation

shashank-boyapally
Copy link
Contributor

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

Hunter is being integrated so that change points can be detected. Refactored code to increase readability. Please ignore the files in the hunter directory.
Below is the sample output

(venv) sboyapal orion (cli) >> orion --hunter-analyze
2024-01-24 16:31:44,530 - Orion - INFO - The test aws-small-scale-cluster-density-v2 has started
2024-01-24 16:31:44,530 - Matcher - INFO - Executing query against index=perf_scale_ci
2024-01-24 16:31:44,768 - Matcher - INFO - Executing query against index=ripsaw-kube-burner*
2024-01-24 16:31:44,908 - Orion - INFO - Collecting podReadyLatency
2024-01-24 16:31:44,908 - Matcher - INFO - Executing query against index=ripsaw-kube-burner*
2024-01-24 16:31:45,044 - Orion - INFO - Collecting apiserverCPU
2024-01-24 16:31:45,045 - Matcher - INFO - Executing query against index=ripsaw-kube-burner*
2024-01-24 16:31:47,468 - Orion - INFO - Collecting ovnCPU
2024-01-24 16:31:47,468 - Matcher - INFO - Executing query against index=ripsaw-kube-burner*
2024-01-24 16:31:50,104 - Orion - INFO - Collecting etcdCPU
2024-01-24 16:31:50,104 - Matcher - INFO - Executing query against index=ripsaw-kube-burner*
time                       uuid                                    P99    apiserverCPU_cpu_avg    ovnCPU_cpu_avg    etcdCPU_cpu_avg
-------------------------  ------------------------------------  -----  ----------------------  ----------------  -----------------
2023-10-16 19:25:28 +0000  e624b10f-148b-4871-acb5-e8a71cf1b605  13000                 10.175            2.42737            7.63694
2023-10-18 15:11:43 +0000  325dd073-b680-4e44-951f-26506f401be0  13000                 10.4876           2.49018            8.12781
2023-10-24 10:52:22 +0000  8cfc4fcf-46c7-4b9f-a464-cd100bfb84bf  13000                 10.3932           2.44498            7.60968
                                                                        ······················  ················                     
                                                                                         +6.1%            +13.8%                     
                                                                        ······················  ················                     
2023-10-31 10:30:02 +0000  3c9e2789-59d4-4d6f-9f07-5c400cf25c93  13000                 11.0129           2.73347            8.83458
2023-11-03 04:08:14 +0000  649228fb-9ff4-458a-8a8b-b9ecb521ca1b  13000                 10.8605           2.82005            8.34146
2023-11-07 10:26:08 +0000  d8e0c965-4548-48d4-8a7a-c065b93ee766  15000                 10.5964           2.6356             7.48679
2023-11-15 12:36:38 +0000  a990f7f9-1faf-41f7-b9fb-3b83f91b6e52  13000                 10.8222           2.81836            8.56302
2023-11-22 12:53:45 +0000  e6c219dc-a9aa-4cf9-854a-fbd846c8ad22  13000                 10.8555           2.84967            8.20129
2023-11-29 12:32:04 +0000  adc95590-791d-423b-9d09-8190178edbf6  13000                 11.4115           2.8615             8.67125
2023-12-06 12:34:57 +0000  5b06c830-be55-4c84-b4da-92720b55c304  13000                 11.05             2.85996            8.69638
2023-12-20 12:37:17 +0000  c5f95336-27d4-4615-94c1-d618e975a36b  13000                 10.9253           2.77954            8.20148
2023-12-27 12:36:59 +0000  bd5e6cad-6bc2-472d-948a-b1fbb350e80a  13000                 11.1967           2.77645            8.63235
2024-01-03 12:33:09 +0000  b91c7156-0b86-4c79-81eb-60dfb7288070  13000                 10.7863           2.80557            8.54677
                                                                                                ················                     
                                                                                                          +11.7%                     
                                                                                                ················                     
2024-01-10 12:38:16 +0000  12a1750e-a7eb-48ed-bc00-35078d2bbad8  13000                 10.9379           3.10667            8.86329
2024-01-10 22:59:25 +0000  c6b4aaa3-cd2c-4f04-af20-3d10c52449a2  13000                 11.4162           3.1338             9.29143
2024-01-15 20:30:57 +0000  90189fbf-7181-4129-8ca5-3cc8d656b595  13000                 10.8181           3.05989            8.48998
2024-01-17 12:35:28 +0000  bff92096-3a8e-4cb4-805b-55638be4aa23  14000                 11.3023           3.03366            9.04686
2024-01-24 12:48:41 +0000  d70a8864-b251-417a-b43b-d85a54604c0f  13000                 10.7205           3.05               8.23406

Related Tickets & Documents

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please describe the System Under Test.
  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

hunter/analysis.py Outdated Show resolved Hide resolved
pylint -d C0103 $(git ls-files '*.py' | grep -v 'hunter/')
Copy link
Member

Choose a reason for hiding this comment

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

since we removed the hunter code, is this still needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it is no longer needed.. I will remove the pipe.

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