Skip to content

Conversation

@ahyattdev
Copy link
Contributor

Summary

Preserve volumeMounts from PodConfig.

This currently handled for Check and Prune, but not Backup, Archive, or Restore.

Issue mentioned here in a comment: #584 (comment)

One use case: if securityContext calls for a read only root FS, you need an emptyDir at /.cache for the restic cache to be written to.

Checklist

For Code changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:operator
  • Commits are signed off
  • Link this PR to related issues
  • I have not made any changes in the charts/ directory.

For Helm Chart changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:chart
  • PR contains the chart label, e.g. chart:k8up
  • Commits are signed off
  • Variables are documented in the values.yaml using the format required by Helm-Docs.
  • Chart Version bumped if immediate release after merging is planned
  • I have run make chart-docs
  • Link this PR to related code release or other issues.

@ahyattdev ahyattdev requested a review from a team as a code owner June 25, 2025 03:48
@ahyattdev ahyattdev requested review from TheBigLee and zugao and removed request for a team June 25, 2025 03:48
@ahyattdev ahyattdev force-pushed the podconfig-volumes branch from d3a5db7 to 97b7ead Compare June 26, 2025 02:46
@ahyattdev
Copy link
Contributor Author

Just saw #1056, which adds an emptyDir for /.cache by default now. That was my use case. So closing this PR.

@ahyattdev ahyattdev closed this Jul 5, 2025
@beniwtv
Copy link

beniwtv commented Jul 7, 2025

I would still want this, for other use-cases. We use a CSI volume for secrets (for example, for S3), so being able to specify additional volumes to be included would still be beneficial.

@ahyattdev ahyattdev reopened this Jul 7, 2025
@TheBigLee TheBigLee requested a review from Kidswiss July 21, 2025 13:48
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.

2 participants