Skip to content

Re-submit cuttlefish tls schema for amqp_client#15767

Merged
michaelklishin merged 1 commit intorabbitmq:mainfrom
amazon-mq:amqp_client_schema-gh-11415-followup
Mar 18, 2026
Merged

Re-submit cuttlefish tls schema for amqp_client#15767
michaelklishin merged 1 commit intorabbitmq:mainfrom
amazon-mq:amqp_client_schema-gh-11415-followup

Conversation

@lukebakken
Copy link
Copy Markdown
Collaborator

Follow-up PR to the following:

Having cuttlefish schemas for amqp_client and amqp10_client still seems useful, so I looked into why this was reverted. Since bazel is no longer used, that reason no longer exists. In my local env, all of these test runs succeed just fine:

make -C deps/rabbit ct-config_schema
make -C deps/amqp10_client tests
make -C deps/amqp_client tests

@lukebakken lukebakken self-assigned this Mar 18, 2026
@lukebakken lukebakken force-pushed the amqp_client_schema-gh-11415-followup branch from db2c95d to e97be87 Compare March 18, 2026 17:56
…amqp_client_schema"

This reverts commit 4b4c0fe, reversing
changes made to b9bbf69.
@michaelklishin
Copy link
Copy Markdown
Collaborator

michaelklishin commented Mar 18, 2026

Thank you, @lukebakken. Re-submitted with some follow-up changes from me #15772.

@michaelklishin michaelklishin merged commit bbb6497 into rabbitmq:main Mar 18, 2026
183 of 184 checks passed
mergify Bot pushed a commit that referenced this pull request Mar 18, 2026
mergify Bot pushed a commit that referenced this pull request Mar 18, 2026
A follow-up to #15767.

(cherry picked from commit 5c0d7f5)
(cherry picked from commit 22df658)
@the-mikedavis the-mikedavis deleted the amqp_client_schema-gh-11415-followup branch March 19, 2026 02:56
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