Review comments

Author: rjeberhard

docs-source/content/release-notes.md

@@ -8,7 +8,7 @@ draft: false
 
 | Date | Version | Introduces backward incompatibilities | Change |
 | --- | --- | --- | --- |
-| June 1, 2020 | v2.6.0 | no | Kubernetes 1.16, 1.17, and 1.18 support. Removal of support for Kubernetes 1.13 and earlier. This release can be run in the same cluster with operators of either 2.5.0 and below, or with 3.x providing an upgrade path. Certified support of Oracle Linux Cloud Native Environment (OLCNE) 1.1 with Kubernetes 1.17.0.
+| June 22, 2020 | v2.6.0 | no | Kubernetes 1.16, 1.17, and 1.18 support. Removal of support for Kubernetes 1.13 and earlier. This release can be run in the same cluster with operators of either 2.5.0 and below, or with 3.x providing an upgrade path. Certified support of Oracle Linux Cloud Native Environment (OLCNE) 1.1 with Kubernetes 1.17.0.
 | May 8, 2020 | v3.0.0-rc1 | yes | Adds Model in Image feature. Support for Kubernetes 1.16. Removal of support for Kubernetes 1.13 and earlier. Removal of support for Helm 2.x. This release candidate cannot be run in the same cluster as another release of the operator.  You can upgrade from the current release of the operator to 3.0.0-rc1 without needing to restart or recreate any existing domains. However, please note that we _do_ plan to support running the final 3.0.0 release in the same cluster with at least one 2.x release of the operator to allow for staged migration. |
 | February 26, 2020 | v2.5.0 | no | Support for Helm 3.x and OpenShift 4.3.  Operator can be installed in a namespace-dedicated mode where operator requires no cluster-level Kubernetes privileges. This version is not supported on Kubernetes 1.16+; check the [prerequisites]({{< relref "/userguide/introduction/introduction#operator-prerequisites" >}}).
 | November 15, 2019 | v2.4.0 | no | Includes fixes for a variety of issues related to FMW infrastructure domains and pod variable substitution.  Operator now uses WebLogic Deploy Tooling 1.6.0 and the latest version of the Kubernetes Java Client.

docs/domains/Domain.json

@@ -172,7 +172,7 @@
       "type": "object",
       "properties": {
         "istio": {
-          "description": "The Istio service mesh integration settings",
+          "description": "The Istio service mesh integration settings.",
           "$ref": "#/definitions/Istio"
         }
       }
@@ -318,7 +318,7 @@
           "$ref": "#/definitions/ServerPod"
         },
         "logHome": {
-          "description": "The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files",
+          "description": "The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files.",
           "type": "string"
         },
         "includeServerOutInPodLog": {

docs/domains/Domain.md

@@ -29,7 +29,7 @@ DomainSpec is a description of a domain.
 | `imagePullPolicy` | string | The image pull policy for the WebLogic Docker image. Legal values are Always, Never and IfNotPresent. Defaults to Always if image ends in :latest, IfNotPresent otherwise. |
 | `imagePullSecrets` | array of [Local Object Reference](k8s1.13.5.md#local-object-reference) | A list of image pull secrets for the WebLogic Docker image. |
 | `includeServerOutInPodLog` | Boolean | If true (the default), then the server .out file will be included in the pod's stdout. |
-| `logHome` | string | The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files |
+| `logHome` | string | The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files. |
 | `logHomeEnabled` | Boolean | Specified whether the log home folder is enabled. Not required. Defaults to true if domainHomeInImage is false. Defaults to false if domainHomeInImage is true.  |
 | `managedServers` | array of [Managed Server](#managed-server) | Configuration for individual Managed Servers. |
 | `replicas` | number | The number of managed servers to run in any cluster that does not specify a replica count. |
@@ -87,7 +87,7 @@ An element representing a cluster in the domain configuration.
 
 | Name | Type | Description |
 | --- | --- | --- |
-| `istio` | [Istio](#istio) | The Istio service mesh integration settings |
+| `istio` | [Istio](#istio) | The Istio service mesh integration settings. |
 
 ### Managed Server
 

docs/domains/index.html

@@ -1092,7 +1092,7 @@
       "type": "object",
       "properties": {
         "istio": {
-          "description": "The Istio service mesh integration settings",
+          "description": "The Istio service mesh integration settings.",
           "$ref": "#/definitions/Istio"
         }
       }
@@ -1238,7 +1238,7 @@
           "$ref": "#/definitions/ServerPod"
         },
         "logHome": {
-          "description": "The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files",
+          "description": "The in-pod name of the directory in which to store the domain, Node Manager, server logs, and server *.out files.",
           "type": "string"
         },
         "includeServerOutInPodLog": {

kubernetes/crd/domain-crd.yaml

@@ -51,7 +51,7 @@ spec:
                 properties:
                   istio:
                     type: object
-                    description: The Istio service mesh integration settings
+                    description: The Istio service mesh integration settings.
                     properties:
                       readinessPort:
                         type: number
@@ -10412,7 +10412,7 @@ spec:
               logHome:
                 type: string
                 description: The in-pod name of the directory in which to store the
-                  domain, Node Manager, server logs, and server *.out files
+                  domain, Node Manager, server logs, and server *.out files.
               includeServerOutInPodLog:
                 type: boolean
                 description: If true (the default), then the server .out file will

kubernetes/crd/domain-v1beta1-crd.yaml

@@ -53,7 +53,7 @@ spec:
               properties:
                 istio:
                   type: object
-                  description: The Istio service mesh integration settings
+                  description: The Istio service mesh integration settings.
                   properties:
                     readinessPort:
                       type: number
@@ -10204,7 +10204,7 @@ spec:
             logHome:
               type: string
               description: The in-pod name of the directory in which to store the
-                domain, Node Manager, server logs, and server *.out files
+                domain, Node Manager, server logs, and server *.out files.
             includeServerOutInPodLog:
               type: boolean
               description: If true (the default), then the server .out file will be

operator/src/main/java/oracle/kubernetes/weblogic/domain/model/Configuration.java

@@ -9,7 +9,7 @@
 import org.apache.commons.lang3.builder.ToStringBuilder;
 
 public class Configuration {
-  @Description("The Istio service mesh integration settings")
+  @Description("The Istio service mesh integration settings.")
   private Istio istio;
 
   public Istio getIstio() {

operator/src/main/java/oracle/kubernetes/weblogic/domain/model/DomainSpec.java

@@ -87,7 +87,7 @@ public class DomainSpec extends BaseConfiguration {
    */
   @Description(
       "The in-pod name of the directory in which to store the domain, Node Manager, server logs, "
-          + "and server *.out files")
+          + "and server *.out files.")
   private String logHome;
 
   /**