restatedev / cdk

MIT License
5 stars 4 forks source link

Implement deregistration of services from Restate #5

Open pcholakov opened 11 months ago

pcholakov commented 11 months ago

Fix several known issues in register-service-handler.ts as of 2023-11-20.

The registration response looks like this:

% curl -X POST http://localhost:9070/endpoints -H 'content-type: application/json' -d '{"arn": "arn:aws:lambda:eu-central-1:663487780041:function:pavel-RestateStack-GreeterA41DC22A-o8knTdFWTHtj:3"}' -m 3
{"id":"YXJuOmF3czpsYW1iZGE6ZXUtY2VudHJhbC0xOjY2MzQ4Nzc4MDA0MTpmdW5jdGlvbjpwYXZlbC1SZXN0YXRlU3RhY2stR3JlZXRlckE0MURDMjJBLW84a25UZEZXVEh0ajoz","services":[{"name":"Greeter","revision":1}]}
### Tasks
- [ ] Handle CloudFormation `Delete` commands - deregister services from Restate
- [x] Add more robust retry and timeout handling to both health check and registration calls
- [x] Validate the endpoint registration response, return (some) output to CFn