Skip to content

Conversation

@whoAbhishekSah
Copy link
Member

Summary

Change env prefix from FRONTIER to FRONTIER_SERVICE to prevent collision with Kubernetes service environment variables.

Problem

Kubernetes automatically creates environment variables for services. For a service named frontier-app, K8s creates:

FRONTIER_APP_PORT=tcp://172.20.58.102:80
FRONTIER_APP_SERVICE_HOST=172.20.58.102
FRONTIER_APP_SERVICE_PORT=80

With Viper's FRONTIER prefix, FRONTIER_APP_PORT maps to app.port and overrides config file values. This causes errors like:

cannot parse 'app.port' as int: strconv.ParseInt: parsing "tcp://172.20.58.102:80": invalid syntax

Solution

Change env prefix to FRONTIER_SERVICE. Now env vars need to be FRONTIER_SERVICE_APP_PORT to affect app.port, avoiding the collision with K8s-generated env vars.

Breaking Change

Existing deployments using env vars with FRONTIER_ prefix will need to update to FRONTIER_SERVICE_ prefix. However, most K8s deployments use config files via -c flag, so impact should be minimal.

Change env prefix from FRONTIER to FRONTIER_SERVICE to prevent collision
with Kubernetes service environment variables.

K8s automatically creates env vars for services (e.g., FRONTIER_APP_PORT
for a service named frontier-app). With Viper's FRONTIER prefix, this
maps to app.port and overrides config file values with invalid URLs like
"tcp://172.20.58.102:80".

With FRONTIER_SERVICE prefix, env vars need to be FRONTIER_SERVICE_APP_PORT
to affect app.port, avoiding the collision.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
frontier Ready Ready Preview, Comment Jan 12, 2026 10:09am

@coveralls
Copy link

Pull Request Test Coverage Report for Build 20915422591

Details

  • 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 37.822%

Totals Coverage Status
Change from base Build 20841098506: 0.003%
Covered Lines: 15814
Relevant Lines: 41812

💛 - Coveralls

@whoAbhishekSah whoAbhishekSah merged commit 52877bb into main Jan 13, 2026
7 checks passed
@whoAbhishekSah whoAbhishekSah deleted the fix/env-prefix-collision branch January 13, 2026 04:19
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.

4 participants