Merge pull request #119 from IATI/fix/remove-logger-api

remove logger from apis template

README.md

@@ -34,6 +34,7 @@ Update the Extractor Tool
   - Take the previous revision offline.
 - Run the extractor tool bash script
 - You should see changes to a number of the ARM templates and policies in `./service`
+- NOTE: you need to manually remove the object with `"type": "Microsoft.ApiManagement/service/diagnostics"` from the `service/apim-iati-dev-apis.template.json` template, this is covered in the `service/apim-iati-dev-loggers.template.json` template.
 - If there are any changes (new parameters) in `service/apim-iati-dev-parameters.json`, ensure that you update the github actions workflows to pass in the new parameters:
   - `apim-ci.yml`
   - `apim-develop.yml`

service/apim-iati-dev-apis.template.json

@@ -9872,22 +9872,6 @@
         }
       }
     },
-    {
-      "apiVersion": "2021-08-01",
-      "type": "Microsoft.ApiManagement/service/diagnostics",
-      "name": "[concat(parameters('apimServiceName'), '/applicationinsights')]",
-      "dependsOn": [],
-      "properties": {
-        "alwaysLog": "allErrors",
-        "loggerId": "/subscriptions/bcaf7a00-7a14-4932-ac41-7bb0dee0d2a9/resourceGroups/rg-apim-dev/providers/Microsoft.ApiManagement/service/apim-iati-dev/loggers/appi-apim-dev",
-        "httpCorrelationProtocol": "Legacy",
-        "logClientIp": true,
-        "sampling": {
-          "samplingType": "fixed",
-          "percentage": 100.0
-        }
-      }
-    },
     {
       "apiVersion": "2021-08-01",
       "type": "Microsoft.ApiManagement/service/products/apis",