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

[AF-2227] Set http router strategy so requests from REST client are s… #310

Merged
merged 1 commit into from
Nov 26, 2019

Conversation

jakubschwan
Copy link
Contributor

…end to same pod

Signed-off-by: Jakub Schwan jschwan@redhat.com

Thanks for submitting your Pull Request!

Please make sure your PR meets the following requirements:

  • Pull Request title is properly formatted: [RHDM-XYZ] Subject
  • Pull Request contains link to the JIRA issue
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
  • Attached commits represent units of work and are properly formatted
  • You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
  • Every commit contains Signed-off-by: Your Name <yourname@example.com> - use git commit -s

…end to same pod

Signed-off-by: Jakub Schwan <jschwan@redhat.com>
@spolti
Copy link
Member

spolti commented Nov 5, 2019

Is this conf only applied to HA envs?
what about the other envs which includes business/decision dentral?

@jakubschwan
Copy link
Contributor Author

Yes, this config is only applied to HA env. Where there are more pods of BC sync rest calls can return 404 as they were redirect to the pod where the operation is not processed. Same was applied for Kie Server HTTP route.
We do not need to set this to other envs as there isn't clustered business central.

@errantepiphany errantepiphany added 7.12.x ⇨ 7.12.0 7.12.xmain branch work for version 7.12.0 release (CR build/test) ready 🚀 work completed and ready to be merged labels Nov 26, 2019
@errantepiphany errantepiphany merged commit 6747c03 into jboss-container-images:master Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
7.12.x ⇨ 7.12.0 7.12.xmain branch work for version 7.12.0 release (CR build/test) ready 🚀 work completed and ready to be merged
3 participants