Skip to content

fix(plugin-fees): remove MULTI_TENANT_SETTINGS_CHECK_INTERVAL_SEC#1232

Merged
jeffersonrodrigues92 merged 1 commit intodevelopfrom
fix/remove-multi-tenant-settings-check-interval
Apr 17, 2026
Merged

fix(plugin-fees): remove MULTI_TENANT_SETTINGS_CHECK_INTERVAL_SEC#1232
jeffersonrodrigues92 merged 1 commit intodevelopfrom
fix/remove-multi-tenant-settings-check-interval

Conversation

@gandalf-at-lerian
Copy link
Copy Markdown
Contributor

Removes MULTI_TENANT_SETTINGS_CHECK_INTERVAL_SEC from plugin-fees configmap template and default values.

Requested by: @jeffersonrodrigues-lerian

…m configmap

Requested-by: @jeffersonrodrigues-lerian
@gandalf-at-lerian gandalf-at-lerian requested a review from a team as a code owner April 17, 2026 14:01
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2f9ee544-bca3-4ac9-8a3d-ba2b5a1833de

📥 Commits

Reviewing files that changed from the base of the PR and between 2ef3048 and f3e7f9b.

📒 Files selected for processing (2)
  • charts/plugin-fees/templates/fees/configmap.yaml
  • charts/plugin-fees/values.yaml
💤 Files with no reviewable changes (2)
  • charts/plugin-fees/values.yaml
  • charts/plugin-fees/templates/fees/configmap.yaml

Walkthrough

Removed the MULTI_TENANT_SETTINGS_CHECK_INTERVAL_SEC environment variable from the plugin-fees Helm chart configuration. The entry was deleted from both the ConfigMap template and the values defaults file.

Changes

Cohort / File(s) Summary
Multi-tenant Configuration Removal
charts/plugin-fees/templates/fees/configmap.yaml, charts/plugin-fees/values.yaml
Removed MULTI_TENANT_SETTINGS_CHECK_INTERVAL_SEC environment variable definition and its default value ("60") from both the ConfigMap template and values file. Other multi-tenant configuration entries remain unchanged.

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

@jeffersonrodrigues92 jeffersonrodrigues92 merged commit fc6abba into develop Apr 17, 2026
3 checks passed
@jeffersonrodrigues92 jeffersonrodrigues92 deleted the fix/remove-multi-tenant-settings-check-interval branch April 17, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants