Skip to content

Bind market-maker health on IPv4 and verify the running image - #83

Merged
LumerinIO merged 4 commits into
mainfrom
dev
Sep 22, 2026
Merged

LumerinIO merged 4 commits into
mainfrom
dev

Conversation

@abs2023

@abs2023 abs2023 commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The market-maker health server listens on IPv4 (0.0.0.0) so the load balancer can reach it.
  • /health reports imageTag, and the deploy verify job compares the running task image to the tag this run built.

Test plan

  • Dev deploy of v1.4.2-dev completed on tsk-col-mar-futures-mm:42. The target is healthy.
  • The verify job failed after the service was already stable: the GitHub Actions role can ecs:DescribeServices and ecs:DescribeTaskDefinition, and it cannot ecs:ListTasks or ecs:DescribeTasks. That failure is the new check, not a rollback.
  • Grant those two actions (dev and LMN) before relying on the verify job on main. Merging this as-is will deploy the image and then fail the same check.

Made with Cursor

abs2023 and others added 4 commits September 22, 2026 14:12
The load balancer checks the task IPv4 address. listen(port) on the current node image binds IPv6 only, so those checks time out and ECS rolls the task back.

Co-authored-by: Cursor <cursoragent@cursor.com>
Bind the market-maker health server on IPv4
…st built

/health now reports imageTag. The verify job compares the running task image and that tag to the release, so a rollback can no longer pass as a stable deploy.

Co-authored-by: Cursor <cursoragent@cursor.com>
Fail market-maker verify unless the running image matches
@LumerinIO
LumerinIO merged commit baa4211 into main Sep 22, 2026
20 of 22 checks passed

This branch was successfully deployed

1 active deployment
dev e63cc70d Deployed Sep 22, 2026 by abs2023 via 🚀 Deploy portfolio #50
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.

2 participants