chore(product-console): merge develop into main#1161
Merged
ferr3ira-gabriel merged 12 commits intomainfrom Apr 2, 2026
Merged
Conversation
Add support for dynamic OpenTelemetry configuration. When otel.enabled=true OR otel.external=true, the chart injects OTEL_URL_* env vars: - If otel.host is set: uses static host (e.g., ClusterIP Service) - If otel.host is empty: injects HOST_IP from pod's node (for DaemonSet with hostPort) Also adds fail-fast validation: if ENABLE_TELEMETRY is 'true' but neither otel.enabled nor otel.external is set, helm template fails with a clear error. This matches the pattern used by midaz chart and supports both: - External OTEL collector with ClusterIP Service - External OTEL collector DaemonSet with hostPort
…host-injection feat(product-console): add dynamic OTEL host injection support
## (2026-04-01) ### ⚠ BREAKING CHANGES * **plugin-access-manager:** Values path changed from 'auth.backend.migrations.image' (string) to 'auth.backend.migrations.image.repository' + '.tag' (object). Same for 'auth.initUser.image' and 'auth.initUser.imagePullPolicy'. Existing values overrides using the old string format will need updating. ### Features * **product-console:** add dynamic OTEL host injection support ([4f473e9](4f473e9)) * add IAM Roles Anywhere sidecar support for fetcher and matcher ([a67b756](a67b756)) * **plugin-br-pix-indirect-btg:** add missing inbound webhook entity envs and security tier config ([ca62f49](ca62f49)) * **plugin-br-bank-transfer:** move CLIENT_IDs to secrets ([d3e36c0](d3e36c0)) * **plugin-br-pix-indirect-btg:** update plugin-br-pix-indirect-btg@1.5.0, plugin-br-pix-indirect-btg-worker-reconciliation@1.5.0, plugin-br-pix-indirect-btg-worker-inbound@1.5.0, plugin-br-pix-indirect-btg-worker-outbound@1.5.0 - new env vars ([80fe4fc](80fe4fc)) ### Bug Fixes * **charts:** add http:// prefix to OTEL_EXPORTER_OTLP_ENDPOINT ([33adde3](33adde3)) * **charts:** add http:// prefix to OTEL_EXPORTER_OTLP_ENDPOINT ([5f8ca7b](5f8ca7b)) * add kindIs guard for backward compat with string image values ([313d9f6](313d9f6)) * **matcher:** add missing env vars for systemplane and multi-tenant ([965da94](965da94)) * add required validation for Roles Anywhere ARNs and README migration note ([50e4718](50e4718)), closes [#1](#1) [#3](#3) [#1113](#1113) * **plugin-br-pix-indirect-btg:** add WEBHOOK_DEFAULT_URL to outbound configmap ([0dd6d81](0dd6d81)) * **matcher:** align default securityContext with distroless nonroot UID ([87f0c59](87f0c59)) * **matcher:** fix configmap archival condition and S3 endpoint for IAM Roles Anywhere ([53232ae](53232ae)) * **reporter:** prevent null env in manager and worker deployments ([51a72c6](51a72c6)) * **reporter:** prevent null env in ScaledJob when no env vars are configured ([7fa29b4](7fa29b4)) * **plugin-br-bank-transfer:** rename chart to include -helm suffix ([f393e35](f393e35)) * **plugin-access-manager:** split migrations and initUser image into repository/tag fields ([5be206a](5be206a)) * **plugin-br-pix-indirect-btg:** update plugin-br-pix-indirect-btg@1.4.1, plugin-br-pix-indirect-btg-worker-reconciliation@1.4.1, plugin-br-pix-indirect-btg-worker-inbound@1.4.1, plugin-br-pix-indirect-btg-worker-outbound@1.4.1 ([3c6970f](3c6970f)) * **plugin-br-pix-indirect-btg:** update plugin-br-pix-indirect-btg@1.5.1, plugin-br-pix-indirect-btg-worker-reconciliation@1.5.1, plugin-br-pix-indirect-btg-worker-inbound@1.5.1, plugin-br-pix-indirect-btg-worker-outbound@1.5.1 ([226f506](226f506)) * **matcher:** update securityContext to run as root ([8de4a13](8de4a13)) * **matcher:** use configmap value for OBJECT_STORAGE_ENDPOINT with IAM Roles Anywhere ([c0f19a4](c0f19a4)) * **reporter:** use unique names for cluster-scoped resources ([5cdaa80](5cdaa80))
Add MONGO_PARAMETERS configmap entry to allow passing additional MongoDB connection string parameters (e.g., tls=true, authSource=admin). Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
…-parameters feat(product-console): add MONGO_PARAMETERS env var for TLS/auth options
…sfer/v2.1.0-20260331145344 fix(plugin-br-bank-transfer): update plugin-br-bank-transfer@2.1.0
## (2026-04-01) ### Features * **product-console:** add dynamic OTEL host injection support ([4f473e9](4f473e9)) * **product-console:** add MONGO_PARAMETERS env var for TLS/auth options ([164468c](164468c)) ### Bug Fixes * **plugin-br-bank-transfer:** update plugin-br-bank-transfer@2.1.0 ([3e29c6c](3e29c6c)) * **plugin-br-pix-indirect-btg:** update plugin-br-pix-indirect-btg@1.5.1, plugin-br-pix-indirect-btg-worker-reconciliation@1.5.1, plugin-br-pix-indirect-btg-worker-inbound@1.5.1, plugin-br-pix-indirect-btg-worker-outbound@1.5.1 ([226f506](226f506)) * **matcher:** use configmap value for OBJECT_STORAGE_ENDPOINT with IAM Roles Anywhere ([c0f19a4](c0f19a4))
- Update image tag from 1.3.0 to 1.5.0 - Update appVersion to 1.5.0 - Add UPGRADE-2.0.md documentation for v2.0 migration Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
…-1.5.0 feat(product-console): update image tag to 1.5.0
WalkthroughThe pull request updates version mappings in the README and bumps two Helm charts to beta versions. Plugin BR Bank Transfer chart version is updated to 1.1.0-beta.1 with app version 2.1.0. Product Console chart is updated to version 2.0.0-beta.3 with app version 1.5.0, including OpenTelemetry configuration refactoring to support dynamic endpoint injection. Changes
Comment |
guimoreirar
approved these changes
Apr 2, 2026
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@charts/plugin-br-bank-transfer/values.yaml`:
- Around line 369-375: The optional secret comments (RABBITMQ_PASSWORD,
RABBITMQ_EVENT_SIGNING_SECRET, WEBHOOK_SIGNING_SECRET,
WEBHOOK_BROKER_EVENT_SIGNING_SECRET) should be moved so they are indented under
the bankTransfer.secrets block in values.yaml; update the comment placement and
indentation to nest them beneath bankTransfer.secrets (rather than at the
top-level bankTransfer) so it's clear these are optional secret values for
bankTransfer.
In `@charts/product-console/docs/UPGRADE-2.0.md`:
- Around line 223-224: Update the two Helm command examples in UPGRADE-2.0.md
that currently use the literal "--version 2.0.0" to the published chart version
"--version 2.0.0-beta.3" so the documented upgrade command matches the actual
release; change both occurrences of the "--version 2.0.0" token in the file (the
two Helm install/upgrade examples) to "--version 2.0.0-beta.3".
In `@README.md`:
- Line 102: The README mapping row currently shows `1.1.0-beta.1` ->
`1.0.0-beta.1`; update that mapped app version to match the chart release by
replacing the right-hand value with `2.1.0` (this release sets
charts/plugin-br-bank-transfer/Chart.yaml appVersion and
charts/plugin-br-bank-transfer/values.yaml default image tag to 2.1.0). Locate
the table cell containing the string `1.0.0-beta.1` in README.md and change it
to `2.1.0` so the README accurately reflects the chart's appVersion and default
image tag.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 13d6169f-6627-4cd8-a85e-7197f2314cb6
📒 Files selected for processing (7)
README.mdcharts/plugin-br-bank-transfer/Chart.yamlcharts/plugin-br-bank-transfer/values.yamlcharts/product-console/Chart.yamlcharts/product-console/docs/UPGRADE-2.0.mdcharts/product-console/templates/deployment.yamlcharts/product-console/values.yaml
guimoreirar
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Merge develop branch into main.
Changes