Skip to content

Commit

Permalink
preserve http_proxy in generated github actions sudo invocatons
Browse files Browse the repository at this point in the history
Summary:
X-link: facebookincubator/zstrong#1102

make it possible to run the generated github actions locally with [act](https://github.com/nektos/act) when behind a proxy by preserving http_proxy in generated github actions sudo invocatons

also updated to consistently write `apt-get` instead of the interactive `apt` (we had a mix)

example usage (substitute myproxy for your actual proxy):
```
mkdir $HOME/act-artifacts
cd ~/fizz
time act -r -j build -W .github/workflows/getdeps_linux.yml --artifact-server-path=$HOME/act-artifacts --artifact-server-addr=127.0.0.1 --cache-server-addr=127.0.0.1 --env http_proxy=http://myproxy:8080 --env https_proxy=myproxy:8080 --action-offline-mode
```

Reviewed By: bigfootjon

Differential Revision: D67864689

fbshipit-source-id: 45695675ca5672e56104c8c33803afe004ff98da
  • Loading branch information
ahornby authored and facebook-github-bot committed Jan 7, 2025
1 parent 70ffd57 commit 5a360b0
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions build/fbcode_builder/getdeps.py
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,13 @@ def run_project_cmd(self, args, loader, manifest):
elif manager == "deb":
packages = sorted(set(all_packages["deb"]))
if packages:
cmd_args = ["sudo", "apt", "install", "-y"] + packages
cmd_args = [
"sudo",
"--preserve-env=http_proxy",
"apt-get",
"install",
"-y",
] + packages
elif manager == "homebrew":
packages = sorted(set(all_packages["homebrew"]))
if packages:
Expand Down Expand Up @@ -1155,7 +1161,7 @@ def write_job_for_platform(self, platform, args): # noqa: C901
build_opts.allow_system_packages
and build_opts.host_type.get_package_manager()
):
sudo_arg = "sudo "
sudo_arg = "sudo --preserve-env=http_proxy "
allow_sys_arg = " --allow-system-packages"
if build_opts.host_type.get_package_manager() == "deb":
out.write(" - name: Update system package info\n")
Expand Down

0 comments on commit 5a360b0

Please sign in to comment.