-
Notifications
You must be signed in to change notification settings - Fork 39
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
feat: access points synchronization #7380
Conversation
b304c10
to
5127f7e
Compare
...c/main/java/io/gravitee/apim/infra/crud_service/access_point/AccessPointCrudServiceImpl.java
Outdated
Show resolved
Hide resolved
e32d55c
to
6d8b0b6
Compare
...s-api/src/main/java/io/gravitee/gateway/handlers/accesspoint/manager/AccessPointManager.java
Outdated
Show resolved
Hide resolved
...i/src/main/java/io/gravitee/gateway/handlers/accesspoint/manager/AccessPointManagerImpl.java
Show resolved
Hide resolved
...ain/java/io/gravitee/gateway/services/sync/process/distributed/mapper/AccessPointMapper.java
Show resolved
Hide resolved
...in/java/io/gravitee/gateway/services/sync/process/repository/fetcher/AccessPointFetcher.java
Outdated
Show resolved
Hide resolved
...in/java/io/gravitee/gateway/services/sync/process/repository/fetcher/AccessPointFetcher.java
Outdated
Show resolved
Hide resolved
...teway/services/sync/process/repository/synchronizer/accesspoint/AccessPointSynchronizer.java
Outdated
Show resolved
Hide resolved
...teway/services/sync/process/repository/synchronizer/accesspoint/AccessPointSynchronizer.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/gravitee/apim/infra/crud_service/access_point/AccessPointCrudServiceImpl.java
Outdated
Show resolved
Hide resolved
b2d83fb
to
56eaefe
Compare
2ad23c9
to
0950239
Compare
@guillaumelamirand Thanks for the comments, I have added a new commit where I have adapted the code to use events for deploy and undeploy instead of just looking at the repository and I've also made changes for other comments. I'm going to work on tests but if you could have a look at the changes that would be great (to note there is some code on the repository that isn't needed anymore if keep the new event sync approach which I will then remove) |
0950239
to
8fbb6fb
Compare
...i/src/main/java/io/gravitee/gateway/handlers/accesspoint/manager/AccessPointManagerImpl.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/gravitee/apim/infra/crud_service/access_point/AccessPointCrudServiceImpl.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/gravitee/apim/infra/crud_service/access_point/AccessPointCrudServiceImpl.java
Outdated
Show resolved
Hide resolved
I had a look at the changes. I am quite ok with them however I think they will be a issue with the use of DELETE/CREATE event where during a small time frame api with return 404. It should be better to use a UPDATE Event to first deploy the new access point and then remove the old one on gateway side. |
40bd8ec
to
5e86b1d
Compare
8e6fc01
to
6648dea
Compare
...va/io/gravitee/repository/mongodb/management/internal/domain/AccessPointMongoRepository.java
Outdated
Show resolved
Hide resolved
...va/io/gravitee/repository/mongodb/management/internal/domain/AccessPointMongoRepository.java
Outdated
Show resolved
Hide resolved
...va/io/gravitee/repository/mongodb/management/internal/domain/AccessPointMongoRepository.java
Outdated
Show resolved
Hide resolved
12c5771
to
8be11e4
Compare
8be11e4
to
8afdd30
Compare
Issue
https://gravitee.atlassian.net/browse/CJ-1748
https://gravitee.atlassian.net/browse/CJ-1754
Description
Context in this epic - https://gravitee.atlassian.net/browse/CJ-1748
Gateway synchronization of access points, only synchronization in this PR, actual usage of the access points not implemented yet
📚 View the storybook of this branch here