Discuss: What's default view for cmd/main.go?

[fanminshi]

Before: the cmd/main.go is generated based on user inputs Kind and APIVersion such as PlayService kind. The reason behind this is that users can just watch their customized resource out of the box as the default behavior.

func main() {
	namespace := "default"
        // old Watch API.
	sdk.Watch(api.PlayServicePlural, namespace, api.PlayService)
 	sdk.Handle(&stub.Handler{})
 	sdk.Run(context.TODO())
}

Now: Should we change the above default behavior to watch on Deployment instead?

func main() {
	namespace := "default"
	sdk.Watch("apps/v1", "Deployment", namespace)
 	sdk.Handle(&stub.Handler{})
 	sdk.Run(context.TODO())
}

[fanminshi]

cc/ @hongchaodeng @hasbro17

[fanminshi]

@hongchaodeng just saw the pr before create this issue.

[hongchaodeng]

Currently we have a story in "Up and running" to let user do nothing (e.g. creating a CR) and still have visual result, test the environment works. I would say the Deployment story (option 2) is a better starting point.

watch their customized resource out of the box

We would enforce user to do it themselves in "Customizing operator logic" section if they are interested in developing an operator. That includes changing the spec/status, watching custom resources, handling the CR object. This would make user understand the workflow of their custom resource better.

[fanminshi]

My thought on project operator-sdk is to show user how to build an operator out of box which involves watching on a customized resource. Hence, I think the default behavior(option 1) achieves that purpose. If the default behavior is to demonstrate how watch.SDK() works, then the default behavior (option 2) makes sense.

I am fine with both.

[fanminshi]

fixed via #42