Wrangler CRD schema generation incorrectly identifies MicroTime as object in resulting openapi spec

[ebauman]

Scenario:

You have a new type, say Foo. Defined as such:

type Foo struct {
... // objectMeta, typeMeta
EventTime *metav1.MicroTime `json:"eventTime"`
}

Now, you run this type through Wrangler's provided ToOpenAPIFromStruct as seen in

wrangler/pkg/schemas/openapi/generate.go

Line 25 in 0bfb013

func ToOpenAPIFromStruct(obj interface{}) (*v1.JSONSchemaProps, error) {

Wrangler incorrectly spits out an OpenAPI schema in which the MicroTime is stored as an object. It should be stored as a string, similar to how metav1.Time is stored.

The problem is in reflection.go,

wrangler/pkg/schemas/reflection.go

Lines 495 to 498 in 0bfb013

	case reflect.Struct:
	if t.Name() == "Time" {
	return "date", nil
	}

Time is correctly compensated for, but not MicroTime.

[ebauman]

It is a valid workaround to define a wrangler:"type=string" tag on the field, but let's fix it at the source, yeah? :)

[ericpromislow]