Additional changes, improved README, still PoC

Signed-off-by: Josh Salomon <jsalomon@redhat.com>

README.md

@@ -18,17 +18,13 @@ You’ll need a Kubernetes cluster to run against. You can use [KIND](https://si
 **Note:** Your controller will automatically use the current context in your kubeconfig file (i.e. whatever cluster `kubectl cluster-info` shows).
 
 ### Running on the cluster
-1. Install Instances of Custom Resources:
 
-```sh
-kubectl apply -f config/samples/
-```
-
-2. Build and push your image to the location specified by `IMG`:
+1. Build and push your image to the location specified by `IMG`:
 
 ```sh
 make docker-build docker-push IMG=<some-registry>/codecoapp-operator:tag
 ```
+
 **Note I:** This is required only after code changes in the operator.   
 **Note II:** For _kind_ k8s cluster there is a need to push also the controller to a registry, 
 do the following for loading it (a one time step) - this is needed if the cluster can't
@@ -36,12 +32,35 @@ access images on _localhost_
 
     > docker push localhost/controller:latest <some-registry>/controller
     
-3. Deploy the controller to the cluster with the image specified by `IMG`:
+2. Deploy the controller to the cluster with the image specified by `IMG`:
 
 ```sh
 make deploy IMG=<some-registry>/controller:latest
 ```
 
+After successful deployment you should see a pod, a service, a deployment and a replicaset in the odecoapp-operator-system namespace - for example:
+```sh
+  >kubectl get all -n codecoapp-operator-system
+  NAME                                                         READY   STATUS    RESTARTS   AGE
+  pod/codecoapp-operator-controller-manager-8664b86964-6k9t4   2/2     Running   0          11s
+
+  NAME                                                            TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)    AGE
+  service/codecoapp-operator-controller-manager-metrics-service   ClusterIP   11.11.11.111   <none>        8443/TCP   11s
+
+  NAME                                                    READY   UP-TO-DATE   AVAILABLE   AGE
+  deployment.apps/codecoapp-operator-controller-manager   1/1     1            1           11s
+
+  NAME                                                               DESIRED   CURRENT   READY   AGE
+  replicaset.apps/codecoapp-operator-controller-manager-8664b86964   1         1         1       11s
+```
+
+The pod should be in _Running_ state and ready 
+
+3. Install Instances of Custom Resources:
+
+```sh
+kubectl apply -f config/samples/
+```
 ### Uninstall CRDs
 To delete the CRDs from the cluster:
 

api/v1alpha1/codecoapp_types.go

@@ -76,7 +76,7 @@ type CodecoAppStatus struct {
 	// Status expresses the CODECO application status by the CODECO framework. Edit codecoapp_types.go to remove/update
 	Status CodecoStatus `json:"status,omitempty"`
 	// ErrorMsg describes the CODECO application error. Edit codecoapp_types.go to remove/update
-	ErrorMsg string `json:"errorMsg,omitempty"`
+	ErrorMsg string `json:"errormsg,omitempty"`
 }
 
 //+kubebuilder:object:root=true

config/crd/bases/codeco.he-codeco.eu_codecoapps.yaml

@@ -59,7 +59,7 @@ spec:
           status:
             description: CodecoAppStatus defines the observed state of CodecoApp
             properties:
-              errorMsg:
+              errormsg:
                 description: ErrorMsg describes the CODECO application error. Edit
                   codecoapp_types.go to remove/update
                 type: string

config/manager/kustomization.yaml

@@ -4,5 +4,5 @@ apiVersion: kustomize.config.k8s.io/v1beta1
 kind: Kustomization
 images:
 - name: controller
-  newName: quay.io/jsalomon/controller
-  newTag: latest
+  newName: quay.io/jsalomon/codecoapp-test
+  newTag: v0.0.1

controllers/codecoapp_controller.go

@@ -60,6 +60,7 @@ func (r *CodecoAppReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 		}
 		return ctrl.Result{}, err
 	}
+	// TODO(user): your logic here
 
 	if codecoAppCR.Status.Status == "" {
 		codecoAppCR.Status.Status = codecov1alpha1.OK
@@ -67,10 +68,16 @@ func (r *CodecoAppReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 		// intentionally do nothing
 	}
 
+	if codecoAppCR.Status.ErrorMsg == "" {
+		codecoAppCR.Status.ErrorMsg = "No errors"
+	} else {
+		// intentionally do nothing
+	}
+
 	fmt.Printf("CodecoApp: %v\n", codecoAppCR)
-	// TODO(user): your logic here
 
-	return ctrl.Result{}, nil
+	err := r.Status().Update(ctx, codecoAppCR)
+	return ctrl.Result{}, err
 }
 
 // SetupWithManager sets up the controller with the Manager.