bindings: add manifest should follow es_model while marshalling OSVersion and OSFeatures

It seems API needs json names for OSVersion and OSFeatures in es_model
ref: https://github.com/containers/podman/blob/main/pkg/domain/entities/manifest.go#L42

So at bindings end ensure that we honor es_model naming convention when
we perform marshalling otherwise API will ignore these fields

Signed-off-by: Aditya R <arajan@redhat.com>

2 files changed, 16 insertions, 14 deletions

diff --git a/pkg/bindings/manifests/types.go b/pkg/bindings/manifests/types.go
index d0b0b2e71..e23ef798d 100644
--- a/pkg/bindings/manifests/types.go
+++ b/pkg/bindings/manifests/types.go
@@ -44,16 +44,18 @@ type RemoveOptions struct {
 type ModifyOptions struct {
 	// Operation values are "update", "remove" and "annotate". This allows the service to
 	//   efficiently perform each update on a manifest list.
-	Operation     *string
-	All           *bool             // All when true, operate on all images in a manifest list that may be included in Images
-	Annotations   map[string]string // Annotations to add to manifest list
-	Arch          *string           // Arch overrides the architecture for the image
-	Features      []string          // Feature list for the image
-	Images        []string          // Images is an optional list of images to add/remove to/from manifest list depending on operation
-	OS            *string           // OS overrides the operating system for the image
-	OSFeatures    []string          // OS features for the image
-	OSVersion     *string           // OSVersion overrides the operating system for the image
-	Variant       *string           // Variant overrides the operating system variant for the image
+	Operation   *string
+	All         *bool             // All when true, operate on all images in a manifest list that may be included in Images
+	Annotations map[string]string // Annotations to add to manifest list
+	Arch        *string           // Arch overrides the architecture for the image
+	Features    []string          // Feature list for the image
+	Images      []string          // Images is an optional list of images to add/remove to/from manifest list depending on operation
+	OS          *string           // OS overrides the operating system for the image
+	// OS features for the image
+	OSFeatures []string `json:"os_features" schema:"os_features"`
+	// OSVersion overrides the operating system for the image
+	OSVersion     *string `json:"os_version" schema:"os_version"`
+	Variant       *string // Variant overrides the operating system variant for the image
 	Authfile      *string
 	Password      *string
 	Username      *string
diff --git a/pkg/bindings/manifests/types_modify_options.go b/pkg/bindings/manifests/types_modify_options.go
index 9d2ed2613..ab00cb2c5 100644
--- a/pkg/bindings/manifests/types_modify_options.go
+++ b/pkg/bindings/manifests/types_modify_options.go
@@ -122,13 +122,13 @@ func (o *ModifyOptions) GetOS() string {
 	return *o.OS
 }
 
-// WithOSFeatures set oS features for the image
+// WithOSFeatures set field OSFeatures to given value
 func (o *ModifyOptions) WithOSFeatures(value []string) *ModifyOptions {
 	o.OSFeatures = value
 	return o
 }
 
-// GetOSFeatures returns value of oS features for the image
+// GetOSFeatures returns value of field OSFeatures
 func (o *ModifyOptions) GetOSFeatures() []string {
 	if o.OSFeatures == nil {
 		var z []string
@@ -137,13 +137,13 @@ func (o *ModifyOptions) GetOSFeatures() []string {
 	return o.OSFeatures
 }
 
-// WithOSVersion set oSVersion overrides the operating system for the image
+// WithOSVersion set field OSVersion to given value
 func (o *ModifyOptions) WithOSVersion(value string) *ModifyOptions {
 	o.OSVersion = &value
 	return o
 }
 
-// GetOSVersion returns value of oSVersion overrides the operating system for the image
+// GetOSVersion returns value of field OSVersion
 func (o *ModifyOptions) GetOSVersion() string {
 	if o.OSVersion == nil {
 		var z string