fix: handle null values for TES volumes

[lvarin]

If spec.template.spec.volumes is null, use [] instead of None.

This is just to make taskmaster more robust

Summary by Sourcery

Fix the handling of null values in spec.template.spec.volumes by defaulting to an empty list to enhance robustness.

Bug Fixes:

• Ensure that if spec.template.spec.volumes is null, it defaults to an empty list instead of None to prevent potential errors.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request addresses a robustness issue in the taskmaster component. It modifies the run_executor function in src/tesk_core/taskmaster.py to handle the case where spec.template.spec.volumes is None by initializing it as an empty list before extending it with volume information.

File-Level Changes

Files	Changes
src/tesk_core/taskmaster.py	Added a check to initialize 'volumes' as an empty list if it is None

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @lvarin - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider adding a brief comment explaining why this None check is necessary. It could help future maintainers understand the reasoning behind this addition.
• It might be worth investigating if the case where 'volumes' is None can be prevented earlier in the code flow. If not, this fix is a good defensive measure.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}