Opened 3 years ago

Closed 2 years ago

#30154 closed defect (fixed)

GH Actions: cygwin: Use incremental archives for the sage-local artifact

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.3
Component: porting: Cygwin Keywords:
Cc: dimpase, embray, slelievre, gh-kliem Merged in:
Authors: Matthias Koeppe Reviewers: Jonathan Kliem
Report Upstream: N/A Work issues:
Branch: 5ccf1ea (Commits, GitHub, GitLab) Commit: 5ccf1eabee4f23e43a12da9cddad7e6474694104
Dependencies: Stopgaps:

Status badges

Description (last modified by mkoeppe)

Only store modified/added files that were not in the previous stages.

https://www.gnu.org/software/tar/manual/html_node/Incremental-Dumps.html#incremental_002dop

Change History (10)

comment:1 Changed 3 years ago by mkoeppe

Description: modified (diff)

comment:2 Changed 2 years ago by mkoeppe

Milestone: sage-9.2sage-9.3

comment:3 Changed 2 years ago by mkoeppe

Description: modified (diff)
Summary: cygwin: CI fixesGH Actions: cygwin: Use incremental archives for the sage-local artifact

comment:4 Changed 2 years ago by mkoeppe

Reviewers: https://github.com/mkoeppe/sage/actions/runs/405202734

comment:5 Changed 2 years ago by mkoeppe

Reviewers: https://github.com/mkoeppe/sage/actions/runs/405202734https://github.com/mkoeppe/sage/actions/runs/407181786

comment:6 Changed 2 years ago by mkoeppe

Branch: u/mkoeppe/gh_actions__cygwin__use_incremental_archives_for_the_sage_local_artifact

comment:7 Changed 2 years ago by mkoeppe

Authors: Matthias Koeppe
Cc: gh-kliem added
Commit: 5ccf1eabee4f23e43a12da9cddad7e6474694104
Reviewers: https://github.com/mkoeppe/sage/actions/runs/407181786https://github.com/mkoeppe/sage/actions/runs/407181786, ...
Status: newneeds_review

This seems to be much more robust - stage iii used to fail to upload its huge artifact, now it goes through


New commits:

7244a9dci-cygwin-standard.yml: Use tar --listed-incremental
5ccf1eatar --remove-files and --listed-incremental are not compatible

comment:8 Changed 2 years ago by gh-kliem

Reviewers: https://github.com/mkoeppe/sage/actions/runs/407181786, ...Jonathan Kliem
Status: needs_reviewpositive_review

LGTM.

comment:9 Changed 2 years ago by mkoeppe

Thank you!

comment:10 Changed 2 years ago by vbraun

Branch: u/mkoeppe/gh_actions__cygwin__use_incremental_archives_for_the_sage_local_artifact5ccf1eabee4f23e43a12da9cddad7e6474694104
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.