feat: Better output of build/deploy/delete commands

[zroubalik]

Signed-off-by: Zbynek Roubalik zroubali@redhat.com

Discussed this with @lkingland, Implementing the spinner properly (so it is not eating the output) may take some time. So I'll proceed with printing some basic info messages, since we are under such a tight deadline.

And we will revisit the Spinner properly for all the commands as a post Dev Preview task.

Build

./faas build -p func                                        
Building function image
Function image has been built, image: docker.io/zroubalik/func:latest

Deploy

 ./faas deploy -p func                                           
Building function image
Function image has been built, image: docker.io/zroubalik/func:latest
Pushing function image to the registry
Deploying function to the cluster
Function deployed at URL: http://func-default.apps.zroubali.devcluster.openshift.com

Update

./faas deploy -p func                                 
Building function image
Function image has been built, image: docker.io/zroubalik/func:latest
Pushing function image to the registry
Deploying function to the cluster
Function updated at URL: http://func-default.apps.zroubali.devcluster.openshift.com

Delete

./faas delete -p func
Removing Knative Service: func

Fixes: #189

[lance]

LGTM generally. Although, I think we are moving away from using using capital "F" for functions. As @rhuss noted in #187

If its to pronounce the concept, then consequently this should be "Function as a Service". But for readability reasons and to avoid a question mark when reading this, I suggest to keep it lower case

[zroubalik]

@lance ok, fixed in the last commit