Skip to content

fix(docker): move entrypoint script to /app to avoid tmpfs masking - #8999

Merged
diegosouzapw merged 6 commits into
cheaper-inference:release/v3.8.50from
yutuknown:release/v3.8.50
Aug 6, 2026
Merged

diegosouzapw merged 6 commits into
cheaper-inference:release/v3.8.50from
yutuknown:release/v3.8.50

Conversation

@yutuknown

Copy link
Copy Markdown
Contributor

Render and other PaaS platforms mount a tmpfs over /tmp, masking the entrypoint script and causing an exit 128 crash on boot.

@yutuknown
yutuknown requested a review from diegosouzapw as a code owner July 30, 2026 17:08
@diegosouzapw
diegosouzapw merged commit 1918156 into cheaper-inference:release/v3.8.50 Aug 6, 2026
3 checks passed
diegosouzapw added a commit that referenced this pull request Aug 6, 2026
…ned suites

Continuing the base-red drain on release/v3.8.50.

- src/i18n/messages/{en,pt-BR,vi}.json: #8523's manual-API-key UI calls
  noAuthProvider.providerAdded / .add / .manualApiKey, but the keys were never
  added to en.json — 'direct translation calls have English messages' was red.
  Added to en (source of truth) plus the two parity-tested locales.
  settings-i18n-keys 10/10.
- src/i18n/messages/pt-BR.json: same 3 keys the vi locale was missing
  (providers.onboardingProviderDescriptions.unorouter, agentBridge.noMappingsDesc,
  agentBridge.addMapping), translated. i18n-pt-br 3/3, i18n-vi-completeness 5/5.
- tests/unit/tls-client-node-docker-binary-7802.test.ts: the assertion pinned the
  literal flag list of the builder's npm ci. #9185 inserted --include=optional
  (LLMLingua optional deps) and broke the pin without touching intent, which is
  the --ignore-scripts PRECONDITION. Regex is now flag-order tolerant and still
  requires --ignore-scripts on the same npm ci. 1/1.
- tests/unit/runtime-env.test.ts: #8999 made withRuntimePortEnv also pin the bind
  address (HOSTNAME, defaulting to 0.0.0.0 when OMNIROUTE_HOSTNAME is unset).
  Expectation updated to the real shape. 2/2.

Refs #9298
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…heaper-inference#8999)

Validated in local merge-train (devbox-vm-06-dev002) @ combined-tip (FAST gates green: static + changed tests + vitest — only pre-existing audit.test.ts flake). Evidence: /home/diegosouzapw/dev/proxys/OmniRoute/.claude/worktrees/merge-train-20260805-213228-suite.log
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