Skip to content

ARTEMIS-5911 Ensure core federation ignores AMQP federation configura…#6249

Merged
clebertsuconic merged 1 commit intoapache:mainfrom
tabish121:ARTEMIS-5911
Mar 3, 2026
Merged

ARTEMIS-5911 Ensure core federation ignores AMQP federation configura…#6249
clebertsuconic merged 1 commit intoapache:mainfrom
tabish121:ARTEMIS-5911

Conversation

@tabish121
Copy link
Contributor

…tion

When parsing the Core federation XML elements we need to ensure that AMQP broker connections with federation configuration does not get picked up as that creates empty configuration entries in the broker configuration.

@tabish121 tabish121 marked this pull request as draft February 24, 2026 23:03
@tabish121 tabish121 force-pushed the ARTEMIS-5911 branch 2 times, most recently from 40c9259 to ee98a00 Compare February 24, 2026 23:24
@tabish121 tabish121 marked this pull request as ready for review February 25, 2026 00:21
@gemmellr gemmellr marked this pull request as draft February 25, 2026 17:40
@tabish121 tabish121 force-pushed the ARTEMIS-5911 branch 3 times, most recently from efcf324 to ffa2e9e Compare March 3, 2026 18:24
…tion

When parsing the Core federation XML elements we need to ensure that AMQP broker
connections with federation configuration does not get picked up as that creates
empty configuration entries in the broker configuration.
@tabish121 tabish121 marked this pull request as ready for review March 3, 2026 19:49
@tabish121
Copy link
Contributor Author

Rebased and updated based on the latest changes to the Core federation parsing.

@clebertsuconic
Copy link
Contributor

LGTM

@clebertsuconic clebertsuconic merged commit e5597c0 into apache:main Mar 3, 2026
6 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.

3 participants