Skip to content

feat: add cronjob to check if a pvc is mounted. #1476

Closed
Alexj9837 wants to merge 35 commits intoDiamondLightSource:mainfrom
Alexj9837:last-used-stamper
Closed

feat: add cronjob to check if a pvc is mounted. #1476
Alexj9837 wants to merge 35 commits intoDiamondLightSource:mainfrom
Alexj9837:last-used-stamper

Conversation

@Alexj9837
Copy link
Copy Markdown

Sets up a cronjob that calls a bashscript in configmap.
The purpose of this script is to automatically annotate PVC with a last-used date.
This is to enable the future of automatically deleting unused PVCs.

Alexj9837 and others added 19 commits April 1, 2026 13:29
* extra comment

* adding the bash script to config map, and pointing to it via the cronjob

* updating release tag

* was calling bin/bash not bin/sh and bitnami/kubectl doesn't have full bash

* updating version
* extra comment

* adding the bash script to config map, and pointing to it via the cronjob

* updating release tag

* was calling bin/bash not bin/sh and bitnami/kubectl doesn't have full bash

* updating version

* changing default mode as hitting perm errors

* update chart version
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.21%. Comparing base (18e04ca) to head (d9c344e).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1476   +/-   ##
=======================================
  Coverage   95.20%   95.21%           
=======================================
  Files          43       43           
  Lines        3128     3132    +4     
=======================================
+ Hits         2978     2982    +4     
  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.

@Alexj9837 Alexj9837 force-pushed the last-used-stamper branch from 6ca30fb to bdf8952 Compare April 8, 2026 15:18
Alexj9837 and others added 6 commits April 9, 2026 09:14
* extra comment

* adding the bash script to config map, and pointing to it via the cronjob

* updating release tag

* was calling bin/bash not bin/sh and bitnami/kubectl doesn't have full bash

* updating version
* extra comment

* adding the bash script to config map, and pointing to it via the cronjob

* updating release tag

* was calling bin/bash not bin/sh and bitnami/kubectl doesn't have full bash

* updating version

* changing default mode as hitting perm errors

* update chart version
@Alexj9837 Alexj9837 force-pushed the last-used-stamper branch from c37270b to 1c22291 Compare April 9, 2026 08:14
@Alexj9837 Alexj9837 closed this Apr 9, 2026
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