Skip to content

fix(ci): handle missing APIFY_HTTPBIN_TOKEN in musllinux aarch64 Python tests#444

Merged
barjin merged 3 commits into
masterfrom
copilot/fix-python-tests-aarch64
May 12, 2026
Merged

fix(ci): handle missing APIFY_HTTPBIN_TOKEN in musllinux aarch64 Python tests#444
barjin merged 3 commits into
masterfrom
copilot/fix-python-tests-aarch64

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 12, 2026

Third-party PRs run without repository secrets, and the musllinux aarch64 Python job was passing APIFY_HTTPBIN_TOKEN to run-on-arch-action in a way that produced a non-flat env value. That caused the job to fail before tests could use their existing fallback behavior.

  • Root cause

    • uraimo/run-on-arch-action requires env values to be flat strings.
    • When secrets.APIFY_HTTPBIN_TOKEN is unavailable (fork PRs), the workflow passed an invalid value shape for this action input.
  • Workflow change

    • Updated the musllinux aarch64 pytest step in .github/workflows/python-test.yaml to always pass a flat string:
    env: |
      APIFY_HTTPBIN_TOKEN: ${{ secrets.APIFY_HTTPBIN_TOKEN || '' }}
  • Impact

    • Keeps secret-backed behavior unchanged when the token exists.
    • Prevents pre-test failure in external PRs by passing an empty string when the secret is absent.

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:

  • httpbin.org
    • Triggering command: /home/REDACTED/work/impit/impit/impit-python/.venv/bin/pytest pytest test/no_client_test.py -k headers_work -q /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/o-lm /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/o-ldl rc/lib.rs /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/o-L /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/out/0�� /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/out/020bc241f4dae036-edwards25519_scalarmuldouble_alt.o /home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/out/8f1694c3e3041570-bignum_montinv_p256.o py-l�� f/aws-lc-sys-0.3-I m/s2n-bignum-imp/home/REDACTED/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/aws-lc-sys-0.3-Wl,--version-script=/home/REDACTED/work/impit/impit/target/release/deps/rustcvf8ftg/list f/aws-lc-sys-0.3-I ry_cached_asm_k3ar f/aws-lc-sys-0.3cq orted/include f/aws-lc-sys-0.3/home/REDACTED/work/impit/impit/target/release/build/aws-lc-sys-e413e39b71482964/o-Wl,-Bstatic (dns block)

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

Copilot AI changed the title [WIP] Fix failing Python tests on aarch64 musllinux CI fix(ci): handle missing APIFY_HTTPBIN_TOKEN in musllinux aarch64 Python tests May 12, 2026
Copilot AI requested a review from barjin May 12, 2026 07:12
Copilot finished work on behalf of barjin May 12, 2026 07:12
@barjin barjin marked this pull request as ready for review May 12, 2026 07:18
@github-actions github-actions Bot added this to the 140th sprint - Tooling team milestone May 12, 2026
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label May 12, 2026
@barjin barjin added the adhoc Ad-hoc unplanned task added during the sprint. label May 12, 2026
@barjin
Copy link
Copy Markdown
Member

barjin commented May 12, 2026

@copilot regenerate the pnpm lockfile to make the Node.JS docs CI job pass.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 12, 2026

@copilot regenerate the pnpm lockfile to make the Node.JS docs CI job pass.

Done in commit 5798eb5 — regenerated impit-node/pnpm-lock.yaml and verified with pnpm install --frozen-lockfile in impit-node.

Copilot finished work on behalf of barjin May 12, 2026 07:26
Copilot AI requested a review from barjin May 12, 2026 07:26
@barjin barjin merged commit f819e64 into master May 12, 2026
34 checks passed
@barjin barjin deleted the copilot/fix-python-tests-aarch64 branch May 12, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: Python tests on aarch64 musllinux are failing in third-party PRs

3 participants