fix: correct boolean default value handling in templates#3076
Conversation
Signed-off-by: redscholar <blacktiledhouse@gmail.com>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: redscholar The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
There was a problem hiding this comment.
Code Review
This pull request refactors the handling of boolean flags in templates, specifically for TLS verification and storage class enablement, by moving away from the default filter in favor of explicit key checking or direct property access. It also introduces default storage class configurations for local and NFS providers and removes redundant conditional checks in task execution. I have no feedback to provide.



What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for reviewers:
Does this PR introduced a user-facing change?
Additional documentation, usage docs, etc.: