Add cache mirror support to redirector update workflow

[igorpecovnik]

Extend the redirector configuration workflow to support cache mirrors:

• Add cache to build platform matrix
• Add Cache-index job to fetch mirrors from NetBox
• Add Cache check job to compare mirrors against source
• Add cache redirector config generation (port 8084)
• Handle cache artifacts mirroring from /cache/artifacts

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The workflows were simplified: detailed cache mirroring, artifact download/compare, per-node mirror logic, SSH install/reload, and many cache-specific status checks were removed or replaced with no-op/touch/status=true steps in infrastructure-update-redirector-config.yml. Multiple lftp timeouts were reduced (5m → 3m). Separately, generate-web-directory.yml gained steps to install an SSH key/known_hosts and perform an SSH reload of the redirector.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Reload redirector after we check all mirrors #178: Modifies the same redirector workflow and SSH-based reload logic that this change removes/affects.
• Prepare code for enabling Armbian controlled mirror when APT repo changed #123: Adds or alters SSH key installation and redirector reload steps in workflow files touched here.
• Enable refactored repository update #117: Changes repository-update/redirector workflow behavior overlapping with cache and redirector steps adjusted in this PR.

Suggested reviewers

• armbianworker

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly and concisely describes the main change: adding cache mirror support to the redirector update workflow, which aligns perfectly with the changeset.
Description check	✅ Passed	The description is well-related to the changeset, providing specific details about cache support including build matrix changes, new jobs, config generation, and artifact handling.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fixing-redi

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.github/workflows/infrastructure-update-redirector-config.yml:
- Around line 454-478: The script uses the uninitialized variable exit_status in
the conditionals (the checks [[ "${exit_status}" -eq 0 ]] and [[
"${exit_status}" -eq 124 ]]) which will error if the curl block is skipped; fix
this by initializing exit_status=0 (or another safe default) before the curl/if
block and ensure any branch that can set it (the lftp mirror command) assigns to
exit_status (e.g., capture its exit code into exit_status) so the subsequent
checks in this step (and the same pattern in the other jobs like Debs-beta,
Debs-stable, Images-stable, Images-archive) always operate on a defined integer
value.