Skip to content

Fix local deploy and pin new version - #1454

Merged
cyberantonz merged 2 commits into
constructorfabric:mainfrom
cyberantonz:fix/seed-placeholder-drift
Jun 24, 2026
Merged

Fix local deploy and pin new version#1454
cyberantonz merged 2 commits into
constructorfabric:mainfrom
cyberantonz:fix/seed-placeholder-drift

Conversation

@cyberantonz

@cyberantonz cyberantonz commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated release automation to ensure consistent version tracking across deployment configurations
    • Version bumped to 0.2.14
    • Enhanced local environment configuration to improve deployment stability

Signed-off-by: Anton Zelenov <antonz@constructor.tech>
@cyberantonz
cyberantonz requested a review from a team as a code owner June 23, 2026 17:00
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c4462ff7-f8fa-42f0-8c3e-9cd254be10a2

📥 Commits

Reviewing files that changed from the base of the PR and between 5f3df70 and cfc8699.

📒 Files selected for processing (3)
  • .github/workflows/build-images.yml
  • deploy/gitops/.insight-version
  • deploy/gitops/environments/local/values.yaml.template

📝 Walkthrough

Walkthrough

The CI publish-chart job is extended to write the computed umbrella version into deploy/gitops/.insight-version (mirroring the existing docs/deploy/ file) and stage it in the release commit. The deploy/gitops/.insight-version file is updated to 0.2.14. The local GitOps values template gains a hardcoded demo tenant GUID under credentials.global.tenantDefaultId.

Changes

GitOps version mirroring and local tenant config

Layer / File(s) Summary
CI: mirror and commit deploy/gitops/.insight-version
.github/workflows/build-images.yml, deploy/gitops/.insight-version
The version-bump step conditionally writes deploy/gitops/.insight-version when that file exists (matching the existing docs/deploy/ mirror pattern), and the release commit step adds it to the git add list. The version file itself reflects the current value 0.2.14.
Local env: add tenantDefaultId for health probe
deploy/gitops/environments/local/values.yaml.template
Adds credentials.global.tenantDefaultId with a seeded demo tenant GUID; comments document that compose-app-secrets.sh reads this to populate ANALYTICS__metric_catalog__tenant_default_id, preventing tenant-resolution HTTP 400 errors on the unauthenticated /health probe.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

  • constructorfabric/insight#1298: Also modifies the publish-chart job in .github/workflows/build-images.yml, changing the chart push flow to extract the pushed digest and attest provenance.
  • constructorfabric/insight#1303: Also modifies publish-chart steps in .github/workflows/build-images.yml to fix umbrella chart appVersion computation.
  • constructorfabric/insight#1379: Also touches deploy/gitops/environments/local/values.yaml.template around .global.tenantDefaultId injection.

Poem

🐇 A version file tucked in gitops land,
Now bumped by CI with an automated hand.
A tenant GUID seeds the local health probe,
No 400 errors crossing the globe.
The rabbit hops forward — release deployed!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: fixing local deployment configuration and pinning a new version across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cyberantonz
cyberantonz merged commit 43982ec into constructorfabric:main Jun 24, 2026
17 checks passed
@cyberantonz
cyberantonz deleted the fix/seed-placeholder-drift branch June 24, 2026 08:45
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