Skip to content

Commit

Permalink
Add APT proxy and document use case
Browse files Browse the repository at this point in the history
  • Loading branch information
usimd committed Nov 4, 2024
1 parent afb6cdd commit 4508232
Show file tree
Hide file tree
Showing 5 changed files with 105 additions and 1 deletion.
37 changes: 37 additions & 0 deletions .github/workflows/integration-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,34 @@ jobs:
- name: Compile action code before test
run: npm ci && npm run package

- name: Select Debian package mirror
id: select-mirror
# This is just to demonstrate that you could determine the mirror to use
# however you fancy.
run: echo "mirror=debian-archive.trafficmanager.net" >> $GITHUB_OUTPUT

- name: Setup APT proxy on runner
run: |
sudo apt-get install -y nginx --no-install-recommends --no-install-suggests
sudo bash -c 'cat > /etc/nginx/sites-enabled/default <<-EOF
server {
listen 3128;
access_log /var/log/nginx/cache-access.log;
error_log /var/log/nginx/cache-error.log;
allow all;
resolver 127.0.0.11;
location / {
if ($host ~* ^deb\.debian\.org) {
proxy_pass $scheme://${{ steps.select-mirror.outputs.mirror }}$request_uri;
break;
}
proxy_pass $scheme://$host$request_uri;
}
}
EOF'
sudo service apache2 stop
sudo service nginx restart
- name: Run pi-gen build
uses: ./
id: build
Expand All @@ -81,6 +109,8 @@ jobs:
timezone: ${{ env.CONFIG_TIMEZONE }}
pubkey-ssh-first-user: ${{ env.CONFIG_PUBLIC_KEY }}
increase-runner-disk-size: ${{ github.event_name != 'workflow_dispatch' || inputs.increase-runner-disk }}
docker-opts: --add-host=host.docker.internal:host-gateway
apt-proxy: http://host.docker.internal:3128

- name: List working directory
run: tree
Expand Down Expand Up @@ -112,4 +142,11 @@ jobs:
with:
labels: test

- name: Debug APT proxy
if: always()
run: |
sudo tail -n 100 /etc/squid-deb-proxy/squid-deb-proxy.conf
sudo service squid status
sudo tail -n 200 /var/log/syslog /var/log/squid/* /var/log/squid-deb-proxy/*
50 changes: 50 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -314,6 +314,56 @@ jobs:
increase-runner-disk-size: true
```
### Use fast APT proxy for `pi-gen`
If you want to speed up your build by improving the package download speed, you can setup
a local APT proxy and let `pi-gen` use it during the build. The proxy should point to a
fast mirror, on GitHub Actions runners this will _very_ likely be `debian-archive.trafficmanager.net`
hosted on Azure. Make sure, though, that the selected mirror contains your targeted
architecture (`debian-archive.trafficmanager.net` does **not** include `armhf`):
```
jobs:
pi-gen-with-fast-apt-proxy:
runs-on: ubuntu-latest
steps:
- name: Select Debian package mirror
id: select-mirror
# This is just to demonstrate that you could determine the mirror to use
# however you fancy.
run: echo "mirror=debian-archive.trafficmanager.net" >> $GITHUB_OUTPUT

- name: Setup APT proxy on runner
run: |
sudo apt-get install -y nginx --no-install-recommends --no-install-suggests
sudo bash -c 'cat > /etc/nginx/sites-enabled/default <<-EOF
server {
listen 3128;
access_log /var/log/nginx/cache-access.log;
error_log /var/log/nginx/cache-error.log;
allow all;
resolver 127.0.0.11;
location / {
if ($host ~* ^deb\.debian\.org) {
proxy_pass $scheme://${{ steps.select-mirror.outputs.mirror }}$request_uri;
break;
}
proxy_pass $scheme://$host$request_uri;
}
}
EOF'
sudo service nginx restart

- uses: usimd/pi-gen-action@v1
with:
image-name: test
stage-list: stage0 stage1 stage2 custom-stage
pi-gen-dir: ${{ inputs.custom-pi-gen-dir }}
# This is important: make the host service available to the pi-gen container
docker-opts: --add-host=host.docker.internal:host-gateway
apt-proxy: http://host.docker.internal:3128
```
## License
The scripts and documentation in this project are released under the [MIT License](LICENSE)
7 changes: 6 additions & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,12 @@ author: Simon Domke

inputs:
# pi-gen variables

apt-proxy:
description: |
If you require the use of an apt proxy, set it here. This proxy setting will not be included in the image,
making it safe to use an apt-cacher or similar package for development.
required: false
default: ''
image-name:
description: Final image name.
required: true
Expand Down
1 change: 1 addition & 0 deletions src/configure.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ export async function configure(): Promise<PiGenConfig> {
userConfig.setfcap = core.getInput('setfcap') || DEFAULT_CONFIG.setfcap
userConfig.piGenRelease =
core.getInput('pi-gen-release') || DEFAULT_CONFIG.piGenRelease
userConfig.aptProxy = core.getInput('apt-proxy') || DEFAULT_CONFIG.aptProxy

await validateConfig(userConfig)

Expand Down
11 changes: 11 additions & 0 deletions src/pi-gen-config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import * as exec from '@actions/exec'
import * as io from '@actions/io'

export interface PiGenConfig {
aptProxy?: string
imgName: string
piGenRelease: string
release: string
Expand Down Expand Up @@ -220,6 +221,16 @@ export async function validateConfig(config: PiGenConfig): Promise<void> {
}
}
}

if (config.aptProxy) {
try {
new URL(config.aptProxy)
} catch (error) {
throw new Error(
'apt-proxy is not a valid URL. Make it point to a correct http/https address.'
)
}
}
}

function camelCaseToSnakeCase(label: string): string {
Expand Down

0 comments on commit 4508232

Please sign in to comment.