fix(maintenance): write backup archive to $RUNNER_TEMP to prevent tar self-inclusion error#225
Conversation
…ilure Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/f2fba114-abbe-42c4-8023-e036d31ccb80 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
|
|
There was a problem hiding this comment.
Pull request overview
This PR fixes the weekly backup workflow failure by ensuring the generated tarball is written outside the directory being archived, preventing tar from observing directory changes while it scans the workspace.
Changes:
- Generate the backup archive under
/tmpinstead of the repository root to avoid tar self-inclusion / directory-change warnings. - Update the computed archive size path to match the new archive location.
- Output the full archive path via
archive_nameso the release upload step can consume it directly.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| tar -czf "/tmp/$ARCHIVE_NAME" \ | ||
| --exclude='.git' \ |
| ARCHIVE_SIZE=$(du -sh "/tmp/$ARCHIVE_NAME" | cut -f1) | ||
| echo "archive_name=/tmp/$ARCHIVE_NAME" >> "$GITHUB_OUTPUT" | ||
| echo "archive_size=$ARCHIVE_SIZE" >> "$GITHUB_OUTPUT" |
Pull request Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
✅ Deploy Preview for whimsical-kitsune-85066d ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for spiffy-liger-8bab58 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@dependabot rebase @copilot rebase |
…archive location Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/28d26abc-53c6-46d1-8b04-6b7006533e10 Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
Branch rebased onto the latest |
$RUNNER_TEMP to prevent tar self-inclusion error
DavidKRK
left a comment
There was a problem hiding this comment.
Deux problèmes bloquants avant de pouvoir fusionner cette PR :
CI incomplet : Les checks requis (build, CodeQL, Gitleaks, Lighthouse) n'ont pas pas été tournés sur cette branche — seuls 6 checks Netlify neutres sont présents.
Branche en retard : La base SHA de la PR ( 3241844) est en retard de plusieurs commits par rapport au HEAD de gh-pages— la branche doit être rebasée.
Action requise : commenter @dependabot rebaseou rebaser manuellement pour déclencher les vérifications CI complètes, puis revalider.
…yment-issues # Conflicts: # .github/workflows/maintenance.yml Co-authored-by: DavidKRK <136035295+DavidKRK@users.noreply.github.com>
Branche mise à jour dans 192aafb — merge de |
DavidKRK
left a comment
There was a problem hiding this comment.
Changement validé. L'archive est maintenant créée directement dans $RUNNER_TEMP au lieu de /tmp + mv, ce qui évite l'erreur tar: file changed as we read it. La variable archive_name exporte désormais le chemin complet (ARCHIVE_PATH), ce qui est cohérent avec l'étape de release. Modification propre et correcte. ✅
origin/gh-pagesdans la branche pour l'aligner sur le HEAD actuel (2489df9).github/workflows/maintenance.ymlen conservant la versionARCHIVE_PATH