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

[RFR] Add analysis test for Cool Store application #183

Merged
merged 8 commits into from
Oct 16, 2024

Conversation

nachandr
Copy link
Collaborator

@nachandr nachandr commented Oct 3, 2024

Note: Analysis of Cool store app fails on Konveyor (latest nightly) installed on OCP. I found that Konveyor was extremely slow . I will try to analyze the app on Konveyor main installed on Minikube.

The current values for Issues and Dependencies are from analysis of the app on MTA 7.1. In the past, I've observed discrepancies in issues and dependencies between d/s and u/s, so , I'd like to get these values after a successful analysis on u/s.

Signed-off-by: Nandini Chandra <[email protected]>
Signed-off-by: Nandini Chandra <[email protected]>
Signed-off-by: Nandini Chandra <[email protected]>
Signed-off-by: Nandini Chandra <[email protected]>
Signed-off-by: Nandini Chandra <[email protected]>
Signed-off-by: Nandini Chandra <[email protected]>
@nachandr nachandr changed the title [WIP] Add analysis test for Cool Store application [RFR] Add analysis test for Cool Store application Oct 15, 2024
@nachandr
Copy link
Collaborator Author

nachandr commented Oct 15, 2024

API_PR_183

Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

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

LGTM

Note, expected analysis output might be not fully correct (but matching to current state of konveyor and its rulesets), so this test might need be updated when following tickets gets solved:
https://issues.redhat.com/browse/MTA-4024
https://issues.redhat.com/browse/MTA-4025
https://issues.redhat.com/browse/MTA-4026
https://issues.redhat.com/browse/MTA-4027

Signed-off-by: Nandini Chandra <[email protected]>
@nachandr nachandr merged commit 975ebc0 into konveyor:main Oct 16, 2024
6 of 7 checks passed
mguetta1 pushed a commit to mguetta1/go-konveyor-tests that referenced this pull request Nov 6, 2024
* Add analysis test for Cool Store application

Signed-off-by: Nandini Chandra <[email protected]>

* Update dependencies

Signed-off-by: Nandini Chandra <[email protected]>

* Update Analysis tags

Signed-off-by: Nandini Chandra <[email protected]>

* Add additional issues

Signed-off-by: Nandini Chandra <[email protected]>

* Update name

Signed-off-by: Nandini Chandra <[email protected]>

* Add GH repo for Cool store app

Signed-off-by: Nandini Chandra <[email protected]>

* Remove redundant file

Signed-off-by: Nandini Chandra <[email protected]>

* Update app name

Signed-off-by: Nandini Chandra <[email protected]>

---------

Signed-off-by: Nandini Chandra <[email protected]>
mguetta1 pushed a commit to mguetta1/go-konveyor-tests that referenced this pull request Nov 18, 2024
* Add analysis test for Cool Store application

Signed-off-by: Nandini Chandra <[email protected]>

* Update dependencies

Signed-off-by: Nandini Chandra <[email protected]>

* Update Analysis tags

Signed-off-by: Nandini Chandra <[email protected]>

* Add additional issues

Signed-off-by: Nandini Chandra <[email protected]>

* Update name

Signed-off-by: Nandini Chandra <[email protected]>

* Add GH repo for Cool store app

Signed-off-by: Nandini Chandra <[email protected]>

* Remove redundant file

Signed-off-by: Nandini Chandra <[email protected]>

* Update app name

Signed-off-by: Nandini Chandra <[email protected]>

---------

Signed-off-by: Nandini Chandra <[email protected]>
mguetta1 pushed a commit to mguetta1/go-konveyor-tests that referenced this pull request Nov 19, 2024
mguetta1 added a commit that referenced this pull request Nov 21, 2024
---------
Signed-off-by: Nandini Chandra <[email protected]>
Co-authored-by: Nandini Chandra <[email protected]>
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