Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rootfs: don't use the Armbian repo when building rootfs from both GHA pipelines and CLI; fixes #7123 #7313

Conversation

rpardini
Copy link
Member

rootfs: don't use the Armbian repo when building rootfs from both GHA pipelines and CLI; fixes #7123

  • rootfs: don't use the Armbian repo when building rootfs from both GHA pipelines and CLI; fixes Framework: disable armbian repository while generating rootfs cache #7123
    • Fixes: 7129643 -- which only set SKIP_ARMBIAN_REPO=yes when rootfs cli shortcut used
      • GHA artifact pipelines are generic across all artifacts, and don't use the cli shortcuts
        • instead, they use the artifact shortcut, with the WHAT=rootfs parameter
        • so in GHA pipelines SKIP_ARMBIAN_REPO=yes was not taking effect
    • rootfs artifact itself now enforces SKIP_ARMBIAN_REPO to be yes and makes it readonly
    • also add SKIP_ARMBIAN_REPO to the general change-tracking for more clarity about configs and hooks etc
    • add change tracking calls before/after generic artifact config adapter for even more clarity

… pipelines and CLI; fixes armbian#7123

- Fixes: 7129643 -- which only set `SKIP_ARMBIAN_REPO=yes` when `rootfs` cli shortcut used
  - GHA artifact pipelines are generic across all artifacts, and don't use the cli shortcuts
    - instead, they use the `artifact` shortcut, with the `WHAT=rootfs` parameter
    - so in GHA pipelines `SKIP_ARMBIAN_REPO=yes` was not taking effect
- rootfs artifact itself now enforces `SKIP_ARMBIAN_REPO` to be `yes` and makes it readonly
- also add `SKIP_ARMBIAN_REPO` to the general change-tracking for more clarity about configs and hooks etc
- add change tracking calls before/after generic artifact config adapter for even more clarity
@github-actions github-actions bot added Framework Framework components size/small PR with less then 50 lines labels Sep 30, 2024
@rpardini rpardini marked this pull request as ready for review September 30, 2024 12:14
@rpardini rpardini requested a review from a team as a code owner September 30, 2024 12:14
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 11 Milestone: Fourth quarter release labels Sep 30, 2024
@igorpecovnik igorpecovnik merged commit 55b8718 into armbian:main Sep 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release Framework Framework components Ready to merge Reviewed, tested and ready for merge size/small PR with less then 50 lines
Development

Successfully merging this pull request may close these issues.

2 participants