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.

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

Fix broker.xml connector and connector-ref when using federation #178

Merged
merged 2 commits into from
Sep 24, 2024
Merged

Fix broker.xml connector and connector-ref when using federation #178

merged 2 commits into from
Sep 24, 2024

Conversation

guidograzioli
Copy link
Member

Fix #177

@guidograzioli guidograzioli added the bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements label Sep 24, 2024
@garethahealy
Copy link
Collaborator

check for the message Connection configuration is null for connectorName in the logs and I think you'll see this fail.

@guidograzioli
Copy link
Member Author

Connection configuration is null for connectorName

It was surrounded with double quotes, I committed the change removing them; also added a task to enforce presence. Please give a look

@guidograzioli guidograzioli changed the title add federation molecule scenario Fix broker.xml connector and connector-ref when using federation Sep 24, 2024
@garethahealy
Copy link
Collaborator

Tested PR and works as expected

@guidograzioli
Copy link
Member Author

Tested PR and works as expected

Brilliant, thanks for all the help.

@guidograzioli guidograzioli merged commit 70a20d6 into ansible-middleware:main Sep 24, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Connectors cant be created for federated
2 participants
@guidograzioli @garethahealy