PR #177 is not backwards compatible

[falfaro]

PR #177 is not backwards compatible. It introduces a new field in the JSON configuration file named authz_domain which didn't exist before the PR.

I have a GKE cluster that was deployed with kubeprod manifests before PR #177. This how the JSON configuration file looks like:

{
  "dnsZone": "...",
  "contactEmail": "...",
  "externalDns": {
    "credentials": "...",
    "project": "..."
  },
  "oauthProxy": {
    "client_id": "...",
    "client_secret": "...",
    "cookie_secret": "...",
    "google_groups": [],
    "google_admin_email": "",
    "google_service_account_json": ""
  }
}

Running kubecfg update after PR #177 results in the following error:

 kubecfg update kubeprod-manifest.jsonnet
ERROR Error reading kubeprod-manifest.jsonnet: RUNTIME ERROR: Field does not exist: authz_domain
        file:///Users/falfaro/go/src/github.com/bitnami/kube-prod-runtime/manifests/platforms/gke.jsonnet:110:35-67     object <anonymous>
        file:///Users/falfaro/go/src/github.com/bitnami/kube-prod-runtime/manifests/lib/kube.libsonnet:32:25-29 thunk from <thunk from <function <anonymous>>>
        <std>:659:15-22 thunk <val> from <function <format_codes_arr>>
        <std>:666:27-30 thunk from <thunk <s> from <function <format_codes_arr>>>
        <std>:536:22-25 thunk from <function <format_code>>
        <builtin>       builtin function <toString>
        <std>:666:15-60 thunk <s> from <function <format_codes_arr>>
        <std>:671:24-25 thunk from <thunk <s_padded> from <function <format_codes_arr>>>
        <std>:451:30-33 thunk from <thunk from <function <pad_left>>>
        <builtin>       builtin function <length>
        ...
        <std>:451:7-38  function <pad_left>
        <std>:451:7-44  function <pad_left>
        <std>:671:15-39 thunk <s_padded> from <function <format_codes_arr>>
        <std>:677:55-63 thunk from <function <format_codes_arr>>
        <std>:677:51-63 thunk from <function <format_codes_arr>>
        <std>:677:11-64 function <format_codes_arr>
        <std>:721:7-46  function <anonymous>
        <std>:203:7-23  function <anonymous>

        During manifestation

[falfaro]

I think the solution consists of modifying the manifests that rely on authz_domain to adopt a sane default value (whichever we were using before PR #177) which I believe was "*" for AKS (not sure about GKE).

[arapulido]

We are closing this PR because #177 was merged before 1.0-rc1 was released, and therefore, we can assume a backwards compatibility