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

[BUG] Performance regression #355

Closed
1 task done
jwmatthews opened this issue Oct 4, 2024 · 2 comments
Closed
1 task done

[BUG] Performance regression #355

jwmatthews opened this issue Oct 4, 2024 · 2 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority Indicates an issue or PR lacks a `priority/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@jwmatthews
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Konveyor version

dcbb845

Priority

Critical

Current Behavior

Note, I am running Kantra against a branch in coolstore here:

https://github.com/konveyor-ecosystem/coolstore/commits/partial-migration/

As I run I see the analysis is taking longer than it used to.
I captured the below 3 runs to get an idea of analysis with the same app and options.

This is latest as of September 18th 2024

./kantra version
version: latest
SHA: dcbb845933ef84dbefdba7bf4f9b0aed6ec68751
image: quay.io/konveyor/kantra

real	7m56.523s
user	0m0.418s
sys	0m0.307s
version: v0.5.0
SHA: 8f2fa1b045ff29807c16b9796ecfa0e62a01150a
image: quay.io/konveyor/kantra

real	4m3.070s
user	0m0.417s
sys	0m0.291s
kantra version  
version: v0.4.0
SHA: 0f878a7539b9d0b50d271cfb26a96c6ddb5c0427
image: quay.io/konveyor/kantra

real 2m18.572s
user 0m0.553s
sys 0m0.250s

Expected Behavior

No response

How Reproducible

Always (Default)

Steps To Reproduce

No response

Environment

  • OS: MacOS arm64

Anything else?

No response

@jwmatthews jwmatthews added kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Oct 4, 2024
@konveyor-ci-bot
Copy link

This issue is currently awaiting triage.
If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.
The triage/accepted label can be added by org members.

@konveyor-ci-bot konveyor-ci-bot bot added the needs-priority Indicates an issue or PR lacks a `priority/foo` label and requires one. label Oct 4, 2024
@jwmatthews
Copy link
Member Author

I am less sure there is a regression in performance. I've run the "latest" Kantra as of Oct 3rd several times and am averaging around 3 minutes to complete the analysis. This sounds reasonable.

"latest" as of Oct 3rd

kantra version  
version: latest
SHA: 9928bb083351904c0582bc1f4af2710985f56d2c
image: quay.io/konveyor/kantra

real	3m4.763s
user	0m0.406s
sys	0m0.273s

Will close this issue as I'm now not convinced there is a performance problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority Indicates an issue or PR lacks a `priority/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
Status: ✅ Done
Development

No branches or pull requests

1 participant