BUG: Set rest server credentials in repository URL

JcGKitten commented 6 months ago

Summary

Adds the rest server credentials to the repository env variable by using the previously set credential env vars. Therefore it was needed to ensure that the "RESTIC_REPOSITORY" var is set after all the others, cause a map isn't ordered. This fixes #397.

Checklist

For Code changes

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

Kidswiss commented 6 months ago

HI @JcGKitten

Thanks for the PR! The code looks good to me.

Can you do a make fmt and then commit the changes? There's a trailing space in one of your comments that our linting caught.

JcGKitten commented 6 months ago

Hey @Kidswiss, thx for the review, sorry that I missed to run make fmt, pushed the formatted code.

Kidswiss commented 6 months ago

Thanks for your contribution! 🎉

k8up-io / k8up