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

Instead of triggering /v1/dispose-remainders-of-deregistered-application , subscription shall be removed for unsuccessful regard-applications #350

Open
PrathibaJee opened this issue Jul 12, 2024 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@PrathibaJee
Copy link
Collaborator

PrathibaJee commented Jul 12, 2024

If OKM could not able to regard an application due to any internal failure ,
then
as of now OKM will send /v1/dispose-remainders-of-deregistered-application
Reason : To ensure that OKM is not sending update-operation-keys to the regarded application's operation-server.
This is modelled so.

Problem :
Incase if the regarded application is one of the TAC and OKM has invariant forwardings for this application.
Then disposal deletes the http-client resulting in non-functioning of the invariant forwardings.

Proposals :
Proposal#1:
For all the application , if it is not regarded successfully , then instead of disposing that application ,
end-subscription shall be initiated to delete the fc-port from the following forwardings,

  1. LinkUpdateNotificationCausesOperationKeyUpdates
  2. CyclicOperationCausesOperationKeyUpdates

Proposal#2:
If there is any invariant forwardings available for the regarded application , then rollback shall not initiate /v1/dispose-remainders-of-deregistered-application , internally it shall delete the subscription from the following forwardings,

  1. LinkUpdateNotificationCausesOperationKeyUpdates
  2. CyclicOperationCausesOperationKeyUpdates
@PrathibaJee
Copy link
Collaborator Author

Temporarily this issue is fixed in the PR #334.
Since the provided proposals add complexity , this issue shall be analyzed deeply in the future releases to provide a simple and transparent solution.

Related issue : #309

@PrathibaJee PrathibaJee added this to the OKM v3.0.0_spec milestone Jul 15, 2024
@PrathibaJee PrathibaJee added the question Further information is requested label Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants