Skip to content

Conversation

Copy link

Copilot AI commented Aug 23, 2025

The GitHub Actions workflow had a circular dependency that prevented manual container builds from working correctly. The generate_matrix job was trying to use its own output needs.generate_matrix.outputs.file_host instead of the workflow input parameter.

This caused workflow failures when trying to rebuild specific containers like ramips-mt7621-v23.05.6, as reported in the issue where the container was missing from the registry despite the underlying ImageBuilder being available.

The fix:

Changed line 147 in .github/workflows/containers.yml from:

FILE_HOST=${{ needs.generate_matrix.outputs.file_host }}

to:

FILE_HOST=${{ github.event.inputs.file_host }}

How to rebuild missing containers:

Maintainers can now successfully rebuild missing containers by:

  1. Going to the Actions tab
  2. Clicking "Run workflow"
  3. Setting the appropriate parameters:
    • ref: The version (e.g., v23.05.6)
    • target: The target architecture (e.g., ramips/mt7621)
  4. Clicking "Run workflow"

This will build and push the container to the registry (e.g., ghcr.io/openwrt/imagebuilder:ramips-mt7621-v23.05.6).

Also included documentation (REBUILD_INSTRUCTIONS.md) explaining the rebuild process for future reference.

Fixes #182.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Aug 23, 2025
Copy link
Author

Copilot AI commented Aug 23, 2025

@aparcar 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Copilot AI and others added 2 commits August 23, 2025 15:44
Co-authored-by: aparcar <16000931+aparcar@users.noreply.github.com>
Co-authored-by: aparcar <16000931+aparcar@users.noreply.github.com>
Copilot AI changed the title [WIP] 23.05.6: missing container for ramips/mt7621 Fix circular dependency in workflow preventing container builds Aug 23, 2025
Copilot AI requested a review from aparcar August 23, 2025 15:45
@aparcar
Copy link
Member

aparcar commented Aug 23, 2025

🤭

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.

23.05.6: missing container for ramips/mt7621

2 participants