Unexpected invocation of NewXxxKindREST() in controller during package v1.init() about apiserver-builder-alpha

[pwittrock]

Interesting. Adding some documentation seems like a good workaround for now. We should try to decouple initializing the storage from the type definitions so that every client doesn't exercise this logic.

[pwittrock]

@ibazhitov You interested in filing a PR?

[ibazhitov]

OK, I'll create a PR with changes in the documentation.

[ibazhitov]

We should try to decouple initializing the storage from the type definitions so that every client doesn't exercise this logic.

I'll check whether this is possible.

[ibazhitov]

On the first glance, replacing a direct call to NewXxxREST() in v1/zz_generated.api.register.go with a function pointer would do the trick.
Also, there's a mention of some mysterious RESTFunc in the definition of versionedResourceBuilder:

	// Store is used to modify the default storage, mutually exclusive with RESTFunc
	StorageBuilder StorageBuilder

	// REST a rest.Store implementation, mutually exclusive with StoreFunc
	REST rest.Storage

I haven't found other mentions of RESTFunc in git history though. Maybe there was some plan to actually replace rest.Storage with RESTFunc in versionedResourceBuilder?

[ibazhitov]

Tested this approach with #97, looks like it works.

[ibazhitov]

Ugh, I should have closed #96 before submitting #97, sorry.
Now we have both bug fix and the workaround in the documentation merged.
@pwittrock, could you please revert #96 since we no longer need a workaround?