[enhancement] add priorityClassName for pre backup pod

amghazanfari commented 5 months ago

Summary

set priorityClassName for pre backup pod
i'm not very sure that it's a good idea to set this thing in pre backup pod, this is an enhancement for #960

[x] 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
[x] Link this PR to related issues
[x] I have not made any changes in the charts/ directory.

[ ] 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
[ ] 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.

Kidswiss commented 5 months ago

Hi @amghazanfari

Thanks for the contribution.

However, this seems redundant to me, it's already possible to pass a complete podSpec to set any custom settings you want.

Do you get any benefits from settings this via a separate field instead of adjusting the podspec directly?

amghazanfari commented 5 months ago

you're right i close the pr