boozallen / aissemble

Booz Allen's lean manufacturing approach for holistically designing, developing and fielding AI solutions across the engineering lifecycle from data processing to model building, tuning, and training to secure operational deployment
Other
34 stars 8 forks source link

BUG: FIx data delivery steps not injecting smallrye when running test-data-delivery-spark-model. #430

Closed jaebchoi closed 1 month ago

jaebchoi commented 1 month ago

Description

We have disabled some data delivery steps as those steps are not injecting smallrye reactive messaging correctly.

Here are missing steps

        {
            "name": "VoidInboundAndMessagingOutboundAsync",
            "type": "asynchronous",
            "outbound": {
                "type": "messaging",
                "channelName": "outboundChannel"
            }
        },

        {
            "name": "MessagingOutboundWithCustomTypesAsync",
            "type": "asynchronous",
            "outbound": {
                "type": "messaging",
                "channelName": "outboundChannel",
                "recordType": {
                    "name": "CustomRecord",
                    "package": "com.boozallen.aiops.mda.pattern.record"
                }
            }
        },

        {
            "name": "MessagingOutboundWithCustomRecAsync",
            "type": "asynchronous",
            "outbound": {
                "type": "messaging",
                "channelName": "outboundChannel",
                "recordType": {
                    "name": "CustomRec",
                    "package": "com.boozallen.aiops.mda.pattern.record"
                }
            }
        },

It seems like any step with asynchronous type, no inbound messaging , and messaging outbound is having issue with running step execution.

Definition of Done

Test Instruction

Steps to Reproduce

Expected Behavior

Actual Behavior

jaebchoi commented 1 month ago

DoD completed with @aaron-gary

jaebchoi commented 1 month ago

OTS Completed with @chang-annie

cpointe-ibllanos commented 1 month ago

Tested - did not experience stalling on data lineage recording in either CI or locally. Builds pass.