Added Webhooks to event store
- [ ] This change is worth documenting at https://docs.all-hands.dev/
- [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below
End-user friendly description of the problem this fixes or functionality this introduces.
Summarize what the PR does, explaining any non-trivial design decisions.
Link of any specific issues this addresses:
To run this PR locally, use the following command:
docker run -it --rm -p 3000:3000 -v /var/run/docker.sock:/var/run/docker.sock --add-host host.docker.internal:host-gateway -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:06bba48-nikolaik --name openhands-app-06bba48 docker.all-hands.dev/all-hands-ai/openhands:06bba48
I backed it out the previous attempt because we decided to go in a different direction and there was no explicit request from the community for it - Leaving it in meant that we would have to build / test / document both sides of a complex custom protocol, and the webhook based solution matched our needs and would likely be more useful to others too. We figured that keeping events in a local file store with a process for external synchronization was far more efficient than using a pure network attached event store.
I'll update the documentation with the with implementation details.
Looks like there are a few issues preventing this PR from being merged!
- GitHub Actions are failing:
- Run Python Unit Tests
If you'd like me to help, just leave a comment, like
@OpenHands please fix the failing actions on PR #8763
Feel free to include any additional details that might help me get this PR into a better state.
You can manage your notification settings