Skip to content

[CRE] OrgResolver with DB-backed cache - #23485

Merged
bolekk merged 1 commit into
developfrom
org_cache
Sep 8, 2026
Merged

bolekk merged 1 commit into
developfrom
org_cache

Conversation

@bolekk

@bolekk bolekk commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

https://smartcontract-it.atlassian.net/browse/CRE-5348

Use the caching OrgResolver implemented in smartcontractkit/chainlink-common#2342
Inject a DB-based cache. Enabled by default with a config flag to disable.

Deployment validation:

  1. Check metric org_resolver_cache_lookups_total with label "result". Expect a spike of "miss" initially and later only "hit"s. No errors.
  2. Metric org_resolver_mapping_changed should be always zero.
  3. No warning or error logs coming out of caching.go

@github-actions

Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@bolekk
bolekk force-pushed the org_cache branch 2 times, most recently from 6dc7da5 to 4d79e91 Compare September 6, 2026 04:16
@bolekk bolekk changed the title [CRE] OrgResolver cache [CRE] OrgResolver with DB-backed cache Sep 6, 2026
@trunk-io

trunk-io Bot commented Sep 6, 2026 •

Copy link
Copy Markdown

Static Badge   Static Badge   Static Badge

View Full Report ↗︎ ⋅ Docs

@bolekk
bolekk force-pushed the org_cache branch 2 times, most recently from e3e706a to bf47f10 Compare September 6, 2026 15:10
@bolekk
bolekk marked this pull request as ready for review September 6, 2026 15:24
@bolekk
bolekk requested review from a team as code owners September 6, 2026 15:24
@bolekk
bolekk requested review from ibrajer, jmank88, patrickhuie19 and pavel-raykov and removed request for skudasov September 6, 2026 15:24
Comment thread core/config/docs/core.toml Resolved
Comment thread .changeset/org-resolver-cache.md Resolved
pavel-raykov
pavel-raykov previously approved these changes Sep 7, 2026
pavel-raykov
pavel-raykov previously approved these changes Sep 7, 2026
Comment thread core/services/cre/org_resolver_store.go Resolved
CREATE TABLE IF NOT EXISTS cre.org_resolver_cache (
workflow_owner TEXT NOT NULL PRIMARY KEY,
org_id TEXT NOT NULL,
updated_at TIMESTAMPTZ NOT NULL DEFAULT NOW()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

a created_at may be useful to extend the o11y layer

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

will add

patrickhuie19
patrickhuie19 previously approved these changes Sep 8, 2026
Cache owner->orgID mappings in Postgres (CRE-5348). DurableCacheEnabled defaults to true. Requires chainlink-common with the merged caching OrgResolver refactor (#2342).
@cl-sonarqube-production

Copy link
Copy Markdown

@bolekk
bolekk added this pull request to the merge queue Sep 8, 2026
Merged via the queue into develop with commit 940eb99 Sep 8, 2026
217 checks passed
@bolekk
bolekk deleted the org_cache branch September 8, 2026 15:57
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.

3 participants