Skip to content

Do not require bug ID on docker compose up#6325

Open
evgenyrp wants to merge 1 commit into
mozilla:masterfrom
evgenyrp:bug_id_fails_docker_compose
Open

Do not require bug ID on docker compose up#6325
evgenyrp wants to merge 1 commit into
mozilla:masterfrom
evgenyrp:bug_id_fails_docker_compose

Conversation

@evgenyrp

Copy link
Copy Markdown
Contributor

This fails other agents that don't require BUG_ID when running from a root compose that nicely loads .env.

FAILURE_TASKS='{"build-linux":"GFjLPsWwRDiMhtFprcwyGQ"}' docker compose up build-repair-agent --build

@evgenyrp
evgenyrp requested review from a team as code owners July 16, 2026 15:48
@suhaibmujahid

Copy link
Copy Markdown
Member

What if we drop the linking to the root compose file? It might be a better route long run. WDYT?

@evgenyrp

Copy link
Copy Markdown
Contributor Author

What if we drop the linking to the root compose file? It might be a better route long run. WDYT?

We could; it's a matter of convenience. Then we should update the README to use docker compose with -f flag to point to specific compose files. And also, a developer would need to load .env manually, which is a bit less convenient. Maybe we could point each individual compose to .env

@suhaibmujahid

Copy link
Copy Markdown
Member

We could; it's a matter of convenience. Then we should update the README to use docker compose with -f flag to point to specific compose files. And also, a developer would need to load .env manually, which is a bit less convenient. Maybe we could point each individual compose to .env

The agent's compose could be loaded from inside the agent's directory. With that, you do not need -f.

For the .env, we could make it point to the root .env and adjacent .env at the same time. I think this need be don from the agent's compose file itself.

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