Skip to content
New issue

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

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

Already on GitHub? # to your account

janitor: don't delete old images #589

Merged
merged 1 commit into from
Sep 30, 2023
Merged

Conversation

aparcar
Copy link
Member

@aparcar aparcar commented Sep 30, 2023

This is overly complicated. Just use a cron script:

cd /mnt/store/ && find . -type d -mtime +10 -exec rm -rf {} ;

This is overly complicated. Just use a cron script:

cd /mnt/store/ && find . -type d -mtime +10 -exec rm -rf {} \;

Signed-off-by: Paul Spooren <mail@aparcar.org>
@codecov
Copy link

codecov bot commented Sep 30, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (9fba880) 86.56% compared to head (e91891a) 87.41%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #589      +/-   ##
==========================================
+ Coverage   86.56%   87.41%   +0.85%     
==========================================
  Files          13       13              
  Lines         945      930      -15     
==========================================
- Hits          818      813       -5     
+ Misses        127      117      -10     
Files Coverage Δ
asu/build.py 80.50% <ø> (-0.17%) ⬇️
asu/janitor.py 76.92% <ø> (+6.44%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aparcar aparcar merged commit a3d8ac9 into openwrt:main Sep 30, 2023
@aparcar aparcar deleted the no-cleanup branch September 30, 2023 09:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant