Skip to content

fix: use the values managed file only if workload imageUpdateStrategy is set#2806

Merged
CasLubbers merged 7 commits intomainfrom
APL-1385
Dec 17, 2025
Merged

fix: use the values managed file only if workload imageUpdateStrategy is set#2806
CasLubbers merged 7 commits intomainfrom
APL-1385

Conversation

@j-zimnowoda
Copy link
Contributor

@j-zimnowoda j-zimnowoda commented Dec 17, 2025

📌 Summary

The values managed file for workload is only required when workload imageStrategy is set.

🔍 Reviewer Notes

n/a

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

@j-zimnowoda j-zimnowoda changed the title fix: use the managed file only if image update strategy is applied fix: use the values managed file only if workload imageUpdateStrategy is set Dec 17, 2025
@CasLubbers CasLubbers merged commit 39d64f5 into main Dec 17, 2025
11 checks passed
@CasLubbers CasLubbers deleted the APL-1385 branch December 17, 2025 15:35
ElderMatt pushed a commit that referenced this pull request Dec 19, 2025
@j-zimnowoda @svcAPLBot
fix: use the values managed file only if workload imageUpdateStrategy…
8b6fff0 
… is set (#2806)

Co-authored-by: svcAPLBot <174728082+svcAPLBot@users.noreply.github.com>
(cherry picked from commit 39d64f5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ElderMatt ElderMatt ElderMatt approved these changes

@CasLubbers CasLubbers CasLubbers approved these changes

@merll merll Awaiting requested review from merll merll is a code owner

@Ani1357 Ani1357 Awaiting requested review from Ani1357 Ani1357 is a code owner

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

@j-zimnowoda @ElderMatt @CasLubbers @svcAPLBot