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

chore: Improve conformance validation #3023

Merged
merged 53 commits into from
Aug 11, 2023

Conversation

matejpopda
Copy link
Contributor

@matejpopda matejpopda commented Aug 8, 2023

Description

Reworked previous work on conformance verification by apiml

Linked to issue 647

Type of change

  • (chore) Chore, repository cleanup, updates the dependencies.

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…Testing' into reboot/GH647/improvedConformanceTesting
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…ents on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…Testing' into reboot/GH647/improvedConformanceTesting
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…ents on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
@matejpopda matejpopda changed the title Reboot/gh647/improved conformance testing chore: Improve conformance validation Aug 8, 2023
@matejpopda matejpopda self-assigned this Aug 8, 2023
Shobhajayanna and others added 3 commits August 8, 2023 17:06
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…Testing' into reboot/GH647/improvedConformanceTesting
matejpopda and others added 18 commits August 9, 2023 13:55
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
… are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
… are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
… are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…ce:test shouldn't fail anymore

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…ce:test shouldn't fail anymore

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
…Testing' into reboot/GH647/improvedConformanceTesting
…ailing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
@sonarcloud
Copy link

sonarcloud bot commented Aug 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

90.8% 90.8% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Copy link
Contributor

@taban03 taban03 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@taban03 taban03 merged commit 4bb9df9 into v2.x.x Aug 11, 2023
33 checks passed
@delete-merged-branch delete-merged-branch bot deleted the reboot/GH647/improvedConformanceTesting branch August 11, 2023 11:47
achmelo pushed a commit that referenced this pull request Aug 15, 2023
* feat(gateway):  Improvement of existing conformance testing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* refactor(gateway):  Added missing comments

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Removed a print statement.

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway):  Better response from the conformance endpoint

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway):  Better response from the conformance endpoint

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* refactor(gateway): Got rid of duplicate discovery client calls

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* docs(gateway): Changed text in gateway-log-messages.yml based on comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway): Addressed the rest of comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway): Addressed the new comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed a method not following a naming convention

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed reason why build was failing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Addressed Pablo's comments on PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Updated an integration test for conformance

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Updated integration tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway):  Improvement of existing conformance testing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* refactor(gateway):  Added missing comments

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Removed a print statement.

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway):  Better response from the conformance endpoint

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* refactor(gateway): Got rid of duplicate discovery client calls

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* docs(gateway): Changed text in gateway-log-messages.yml based on comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway): Addressed the rest of comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* feat(gateway): Addressed the new comments on the PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed a method not following a naming convention

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed reason why build was failing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Addressed Pablo's comments on PR

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Updated an integration test for conformance

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Updated integration tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed code smells

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed 1 bug from sonar

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Disabled caching on a github action as a test

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Enabled cache again + removed some tests to see if they are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Enabled cache again + removed some tests to see if they are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Enabled cache again + removed some tests to see if they are the reason why the build fails

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Disabled cache again, changed unit tests :gateway-service:test shouldn't fail anymore

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Disabled cache again, changed unit tests :gateway-service:test shouldn't fail anymore

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Modified unit tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Modified unit tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Modified unit tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Fixed unit tests

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Enabled build cache

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Disabled build cache

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Enabled build cache + should create an artifact after failing

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

* fix(gateway): Disabled the creation of artifact from previous commit

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>

---------

Signed-off-by: matejpopda <20053580+matejpopda@users.noreply.github.com>
Co-authored-by: ShobhaJayanna <36433611+Shobhajayanna@users.noreply.github.com>
Co-authored-by: Andrea Tabone <39694626+taban03@users.noreply.github.com>
Signed-off-by: achmelo <a.chmelo@gmail.com>
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.

3 participants