Skip to content

Add Pub/Sub publishing to CI metadata upload#4063

Open
openroad-ci wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:pubsub-upload
Open

Add Pub/Sub publishing to CI metadata upload#4063
openroad-ci wants to merge 2 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:pubsub-upload

Conversation

@openroad-ci
Copy link
Copy Markdown
Collaborator

Summary

  • Dual-write build metrics to both Firestore and GCP Pub/Sub during CI runs
  • Pub/Sub publishing is opt-in (requires --pubsubProjectID) and non-blocking — failures are logged but don't affect Firestore uploads
  • Firestore and Pub/Sub code paths are clearly separated with marker comments for future Firebase removal
  • Fix rules file path (metadata.json -> rules-{variant}.json) so design validation rules are actually loaded
  • Bump jenkins-ci shared library to orfs-v2.3.8

Test plan

  • Tested locally against Pub/Sub emulator via palantir AppHost
  • Verified all 58 designs across 6 platforms published and ingested successfully
  • Verified metric rules are correctly loaded and ingested
  • Verify pubsub-publisher-svc Jenkins credential is configured

@sombraSoft
Copy link
Copy Markdown
Contributor

sombraSoft commented Mar 28, 2026

Metrics were successfully uploaded to both current dashboard and the beta dashboard ✅
Current dashboard PR-4063-head
Beta dashboard PR-4063-head

Dual-write build metrics to both Firestore and GCP Pub/Sub.
Pub/Sub publishing is opt-in (requires --pubsubProjectID) and
non-blocking. Firestore and Pub/Sub code paths are clearly
separated for future Firebase removal.

Also fix rules file path (metadata.json -> rules-{variant}.json)
and bump jenkins-ci library to orfs-v2.3.8.

Signed-off-by: Joao Luis Sombrio <sombrio@sombrasoft.dev>
Signed-off-by: Joao Luis Sombrio <sombrio@sombrasoft.dev>
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