Skip to content

Solace pubsub conformance test fix #3802

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Apr 23, 2025
Merged

Solace pubsub conformance test fix #3802

merged 4 commits into from
Apr 23, 2025

Conversation

antontroshin
Copy link
Contributor

@antontroshin antontroshin commented Apr 23, 2025 •
edited
Loading

Description

Bug

Fix the failing conformance test for Solace
Docker container was failing with the error on low nofile ulimits

ERROR    POST Violation [022]:Required system resource missing, Hard resource limit nofile 1048576 is required, 6592 detected

Update the numbers as recommended by the Solace documentation
https://docs.solace.com/Software-Broker/System-Resource-Requirements.htm#concurrent-open-files-considerations

Additional diagnostic step

Added a new step in .github/workflows/conformance.yml, "Retrieve infrastructure failure logs".
When the infra "Setup" stage fails, some components' infra do not print logs right away, like in this case, docker compose is used with -d flag, running in the background.
With this step, can add a special script to run when the "Run tests" ends with failure and the log script is configured in the test matrix parameters for this component test type.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #3789

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@antontroshin
Solace pubsub conformance test fix
da10c49 
Signed-off-by: Anton Troshin <[email protected]>
@antontroshin
increase Solace nofile ulimits
ec7e3db 
Signed-off-by: Anton Troshin <[email protected]>
@antontroshin
Rename step
6f9d82f 
Signed-off-by: Anton Troshin <[email protected]>
@antontroshin antontroshin marked this pull request as ready for review April 23, 2025 02:02
@antontroshin antontroshin requested review from a team as code owners April 23, 2025 02:02
@yaron2 yaron2 merged commit a68ca21 into dapr:main Apr 23, 2025
173 of 175 checks passed
@antontroshin antontroshin deleted the solace-conformance-fix branch April 23, 2025 16:24
abossard pushed a commit to abossard/components-contrib that referenced this pull request Apr 29, 2025
@antontroshin @yaron2
Solace pubsub conformance test fix (dapr#3802)
4630f5c 
Signed-off-by: Anton Troshin <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
Signed-off-by: Andre Bossard <[email protected]>
abossard pushed a commit to abossard/components-contrib that referenced this pull request Apr 29, 2025
Squashed commit of the following:
5642742 
commit 3549843
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9a11502
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 8014a1d
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 72ffcd1
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 12be96f
Author: Yaron Schneider <[email protected]>
Date:   Wed Jan 29 13:41:28 2025 -0800

    Add deepseek support, update Go to 1.23.5 (dapr#3659)

    Signed-off-by: yaron2 <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit a00c8f3
Merge: a395f74 397766a
Author: Andre Bossard <[email protected]>
Date:   Tue Apr 29 11:41:27 2025 +0200

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit a395f74
Author: Andre Bossard <[email protected]>
Date:   Mon Apr 28 10:33:59 2025 +0200

    Refactor tryGetProperty to use strings.EqualFold for case-insensitive key matching

    Signed-off-by: Andre Bossard <[email protected]>

commit 1d4f3a1
Author: Anton Troshin <[email protected]>
Date:   Thu Apr 24 15:19:04 2025 -0500

    Support Oracle Connect Descriptors (dapr#3800)

    Signed-off-by: Anton Troshin <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 4630f5c
Author: Anton Troshin <[email protected]>
Date:   Wed Apr 23 07:34:59 2025 -0500

    Solace pubsub conformance test fix (dapr#3802)

    Signed-off-by: Anton Troshin <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit ed3f7a8
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit af31fae
Author: MikelRev <[email protected]>
Date:   Thu Apr 17 11:54:52 2025 -0400

    Updated sqlserver auth to utilize default scope. (dapr#3698)

    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit c5a0407
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:39:26 2025 +0200

    Fix metadata handling by removing unnecessary value escaping in RabbitMQ message processing

    Signed-off-by: Andre Bossard <[email protected]>

commit 5bcb186
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:13:24 2025 +0200

    Refactor RabbitMQ metadata handling to use case-insensitive key retrieval

    Signed-off-by: Andre Bossard <[email protected]>

commit 4f17a7f
Author: Emmanuel Auffray <[email protected]>
Date:   Thu Apr 10 10:06:37 2025 +1200

    Adding GoogleAI models too (dapr#3689)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Josh van Leeuwen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Co-authored-by: Cassie Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 652c733
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 71e9a4b
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 621f7ef
Author: Emmanuel Auffray <[email protected]>
Date:   Mon Mar 24 05:38:17 2025 +1300

    Adding Ollama as a conversation component for local dev/running of LLMs (dapr#3688)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Mike Nguyen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 229524f
Author: Emmanuel Auffray <[email protected]>
Date:   Sat Mar 22 00:52:15 2025 +1300

    Fix reference links of conversation components (dapr#3690)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9dbe6e3
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 12:19:53 2025 +0200

    Add metadata handling for RabbitMQ messages and corresponding test

    Signed-off-by: Andre Bossard <[email protected]>

commit 3f0ec0b
Author: Andre Bossard <[email protected]>
Date:   Thu Mar 20 03:25:29 2025 +0100

    Add RabbitMQ metadata properties test and configuration

    Signed-off-by: Andre Bossard <[email protected]>

commit b6acda8
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 12:25:31 2025 +0100

    Add integration test for publishing metadata properties to RabbitMQ

    Signed-off-by: Andre Bossard <[email protected]>

commit f726110
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:26:49 2025 +0100

    Refactor RabbitMQ publishing: consolidate metadata application into a single function

    Signed-off-by: Andre Bossard <[email protected]>

commit 4cb5600
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:16:14 2025 +0100

    Refactor RabbitMQ metadata handling: consolidate metadata retrieval into common package

    Signed-off-by: Andre Bossard <[email protected]>

commit 069b12f
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 08:12:25 2025 +0100

    Refactor metadata function names for consistency: update messageId and correlationId to messageID and correlationID

commit d508479
Merge: af3fca0 d8ac01b
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 07:58:36 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit af3fca0
Merge: 57ae111 637d18d
Author: Andre Bossard <[email protected]>
Date:   Wed Mar 12 03:28:26 2025 -0700

    Merge branch 'main' into supporting_more_properties_for_rabbitmq_on_main

commit 57ae111
Merge: d0c75f6 2997e47
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 4 08:49:52 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit d0c75f6
Author: Andre Bossard <[email protected]>
Date:   Wed Feb 5 11:43:03 2025 +0100

    Add support for messageId, correlationId, and type in RabbitMQ bindings

    Signed-off-by: Andre Bossard <[email protected]>
abossard pushed a commit to abossard/components-contrib that referenced this pull request Apr 29, 2025
Squashed commit of the following:
76ad9bb 
commit 3549843
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9a11502
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 8014a1d
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 72ffcd1
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 12be96f
Author: Yaron Schneider <[email protected]>
Date:   Wed Jan 29 13:41:28 2025 -0800

    Add deepseek support, update Go to 1.23.5 (dapr#3659)

    Signed-off-by: yaron2 <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit a00c8f3
Merge: a395f74 397766a
Author: Andre Bossard <[email protected]>
Date:   Tue Apr 29 11:41:27 2025 +0200

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit a395f74
Author: Andre Bossard <[email protected]>
Date:   Mon Apr 28 10:33:59 2025 +0200

    Refactor tryGetProperty to use strings.EqualFold for case-insensitive key matching

    Signed-off-by: Andre Bossard <[email protected]>

commit 1d4f3a1
Author: Anton Troshin <[email protected]>
Date:   Thu Apr 24 15:19:04 2025 -0500

    Support Oracle Connect Descriptors (dapr#3800)

    Signed-off-by: Anton Troshin <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 4630f5c
Author: Anton Troshin <[email protected]>
Date:   Wed Apr 23 07:34:59 2025 -0500

    Solace pubsub conformance test fix (dapr#3802)

    Signed-off-by: Anton Troshin <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit ed3f7a8
Author: Cassie Coyle <[email protected]>
Date:   Wed Apr 23 07:33:24 2025 -0500

    Add baggage header support to http binding (dapr#3723)

    Signed-off-by: Cassandra Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit af31fae
Author: MikelRev <[email protected]>
Date:   Thu Apr 17 11:54:52 2025 -0400

    Updated sqlserver auth to utilize default scope. (dapr#3698)

    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit c5a0407
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:39:26 2025 +0200

    Fix metadata handling by removing unnecessary value escaping in RabbitMQ message processing

    Signed-off-by: Andre Bossard <[email protected]>

commit 5bcb186
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 13:13:24 2025 +0200

    Refactor RabbitMQ metadata handling to use case-insensitive key retrieval

    Signed-off-by: Andre Bossard <[email protected]>

commit 4f17a7f
Author: Emmanuel Auffray <[email protected]>
Date:   Thu Apr 10 10:06:37 2025 +1200

    Adding GoogleAI models too (dapr#3689)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Josh van Leeuwen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Co-authored-by: Cassie Coyle <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 652c733
Author: Josh van Leeuwen <[email protected]>
Date:   Wed Mar 26 19:55:55 2025 -0300

    CVE go mod dep update (dapr#3716)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 71e9a4b
Author: Josh van Leeuwen <[email protected]>
Date:   Mon Mar 24 17:50:50 2025 -0300

    go.mod: CVE updates (dapr#3713)

    Signed-off-by: joshvanl <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 621f7ef
Author: Emmanuel Auffray <[email protected]>
Date:   Mon Mar 24 05:38:17 2025 +1300

    Adding Ollama as a conversation component for local dev/running of LLMs (dapr#3688)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Co-authored-by: Mike Nguyen <[email protected]>
    Co-authored-by: Yaron Schneider <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 229524f
Author: Emmanuel Auffray <[email protected]>
Date:   Sat Mar 22 00:52:15 2025 +1300

    Fix reference links of conversation components (dapr#3690)

    Signed-off-by: Emmanuel Auffray <[email protected]>
    Signed-off-by: Andre Bossard <[email protected]>

commit 9dbe6e3
Author: Andre Bossard <[email protected]>
Date:   Wed Apr 16 12:19:53 2025 +0200

    Add metadata handling for RabbitMQ messages and corresponding test

    Signed-off-by: Andre Bossard <[email protected]>

commit 3f0ec0b
Author: Andre Bossard <[email protected]>
Date:   Thu Mar 20 03:25:29 2025 +0100

    Add RabbitMQ metadata properties test and configuration

    Signed-off-by: Andre Bossard <[email protected]>

commit b6acda8
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 12:25:31 2025 +0100

    Add integration test for publishing metadata properties to RabbitMQ

    Signed-off-by: Andre Bossard <[email protected]>

commit f726110
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:26:49 2025 +0100

    Refactor RabbitMQ publishing: consolidate metadata application into a single function

    Signed-off-by: Andre Bossard <[email protected]>

commit 4cb5600
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 09:16:14 2025 +0100

    Refactor RabbitMQ metadata handling: consolidate metadata retrieval into common package

    Signed-off-by: Andre Bossard <[email protected]>

commit 069b12f
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 08:12:25 2025 +0100

    Refactor metadata function names for consistency: update messageId and correlationId to messageID and correlationID

commit d508479
Merge: af3fca0 d8ac01b
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 18 07:58:36 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit af3fca0
Merge: 57ae111 637d18d
Author: Andre Bossard <[email protected]>
Date:   Wed Mar 12 03:28:26 2025 -0700

    Merge branch 'main' into supporting_more_properties_for_rabbitmq_on_main

commit 57ae111
Merge: d0c75f6 2997e47
Author: Andre Bossard <[email protected]>
Date:   Tue Mar 4 08:49:52 2025 +0100

    Merge branch 'dapr:main' into supporting_more_properties_for_rabbitmq_on_main

commit d0c75f6
Author: Andre Bossard <[email protected]>
Date:   Wed Feb 5 11:43:03 2025 +0100

    Add support for messageId, correlationId, and type in RabbitMQ bindings

    Signed-off-by: Andre Bossard <[email protected]>

Signed-off-by: Andre Bossard <[email protected]>
jjcollinge pushed a commit to jjcollinge/components-contrib that referenced this pull request May 12, 2025
@antontroshin @yaron2 @jjcollinge
Solace pubsub conformance test fix (dapr#3802)
019ab00 
Signed-off-by: Anton Troshin <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yaron2 yaron2 yaron2 approved these changes

@nelson-parente nelson-parente nelson-parente approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pubsub.solace flaky conf test
3 participants
@antontroshin @yaron2 @nelson-parente