Skip to content

resolve service environment "just in time" #12946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Jun 16, 2025

What I did
resolve service environment just before container get created
doing so, a provider service depends on can inject variables, used for interpolation
example usage:

services: Docker Desktop   o View Config   w Enable Watch
  test:
    image: ...
    environment:
      - OPENAI_API_BASE=${LLM_URL}
    depends_on:
      - llm

  llm:
    provider:
      type: model
      options:
        model: llama123

note this is just a draft, need to play with this branch to better understand the possible side effects

Related issue
requires compose-spec/compose-go#797

(not mandatory) A picture of a cute animal, if possible in relation to what you did

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof force-pushed the JIT_env_resolve branch 2 times, most recently from c57493a to 1f8a052 Compare June 18, 2025 15:25
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
ndeloof and others added 2 commits June 18, 2025 18:11
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Signed-off-by: Guillaume Lours <705411+glours@users.noreply.github.com>
@glours glours force-pushed the JIT_env_resolve branch from 501f7d8 to cc22628 Compare June 20, 2025 09:15
glours and others added 2 commits June 20, 2025 11:41
Signed-off-by: Guillaume Lours <705411+glours@users.noreply.github.com>
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
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