Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 15, 2025

The graph command didn't accept the --updater-env flag that update already supports for passing environment variables to the updater container.

Changes

  • graph.go: Added --updater-env flag registration and wired it through to infra.Run()
  • test.go: Wired existing --updater-env flag through to infra.Run() (flag was registered but not forwarded)

Usage is now consistent across commands:

dependabot graph go_modules dependabot/cli -e FOO=bar -e BAZ=qux
dependabot update go_modules dependabot/cli -e FOO=bar -e BAZ=qux
dependabot test -f smoke.yml -e FOO=bar -e BAZ=qux

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • 172.20.0.1
    • Triggering command: REDACTED, pid is -1 (packet block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Have graph command take --updater-env</issue_title>
<issue_description>The update command takes -e, --updater-env stringArray additional environment variables to set in the update container. The graph command doesn't accept that param.

Make it so that it does, and forward those appropriately.</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add --updater-env flag registration to graph command
- Pass UpdaterEnvironmentVariables to infra.Run() in graph command
- Fix test command to pass UpdaterEnvironmentVariables to infra.Run()
- Both commands now properly forward environment variables to the updater container

Co-authored-by: jurre <749864+jurre@users.noreply.github.com>
Copilot AI changed the title [WIP] Add --updater-env option to graph command Add --updater-env flag to graph command Dec 15, 2025
Copilot AI requested a review from jurre December 15, 2025 17:28
@jurre jurre marked this pull request as ready for review December 15, 2025 17:29
@jurre jurre requested review from a team as code owners December 15, 2025 17:29
@jurre jurre added this pull request to the merge queue Dec 15, 2025
Merged via the queue into main with commit 7c11dec Dec 15, 2025
89 checks passed
@jurre jurre deleted the copilot/add-updater-env-to-graph-command branch December 15, 2025 18:56
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.

Have graph command take --updater-env

4 participants