Skip to content

feat: Add cronjob for time-stamper in ConfigMap #1481

Open
Alexj9837 wants to merge 6 commits intomainfrom
time-stamper-cron
Open

feat: Add cronjob for time-stamper in ConfigMap #1481
Alexj9837 wants to merge 6 commits intomainfrom
time-stamper-cron

Conversation

@Alexj9837
Copy link
Copy Markdown

@Alexj9837 Alexj9837 commented Apr 9, 2026

Context
DAQ core plans to implement auto deletion of unused PVC, for this we need a way of telling how long something hasn't been mounted.

Description
By making use of a Cronjob this runs a command every 5 minutes to run a bash script (time-stamper.sh). the bash script loops over the PVC and annotates them, it does this by gathering all PVC's in the blueapi,then all the mounted ones, loops through these annotating the ones that either are mounted or don't have a "last-used" annotation. This is linked via the config map.

@Alexj9837 Alexj9837 requested a review from a team as a code owner April 9, 2026 14:57
@Alexj9837 Alexj9837 changed the title Add cronjob for time-stamper in ConfigMap feat: Add cronjob for time-stamper in ConfigMap Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.21%. Comparing base (65b7e75) to head (09cc9c2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1481   +/-   ##
=======================================
  Coverage   95.21%   95.21%           
=======================================
  Files          43       43           
  Lines        3132     3132           
=======================================
  Hits         2982     2982           
  Misses        150      150           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant