Skip to content

fix: log envs are imported already#405

Merged
bosbaber merged 1 commit intomainfrom
stephan/20260331-logs-config
Mar 31, 2026
Merged

fix: log envs are imported already#405
bosbaber merged 1 commit intomainfrom
stephan/20260331-logs-config

Conversation

@bosbaber
Copy link
Copy Markdown
Collaborator

No description provided.

@bosbaber bosbaber requested a review from a team as a code owner March 31, 2026 12:18
Copilot AI review requested due to automatic review settings March 31, 2026 12:18
@bosbaber bosbaber self-assigned this Mar 31, 2026
@bosbaber bosbaber merged commit 9696b61 into main Mar 31, 2026
4 checks passed
@bosbaber bosbaber deleted the stephan/20260331-logs-config branch March 31, 2026 12:19
github-actions bot added a commit that referenced this pull request Mar 31, 2026
Automated chart versioning after merge of PR #405.
This PR updates Chart.yaml versions, packages updated charts into
docs/interledger,
and regenerates the Helm index.

Co-authored-by: bosbaber <1615407+bosbaber@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes redundant LOG_LEVEL / LOG_PRETTY environment variable entries from the Interledger App frontend chart values, relying on the existing envFrom configMap import instead.

Changes:

  • Removed explicit LOG_LEVEL and LOG_PRETTY entries from server.env in values.yaml since they are already provided via server.envFrom (configMap {{ include "common.fullname" . }}-server).

@bosbaber bosbaber restored the stephan/20260331-logs-config branch March 31, 2026 12:58
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