Skip to content

Process_CI: fix broken release-target links (wrong branch + workflow name) - #952

Merged
igorpecovnik merged 2 commits into
mainfrom
fix/process-ci-release-target-links
Aug 5, 2026
Merged

Process_CI: fix broken release-target links (wrong branch + workflow name)#952
igorpecovnik merged 2 commits into
mainfrom
fix/process-ci-release-target-links

Conversation

@igorpecovnik

@igorpecovnik igorpecovnik commented Aug 5, 2026

Copy link
Copy Markdown
Member

What

Several links in Process_CI › Prepare Standard Support images for release (and the apps section) 404.

Why

The generated release-target YAMLs live on the data branch at data/release-targets/, not on main/release-targets/ (which only contains the .blacklist / .manual / .map source files). Verified via the GitHub API:

  • armbian.github.io@main:release-targets/ → has *.blacklist, *.manual, targets-extensions.mapno *.yaml
  • armbian.github.io@data:data/release-targets/ → has targets-release-standard-support.yaml, targets-release-apps.yaml, targets-release-nightly.yaml, …

The nightly link in the same doc was already correct (blob/data/data/release-targets/…); this just makes standard-support and apps consistent.

Fixes

Was (404) Now
blob/main/release-targets/targets-release-standard-support.yaml (×2) blob/data/data/release-targets/targets-release-standard-support.yaml
blob/main/release-targets/targets-release-apps.yaml (×2) blob/data/data/release-targets/targets-release-apps.yaml
[generate-targets workflow](…/generate-targets.yaml) [generate-build-lists workflow](…/generate-build-lists.yaml)

generate-targets.yaml doesn't exist on armbian.github.io; generate-build-lists.yaml is the workflow that actually runs generate_targets.py.

Create docs preview on PR

Documentation website preview will be available shortly:


 Open WWW preview 

…ame)

The generated release-target YAMLs live on the `data` branch at
`data/release-targets/`, not on `main/release-targets/` (which only holds the
`.blacklist`/`.manual`/`.map` sources), so these links 404:

- targets-release-standard-support.yaml (×2)
- targets-release-apps.yaml (×2)

Point them at `blob/data/data/release-targets/...`, matching the already-correct
nightly link. Also fix the "generate-targets workflow" link: there is no
`generate-targets.yaml` on armbian.github.io; the workflow that runs
`generate_targets.py` is `generate-build-lists.yaml`.

Signed-off-by: Igor Pecovnik <igor@armbian.com>
@github-actions github-actions Bot added size/small PR with less then 50 lines Needs review Seeking for review labels Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: fdc78998-173e-496c-bd34-790b0bb36642

📥 Commits

Reviewing files that changed from the base of the PR and between b4faaa4 and c2bb98e.

📒 Files selected for processing (1)
  • docs/Process_CI.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/Process_CI.md

Walkthrough

Updated five CI documentation links in docs/Process_CI.md. The kernel, standard-support, standard image, and application build-list links now use the data branch. The application documentation now references generate-build-lists instead of generate-targets.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the Process_CI documentation fix for broken release-target links and the incorrect branch and workflow name.
Description check ✅ Passed The description directly explains the broken links, their causes, and the documentation changes made to fix them.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/process-ci-release-target-links

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

Comment @coderabbitai help to get the list of available commands.

github-actions Bot pushed a commit that referenced this pull request Aug 5, 2026
…efix)

github.armbian.com serves the data branch at root, so data/release-targets/X
is published as github.armbian.com/release-targets/X. The link used
github.armbian.com/data/release-targets/kernel-description.json which 404s;
drop the data/ prefix.

Signed-off-by: Igor Pecovnik <igor@armbian.com>
github-actions Bot pushed a commit that referenced this pull request Aug 5, 2026
@igorpecovnik
igorpecovnik merged commit 756f6db into main Aug 5, 2026
5 checks passed
@igorpecovnik
igorpecovnik deleted the fix/process-ci-release-target-links branch August 5, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs review Seeking for review size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

1 participant