Cleanup bad JSON files

- Add missing commas
- Replace single quote with double quote
- events field contains a list of events (event-create-req.json)
- Remove empty files
- Adds a JSON validation test to the docs pipeline

Co-Authored-By: Nicolas Simonds <nic@metacloud.com>
Change-Id: I9a4a9b6b8e2da73b5f25145be3b2faa44ebdb340

doc/api_samples/os-floating-ip-dns/floating-ip-dns-create-or-update-entry-req.json

@@ -1,7 +1,6 @@
 {
-  'dns_entry' :
-    {
-      'ip': '192.168.53.11',
-      'dns_type': 'A'
-    }
+  "dns_entry": {
+    "ip": "192.168.53.11",
+    "dns_type": "A"
+  }
 }

doc/api_samples/os-floating-ip-dns/floating-ip-dns-create-or-update-req.json

@@ -1,8 +1,7 @@
 {
-  'domain_entry' :
-    {
-      'domain': 'domain1.example.org',
-      'scope': 'public',
-      'project': 'project1'
-   }
+  "domain_entry": {
+    "domain": "domain1.example.org",
+    "scope": "public",
+    "project": "project1"
+  }
 }

doc/v3/api_samples/os-availability-zone/server-post-req.json

@@ -3,7 +3,7 @@
         "name" : "new-server-test",
         "imageRef" : "http://openstack.example.com/openstack/images/70a599e0-31e7-49b7-b260-868f441e862b",
         "flavorRef" : "http://openstack.example.com/openstack/flavors/1",
-        "availability_zone" : "test"
+        "availability_zone" : "test",
         "metadata" : {
             "My Server Name" : "Apache1"
         }

doc/v3/api_samples/os-floating-ips-bulk/floating-ips-bulk-create-req.json

@@ -1,6 +1,6 @@
 {
     "floating_ips_bulk_create": {
-        "ip_range": "192.168.1.0/24"
+        "ip_range": "192.168.1.0/24",
         "pool": "nova",
         "interface": "eth0"
     }

doc/v3/api_samples/os-server-external-events/event-create-req.json

@@ -1,9 +1,10 @@
 {
-    "events": {[
-     {
-        "name": "test-event",
-        "tag": "foo",
-        "status": "completed",
-        "server_uuid": "3df201cf-2451-44f2-8d25-a4ca826fc1f3"
-     }]}
-}
+    "events": [
+        {
+            "name": "test-event",
+            "tag": "foo",
+            "status": "completed",
+            "server_uuid": "3df201cf-2451-44f2-8d25-a4ca826fc1f3"
+        }
+    ]
+}

tox.ini

@@ -59,7 +59,9 @@ commands =
 commands = {posargs}
 
 [testenv:docs]
-commands = python setup.py build_sphinx
+commands =
+  python setup.py build_sphinx
+  bash -c '! find doc/ -type f -name *.json | xargs -t -n1 python -m json.tool 2>&1 > /dev/null | grep -B1 -v ^python'
 
 [flake8]
 # E125 is deliberately excluded. See https://github.com/jcrocholl/pep8/issues/126