Skip to content

feat: add registry store for tracking agent registrations#104

Open
jithinkunjachan wants to merge 3 commits intomainfrom
feat/add-registraion-store
Open

feat: add registry store for tracking agent registrations#104
jithinkunjachan wants to merge 3 commits intomainfrom
feat/add-registraion-store

Conversation

@jithinkunjachan
Copy link
Copy Markdown
Contributor

@jithinkunjachan jithinkunjachan commented Apr 10, 2026

Introduce a PostgreSQL-backed registry store that records agent instances and their heartbeat status during registration.
The agent API now requires an X-Agent-Id header and upserts registry entries on each registration request.

Introduce a PostgreSQL-backed registry store that records agent
instances and their heartbeat status during registration. The agent
API now requires an X-Agent-Id header and upserts registry entries
on each registration request.

# Conflicts:
#	pkg/api/agents/client.go
@jithinkunjachan jithinkunjachan force-pushed the feat/add-registraion-store branch from 1ec3e72 to 404b8b6 Compare April 10, 2026 07:24
@jithinkunjachan jithinkunjachan self-assigned this Apr 10, 2026
@jithinkunjachan jithinkunjachan marked this pull request as ready for review April 10, 2026 07:24
@push-tags-from-workflow push-tags-from-workflow bot added dependencies Pull requests that update a dependency file tests feature labels Apr 10, 2026
Copy link
Copy Markdown

@apatsap apatsap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Just some naming remarks

@jithinkunjachan jithinkunjachan marked this pull request as draft April 10, 2026 12:49
@jithinkunjachan jithinkunjachan requested a review from apatsap April 10, 2026 13:32
@jithinkunjachan jithinkunjachan marked this pull request as ready for review April 10, 2026 13:32
Copy link
Copy Markdown

@apatsap apatsap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Turned out nicely!

Copy link
Copy Markdown
Contributor

@fabenan-f fabenan-f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file feature tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants