Skip to content
This repository has been archived by the owner on Sep 26, 2022. It is now read-only.

fix: Fixes #256, added ability to associate all fields after processing #263

Merged
merged 14 commits into from
Feb 11, 2022
Merged

fix: Fixes #256, added ability to associate all fields after processing #263

merged 14 commits into from
Feb 11, 2022

Conversation

scottd018
Copy link
Contributor

No description provided.

Dustin Scott added 14 commits February 8, 2022 08:38
Signed-off-by: Dustin Scott <sdustin@vmware.com>
This commit addresses a panic situation when the workload itself references a
collection field, however the workload itself is not using a collection field
as an input to any of its fields.

Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
This simply is a reminder to prepare for a test case for #271.  It does not
include an empty line, as an empty line would break the test case.  Once
in order to produce our functional test to account for this.

Signed-off-by: Dustin Scott <sdustin@vmware.com>
This adds the 'omitempty' tag to all fields within the API.  This allows the
unmarshal to unmarshal with empty fields.  Without the 'omitempty' tag, the
unmarshal event results in zero values before it can be processed by the CRD
which is there to set the default values.

Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
Signed-off-by: Dustin Scott <sdustin@vmware.com>
@lander2k2 lander2k2 merged commit 03b331a into vmware-archive:main Feb 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants