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

[KOGITO-9763] Adding metadata to node #3195

Merged
merged 2 commits into from
Aug 30, 2023
Merged

Conversation

fjtirado
Copy link
Contributor

@fjtirado fjtirado commented Aug 29, 2023

Adding state, action and branch metadata to each node.

@@ -94,7 +92,7 @@ public T doGetProcessNodes(String processId) {
Map<String, Object> data = new HashMap<>();
data.put("id", n.getId());
data.put("uniqueId", ((Node) n).getUniqueId());
data.put("nodeDefinitionId", n.getMetaData().get(UNIQUE_ID));
data.put("metadata", n.getMetaData());
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now that we are including all node metadata in the response, I think we can remove the nodeDefinitionId, which is kind of duplicated of uniqueId

@kie-ci3
Copy link
Contributor

kie-ci3 commented Aug 29, 2023

(tests) - kogito-runtimes job #3119 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer

export BUILD_MVN_OPTS_CURRENT=-T 1C -Dvalidate-formatting -Prun-code-coverage
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-runtimes -u https://github.com/kiegroup/kogito-runtimes/pull/3195 --skipParallelCheckout

NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.kogito-runtimes/3119/display/redirect

Test results:

  • PASSED: 3367
  • FAILED: 2

Those are the test failures:

org.kie.kogito.integrationtests.quarkus.ManagementAddOnIT.testGetProcessNodes 1 expectation failed.
JSON path [0].nodeDefinitionId doesn't match.
Expected: not (null or an empty string)
Actual: null
org.kie.kogito.integrationtests.springboot.ManagementAddOnTest.testGetProcessNodes 1 expectation failed.
JSON path [0].nodeDefinitionId doesn't match.
Expected: not (null or an empty string)
Actual: null

@kie-ci3
Copy link
Contributor

kie-ci3 commented Aug 29, 2023

(tests) - serverless-workflow-examples job #1410 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer

export BUILD_MVN_OPTS_CURRENT=
build-chain build cross_pr -f 'https://raw.githubusercontent.com/kiegroup/kogito-pipelines/main/.ci/pull-request-config.yaml' -o 'bc' -p kiegroup/kogito-examples -u https://github.com/kiegroup/kogito-runtimes/pull/3195 --skipParallelCheckout

NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.serverless-workflow-examples/1410/display/redirect

Test results:

  • PASSED: 81
  • FAILED: 1

Those are the test failures:

org.kie.kogito.examples.OpenVinoRestIT.testOpenvino 1 expectation failed.
JSON path workflowdata.group doesn't match.
Expected: a string containing "flat-coated retriever"
Actual: <[]>

@sonarcloud
Copy link

sonarcloud bot commented Aug 30, 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

82.8% 82.8% Coverage
0.0% 0.0% Duplication

@fjtirado fjtirado merged commit a3e2868 into apache:main Aug 30, 2023
7 checks passed
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.

4 participants