Skip to content

[product-catalog] use database for products#2859

Merged
julianocosta89 merged 8 commits intoopen-telemetry:mainfrom
puckpuck:product-catalog-database
Jan 7, 2026
Merged

[product-catalog] use database for products#2859
julianocosta89 merged 8 commits intoopen-telemetry:mainfrom
puckpuck:product-catalog-database

Conversation

@puckpuck
Copy link
Copy Markdown
Contributor

@puckpuck puckpuck commented Jan 6, 2026

Changes

  • Adds a postgres DB schema for the product catalog.
  • The product catalog service will make a database query on each request (List, Get, Search). No more local cache for the product catalog that gets refereshed every 10 seconds
  • All database calls are instrumented using XSAM/otelsql with recent semantic conventions applied
  • Updates the Postgres Docker deployment to use the official Postgres image with a mounted volume for the init db, instead of building an image with this file for the demo. We can use a configmap to achieve similar in Helm/K8s.

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the [docs][]
  • Appropriate Helm chart updates in the [helm-charts][]

@puckpuck puckpuck requested a review from a team as a code owner January 6, 2026 19:02
@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Jan 6, 2026
@julianocosta89 julianocosta89 merged commit 562ff1c into open-telemetry:main Jan 7, 2026
34 checks passed
@puckpuck puckpuck deleted the product-catalog-database branch January 7, 2026 13:17
kkn pushed a commit to liteverge/opentelemetry-demo that referenced this pull request Jan 11, 2026
* use database for product catalog

* remove RELOAD_INTERVAL

* use postgres database for product-catalog

* fix trailing space

* remove postgres build image

* remove copy for products json

---------

Co-authored-by: Juliano Costa <julianocosta89@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

helm-update-required Requires an update to the Helm chart when released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants