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

[12.0][BKP] edi_endpoint_oca #1074

Open
wants to merge 42 commits into
base: 12.0
Choose a base branch
from

Conversation

JordiMForgeFlow
Copy link
Contributor

No description provided.

simahawk and others added 30 commits November 22, 2024 13:10
Block archiving of backends if they have active endpoints exposed.
* more efficient w/ read_group
* consider only active records explicitly
I don't know why but  when test_edi_endpoint runs before these tests
the rollback of the exception catched within the test `test_archive_check`
make the controller lookup fail.
Since the whole routing registry machinery is going to be refactored
in OCA#633
let's survive w/ this forced registration for now.
Former version of `endpoint_route_handler` had a major flaw:

routing rule registry was not properly shared across workers
forcing us to restart the instance to make sure all envs were inline w/
it.

This change adapts edi_endpoint_oca to the new version which contains
some refactoring.
It's very handy to know which endpoint generated a record.
@JordiMForgeFlow JordiMForgeFlow force-pushed the 12.0-bkp-edi_endpoint_oca branch from 74a7957 to db2e7b1 Compare November 22, 2024 12:16
@JordiMForgeFlow JordiMForgeFlow force-pushed the 12.0-bkp-edi_endpoint_oca branch from db2e7b1 to 60c5721 Compare November 22, 2024 13:53
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.

8 participants