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.

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

Use AMQP fallback endpoints when available #30

Merged
merged 1 commit into from
Oct 30, 2017

Conversation

gberginc
Copy link
Contributor

@gberginc gberginc commented Oct 27, 2017

Qpid proton allows for automatic failover in case an error occurs for
the currently active connection. When the Nuage provider is associated
with endpoints with roles amqp-fallback1 and amqp-fallback2, these
will be used to configure the message handler.

Video showing the behaviour of the fallback: https://youtu.be/w-AekPyMOmo.
@miq-bot add_label events

@miq-bot miq-bot added the events label Oct 27, 2017
@gberginc
Copy link
Contributor Author

@miq-bot add_label wip

Marking wip because we are working with the UI PR in parallel

@miq-bot miq-bot changed the title Use AMQP fallback endpoints when available [WIP] Use AMQP fallback endpoints when available Oct 27, 2017
@miq-bot miq-bot added the wip label Oct 27, 2017
Qpid proton allows for automatic failover in case an error occurs for
the currently active connection. When the Nuage provider is associated
with endpoints with roles `amqp-fallback1` and `amqp-fallback2`, these
will be used to configure the message handler.

Signed-off-by: Gregor Berginc <[email protected]>
@miq-bot
Copy link
Member

miq-bot commented Oct 27, 2017

Checked commit gberginc@3833be8 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks fine. 👍

@gberginc
Copy link
Contributor Author

@miq-bot remove_label wip

@miq-bot assign @blomquisg

This PR adds the support for multiple URLs. The PR description has a video that demonstrates the behaviour.

@miq-bot miq-bot changed the title [WIP] Use AMQP fallback endpoints when available Use AMQP fallback endpoints when available Oct 27, 2017
@miq-bot miq-bot removed the wip label Oct 27, 2017
@juliancheal
Copy link
Member

LGTM 👍 Love the video!

@blomquisg blomquisg merged commit 6e9316b into ManageIQ:master Oct 30, 2017
@blomquisg blomquisg added this to the Sprint 72 Ending Oct 30, 2017 milestone Oct 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants