Skip to content

fix(maintenance): write backup archive to $RUNNER_TEMP to prevent tar self-inclusion error#225

Merged
DavidKRK merged 4 commits into
gh-pagesfrom
copilot/fix-deployment-issues
May 19, 2026
Merged

fix(maintenance): write backup archive to $RUNNER_TEMP to prevent tar self-inclusion error#225
DavidKRK merged 4 commits into
gh-pagesfrom
copilot/fix-deployment-issues

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

  • Merger origin/gh-pages dans la branche pour l'aligner sur le HEAD actuel (2489df9)
  • Résoudre le conflit sur .github/workflows/maintenance.yml en conservant la version ARCHIVE_PATH
  • Pousser le merge commit pour déclencher les checks CI (build, CodeQL, Gitleaks, Lighthouse)

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@DavidKRK DavidKRK requested a review from Copilot May 1, 2026 23:36
@DavidKRK DavidKRK marked this pull request as ready for review May 1, 2026 23:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 /tmp instead 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_name so the release upload step can consume it directly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/maintenance.yml Outdated
Comment thread .github/workflows/maintenance.yml Outdated
Comment on lines 131 to 132
tar -czf "/tmp/$ARCHIVE_NAME" \
--exclude='.git' \
Comment thread .github/workflows/maintenance.yml Outdated
Comment on lines 138 to 140
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>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for whimsical-kitsune-85066d ready!

Name Link
🔨 Latest commit ba6bfb0
🔍 Latest deploy log https://app.netlify.com/projects/whimsical-kitsune-85066d/deploys/6a00e559734f350008ee89f1
😎 Deploy Preview https://deploy-preview-225--whimsical-kitsune-85066d.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for spiffy-liger-8bab58 ready!

Name Link
🔨 Latest commit ba6bfb0
🔍 Latest deploy log https://app.netlify.com/projects/spiffy-liger-8bab58/deploys/6a00e55905ca3d0008845093
😎 Deploy Preview https://deploy-preview-225--spiffy-liger-8bab58.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@DavidKRK
Copy link
Copy Markdown
Owner

@dependabot rebase @copilot rebase

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 19, 2026

@dependabot rebase @copilot rebase

Branch rebased onto the latest gh-pages and the reviewer suggestions addressed in faf7a13ARCHIVE_PATH="${RUNNER_TEMP}/${ARCHIVE_NAME}" is now defined once and reused for tar, du, and the archive_name output.

Copilot AI changed the title fix(maintenance): write backup archive to /tmp to prevent tar self-inclusion error fix(maintenance): write backup archive to $RUNNER_TEMP to prevent tar self-inclusion error May 19, 2026
Copilot AI requested a review from DavidKRK May 19, 2026 04:03
Copilot stopped work on behalf of DavidKRK due to an error May 19, 2026 04:08
Copy link
Copy Markdown
Owner

@DavidKRK DavidKRK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ BLOQUEUR — Rebase requis + CI manquants

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>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 19, 2026

⚠️ BLOQUEUR — Rebase requis + CI manquants

Deux problèmes bloquants avant de pouvoir fusionner cette PR :...

Branche mise à jour dans 192aafb — merge de origin/gh-pages (HEAD 2489df9) dans la branche pour l'aligner et déclencher les checks CI complets (build, CodeQL, Gitleaks, Lighthouse). La version ARCHIVE_PATH="${RUNNER_TEMP}/${ARCHIVE_NAME}" est conservée dans la résolution du conflit.

Copilot AI requested a review from DavidKRK May 19, 2026 04:21
Copy link
Copy Markdown
Owner

@DavidKRK DavidKRK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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. ✅

@DavidKRK DavidKRK merged commit a4f00f8 into gh-pages May 19, 2026
11 of 12 checks passed
@DavidKRK DavidKRK deleted the copilot/fix-deployment-issues branch May 19, 2026 04:45
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.

5 participants