Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

update #1

Merged
merged 12 commits into from
Jul 15, 2024
Merged

update #1

merged 12 commits into from
Jul 15, 2024

Conversation

eunames
Copy link
Owner

@eunames eunames commented Jul 15, 2024

Summary

  • Short summary of what's included in the PR
  • Give special note to breaking changes: List the exact changes or provide links to documentation.

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.

afrimberger and others added 12 commits June 1, 2024 19:57
@afrimberger
feat(chart:k8up) add security context for cleanup hook
8b39c30 
Signed-off-by: André Frimberger <[email protected]>
@afrimberger
fix(chart:k8up) labels for Dashboard
2a23a99 
Signed-off-by: André Frimberger <[email protected]>
@afrimberger
doc(chart:k8up) update README.md
77180bf 
Signed-off-by: André Frimberger <[email protected]>
@johbo
Ignore terminated Pods in mapping of PVCs to Pods
34c9104 
Signed-off-by: Johannes Bornhold <[email protected]>
@johbo
Add integration test to verify that succeeded Pods are ignored
a7fb8f5 
Signed-off-by: Johannes Bornhold <[email protected]>
@johbo
Refactor logic to exclude all Pod which are not in the phase "Running"
7bae817 
Signed-off-by: Johannes Bornhold <[email protected]>
@Kidswiss
Merge pull request #979 from johbo/terminated-pods
1da3f1f 
Ignore terminated Pods in mapping of PVCs to Pods
@johbo
Make log messages more explicit
7be70a0 
Signed-off-by: Johannes Bornhold <[email protected]>
@johbo
Add integration test for RWO PVC without NodeAffinity
18e11b8 
Signed-off-by: Johannes Bornhold <[email protected]>
@johbo
Also add RWO PVC without NodeAffinity from its PV
fa178ec 
Signed-off-by: Johannes Bornhold <[email protected]>
@Kidswiss
Merge pull request #980 from johbo/unmounted-rwo-volume
debabf3 
Unmounted RWO PVCs without NodeAffinity on the PV should be backed up as well
@Kidswiss
Merge pull request #966 from afrimberger/feature/security-context-cle…
365b38b 
…anup-job

feat: add security context for cleanup hook
@eunames eunames merged commit b8be396 into eunames:master Jul 15, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@eunames @afrimberger @johbo @Kidswiss