Don't generate schema elements for volatile fields

Author: russgold

docs/domains/Domain.json

@@ -302,9 +302,6 @@
           "type": "number",
           "minimum": 0
         },
-        "modified": {
-          "type": "boolean"
-        },
         "startTime": {
           "description": "RFC 3339 date and time at which the operator started the domain. This will be when the operator begins processing and will precede when the various servers or clusters are available.",
           "$ref": "#/definitions/DateTime"
@@ -320,10 +317,7 @@
           "description": "A human readable message indicating details about why the domain is in this condition.",
           "type": "string"
         }
-      },
-      "required": [
-        "modified"
-      ]
+      }
     },
     "KubernetesResource": {
       "type": "object",

docs/domains/Domain.md

@@ -46,7 +46,6 @@ DomainStatus represents information about the status of a domain. Status may tra
 | --- | --- | --- |
 | conditions | array of [Domain Condition](#domain-condition) | Current service state of domain. |
 | message | string | A human readable message indicating details about why the domain is in this condition. |
-| modified | boolean |  |
 | reason | string | A brief CamelCase message indicating details about why the domain is in this state. |
 | replicas | number | The number of running managed servers in the WebLogic cluster if there is only one cluster in the domain and where the cluster does not explicitly configure its replicas in a cluster specification. |
 | servers | array of [Server Status](#server-status) | Status of WebLogic servers in this domain. |

docs/domains/index.html

@@ -1222,9 +1222,6 @@
           "type": "number",
           "minimum": 0.0
         },
-        "modified": {
-          "type": "boolean"
-        },
         "startTime": {
           "description": "RFC 3339 date and time at which the operator started the domain. This will be when the operator begins processing and will precede when the various servers or clusters are available.",
           "$ref": "#/definitions/DateTime"
@@ -1240,10 +1237,7 @@
           "description": "A human readable message indicating details about why the domain is in this condition.",
           "type": "string"
         }
-      },
-      "required": [
-        "modified"
-      ]
+      }
     },
     "KubernetesResource": {
       "type": "object",

json-schema/src/main/java/oracle/kubernetes/json/SchemaGenerator.java

@@ -186,13 +186,17 @@ void generateFieldIn(Map<String, Object> map, Field field) {
   }
 
   private boolean includeInSchema(Field field) {
-    return !isStatic(field) && !ignoreAsDeprecated(field);
+    return !isStatic(field) && !isVolatile(field) && !ignoreAsDeprecated(field);
   }
 
   private boolean isStatic(Field field) {
     return Modifier.isStatic(field.getModifiers());
   }
 
+  private boolean isVolatile(Field field) {
+    return Modifier.isVolatile(field.getModifiers());
+  }
+
   private boolean ignoreAsDeprecated(Field field) {
     return !includeDeprecated && field.getAnnotation(Deprecated.class) != null;
   }

json-schema/src/test/java/oracle/kubernetes/json/SchemaGeneratorTest.java

@@ -90,6 +90,16 @@ public void generateSchemaForIntArray() throws NoSuchFieldException {
   @SuppressWarnings("unused")
   private int[] intArray;
 
+  @Test
+  public void doNotGenerateSchemaForVolatileFields() throws NoSuchFieldException {
+    Object schema = generateForField(getClass().getDeclaredField("ignoreMe"));
+
+    assertThat(schema, hasNoJsonPath("$.ignoreMe"));
+  }
+
+  @SuppressWarnings("unused")
+  private volatile boolean ignoreMe;
+
   @Test
   public void generateSchemaForEnum() throws NoSuchFieldException {
     Object schema = generateForField(getClass().getDeclaredField("colors"));