Merge pull request openshift#1004 from alanconway/condition-doc

LOG-1012 Clarify meaning of Ready and Degraded conditions.

Author: openshift-merge-robot

pkg/apis/logging/v1/conditions.go

@@ -20,9 +20,29 @@ func NewCondition(t status.ConditionType, s corev1.ConditionStatus, r status.Con
 }
 
 const (
-	// Ready object is providing service. All objects support this condition type.
+	// Ready indicates the service is ready.
+	//
+	// Ready=True means the operands are running and providing some service.
+	// See the Degraded condition to distinguish full service from partial service.
+	//
+	// Ready=False means the operands cannot provide any service, and
+	// the operator cannot recover without some external change. Either
+	// the spec is invalid, or there is some environmental problem that is
+	// outside of the the operator's control.
+	//
+	// Ready=Unknown means the operator is in transition.
+	//
 	ConditionReady status.ConditionType = "Ready"
-	// Degraded object can provide some service, but not everything requested in the spec.
+
+	// Degraded indicates partial service is available.
+	//
+	// Degraded=True means the operands can fulfill some of the `spec`, but not all,
+	// even when Ready=True.
+	//
+	// Degraded=False with Ready=True means the operands are providing full service.
+	//
+	// Degraded=Unknown means the operator is in transition.
+	//
 	ConditionDegraded status.ConditionType = "Degraded"
 )