Open sagikazarmark opened 3 years ago
Describe the bug Updating a resource always ends up in an error: Failed to unmarshal manifest: EOF
Failed to unmarshal manifest: EOF
Steps to reproduce the issue:
k8s_manifest
Expected behavior Update should complete without errors
Additional context
I believe #79 introduced two serious breaking changes that led to this bug:
override_content
ignore_fields
content
Looking at the PR override_content also seems to be redundant and unnecessary since ignored fields can be computed without storing it.
Describe the bug Updating a resource always ends up in an error:
Failed to unmarshal manifest: EOF
Steps to reproduce the issue:
k8s_manifest
resourceExpected behavior Update should complete without errors
Additional context
I believe #79 introduced two serious breaking changes that led to this bug:
override_content
argument was introduced to track the original content, but it's only set whenignore_fields
is also providedcontent
values are not populated properly, unlessignore_fields
is also providedLooking at the PR
override_content
also seems to be redundant and unnecessary since ignored fields can be computed without storing it.