Merge pull request #887 from Kong/release/1.0.0-rc.1

mflendrich · web-flow · commit 8dd3603454cd · 2020-10-01T23:00:01.000+02:00
Release 1.0.0-rc.1 to next From #887
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 # Table of Contents
 
+ - [1.0.0-rc.1](#0100rc1---20201002)
  - [0.10.0](#0100---20200915)
  - [0.9.1](#091---20200608)
  - [0.9.0](#090---20200526)
@@ -23,6 +24,49 @@
  - [0.0.5](#005---20180602)
  - [0.0.4 and prior](#004-and-prior)
 
+## [1.0.0-rc.1] - 2020/10/02
+
+#### Breaking changes
+
+- The controller no longer supports versions of Kong prior to 2.0.0.
+  [#875](https://github.com/Kong/kubernetes-ingress-controller/pull/875)
+- Deprecated 0.x.x flags are no longer supported. Please see [the documentation
+  changes](https://github.com/Kong/kubernetes-ingress-controller/pull/866/files#diff-9a686fb3bf9c18ab81952a0933fb5c00)
+  for a complete list of removed flags and their replacements. Note that this
+  change applies to both flags and their equivalent environment variables, e.g.
+  for `--admin-header`, if you set `CONTROLLER_ADMIN_HEADER`, you should now
+  use `CONTROLLER_KONG_ADMIN_HEADER`.
+  [#866](https://github.com/Kong/kubernetes-ingress-controller/pull/866)
+- KongCredential custom resources are no longer supported. You should convert
+  any KongCredential resources to [credential Secrets](https://github.com/Kong/kubernetes-ingress-controller/blob/next/docs/guides/using-consumer-credential-resource.md#provision-a-consumer)
+  before upgrading to 1.0.0.
+  [#862](https://github.com/Kong/kubernetes-ingress-controller/pull/862)
+- Deprecated 0.x.x annotations are no longer supported. Please see [the
+  documentation changes](https://github.com/Kong/kubernetes-ingress-controller/pull/873/files#diff-777e08783d63482620961c4f93a1e1f6)
+  for a complete list of removed annotations and their replacements.
+  [#873](https://github.com/Kong/kubernetes-ingress-controller/pull/873)
+
+#### Added
+
+- The controller Docker registry now has minor version tags. These always point
+  to the latest patch release for a given minor version, e.g. if `1.0.3` is the
+  latest patch release for the `1.0.x` series, the `1.0` Docker tag will point
+  to `1.0.3`.
+  [#747](https://github.com/Kong/kubernetes-ingress-controller/pull/747)
+- Custom resources now all have a status field. For 1.0.0, this field is a
+  placeholder, and does not contain any actual status information. Future
+  versions will add status information that reflects whether the controller has
+  created Kong configuration for that custom resource.
+  [#824](https://github.com/Kong/kubernetes-ingress-controller/pull/824)
+- Version compatibility documentation now includes [information about supported
+  Kubernetes versions for a given controller version](https://github.com/Kong/kubernetes-ingress-controller/blob/main/docs/references/version-compatibility.md#kubernetes).
+  [#820](https://github.com/Kong/kubernetes-ingress-controller/pull/820)
+
+#### Fixed
+
+- EKS documentation now uses hostnames rather than IP addresses.
+  [#877](https://github.com/Kong/kubernetes-ingress-controller/pull/877)
+
 ## [0.10.0] - 2020/09/15
 
 #### Breaking changes
diff --git a/CODEOWNERS b/CODEOWNERS
@@ -1 +1 @@
-* @hbagdi @rainest
+* @hbagdi @rainest @mflendrich
diff --git a/MAINTAINERS.md b/MAINTAINERS.md
@@ -2,6 +2,6 @@
 
 ## Current maintainers
 
-- Harry Bagdi ([@hbagdi](https://github.com/hbagdi)
+- Harry Bagdi ([@hbagdi](https://github.com/hbagdi))
 - Travis Raines ([@rainest](https://github.com/hbagdi))
-
+- Michał Flendrich ([@mflendrich](https://github.com/mflendrich))
diff --git a/Makefile b/Makefile
@@ -1,5 +1,5 @@
 REGISTRY?=kong-docker-kubernetes-ingress-controller.bintray.io
-TAG?=0.10.0
+TAG?=1.0.0-rc.1
 REPO_INFO=$(shell git config --get remote.origin.url)
 IMGNAME?=kong-ingress-controller
 IMAGE = $(REGISTRY)/$(IMGNAME)
diff --git a/deploy/manifests/base/kong-ingress-dbless.yaml b/deploy/manifests/base/kong-ingress-dbless.yaml
@@ -102,7 +102,7 @@ spec:
             fieldRef:
               apiVersion: v1
               fieldPath: metadata.namespace
-        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:0.10.0
+        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.0.0-rc.1
         imagePullPolicy: IfNotPresent
         ports:
         - name: webhook
diff --git a/deploy/single/all-in-one-dbless-k4k8s-enterprise.yaml b/deploy/single/all-in-one-dbless-k4k8s-enterprise.yaml
@@ -698,7 +698,7 @@ spec:
             fieldRef:
               apiVersion: v1
               fieldPath: metadata.namespace
-        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:0.10.0
+        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.0.0-rc.1
         imagePullPolicy: IfNotPresent
         livenessProbe:
           failureThreshold: 3
diff --git a/deploy/single/all-in-one-dbless.yaml b/deploy/single/all-in-one-dbless.yaml
@@ -693,7 +693,7 @@ spec:
             fieldRef:
               apiVersion: v1
               fieldPath: metadata.namespace
-        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:0.10.0
+        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.0.0-rc.1
         imagePullPolicy: IfNotPresent
         livenessProbe:
           failureThreshold: 3
diff --git a/deploy/single/all-in-one-postgres-enterprise.yaml b/deploy/single/all-in-one-postgres-enterprise.yaml
@@ -767,7 +767,7 @@ spec:
             secretKeyRef:
               key: password
               name: kong-enterprise-superuser-password
-        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:0.10.0
+        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.0.0-rc.1
         imagePullPolicy: IfNotPresent
         livenessProbe:
           failureThreshold: 3
diff --git a/deploy/single/all-in-one-postgres.yaml b/deploy/single/all-in-one-postgres.yaml
@@ -711,7 +711,7 @@ spec:
             fieldRef:
               apiVersion: v1
               fieldPath: metadata.namespace
-        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:0.10.0
+        image: kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.0.0-rc.1
         imagePullPolicy: IfNotPresent
         livenessProbe:
           failureThreshold: 3
diff --git a/docs/deployment/eks.md b/docs/deployment/eks.md
@@ -59,14 +59,14 @@ Execute the following command to get the IP address at which Kong is accessible:
 
 ```bash
 $ kubectl get services -n kong
-NAME         TYPE           CLUSTER-IP      EXTERNAL-IP     PORT(S)                      AGE
-kong-proxy   LoadBalancer   10.63.250.199   203.0.113.42   80:31929/TCP,443:31408/TCP   57d
+NAME         TYPE           CLUSTER-IP      EXTERNAL-IP                           PORT(S)                      AGE
+kong-proxy   LoadBalancer   10.63.250.199   example.eu-west-1.elb.amazonaws.com   80:31929/TCP,443:31408/TCP   57d
 ```
 
-Create an environment variable to hold the IP address:
+Create an environment variable to hold the ELB hostname:
 
 ```bash
-$ export PROXY_IP=$(kubectl get -o jsonpath="{.status.loadBalancer.ingress[0].ip}" service -n kong kong-proxy)
+$ export PROXY_IP=$(kubectl get -o jsonpath="{.status.loadBalancer.ingress[0].hostname}" service -n kong kong-proxy)
 ```
 
 > Note: It may take some time for Amazon to actually associate the
diff --git a/docs/references/cli-arguments.md b/docs/references/cli-arguments.md
@@ -30,7 +30,6 @@ Following table describes all the flags that are available:
 | --admission-webhook-cert             |`string`   | none                            | PEM-encoded certificate string for TLS handshake.|
 | --admission-webhook-key              |`string`   | none                            | PEM-encoded private key string for TLS handshake.|
 | --admission-webhook-listen           |`string`   | `off`                           | The address to start admission controller on (ip:port). Setting it to 'off' disables the admission controller.|
-| --alsologtostderr                    |`boolean`  | `false`                         | Logs are written to standard error as well as to files.|
 | --anonymous-reports                  |`string`   | `true`                          | Send anonymized usage data to help improve Kong.|
 | --apiserver-host                     |`string`   | none                            | The address of the Kubernetes Apiserver to connect to in the format of protocol://address:port, e.g., "http://localhost:8080. If not specified, the assumption is that the binary runs inside a Kubernetes cluster and local discovery is attempted.|
 | --disable-ingress-extensionsv1beta1  |`boolean`  | `false`                         | Disable processing Ingress resources with apiVersion `extensions/v1beta1`.|
@@ -49,11 +48,10 @@ Following table describes all the flags that are available:
 | --kong-admin-url                     |`string`   | `http://localhost:8001`         | The address of the Kong Admin URL to connect to in the format of `protocol://address:port`.|
 | --kong-workspace                     |`string`   | `default`                       | Workspace in Kong Enterprise to be configured.|
 | --kong-custom-entities-secret        |`string`   | none                            | Secret containing custom entities to be populated in DB-less mode, takes the form `namespace/name`.|
+| --log-format                         |`string`   | `text`                          | Format of logs of the controller. Allowed values are `text` and `json`. |
+| --log-level                          |`string`   | `info`                          | Level of logging for the controller. Allowed values are `trace`, `debug`, `info`, `warn`, `error`, `fatal` and `panic`. |
 | --enable-reverse-sync                |`bool`     | `false`                         | Enable reverse checks from Kong to Kubernetes. Use this option only if a human has edit access to Kong's Admin API. |
 | --kubeconfig                         |`string`   | none                            | Path to kubeconfig file with authorization and master location information.|
-| --log_backtrace_at                   |`string`   | none                            | When set to a file and line number holding a logging statement, such as -log_backtrace_at=gopherflakes.go:234 a stack trace will be written to the Info log whenever execution hits that statement. (Unlike with -vmodule, the ".go" must be present.)|
-| --log_dir                            |`string`   | none                            | If non-empty, write log files in this directory.|
-| --logtostderr                        |`boolean`  | `true`                          | Logs to standard error instead of files.|
 | --profiling                          |`boolean`  | `true`                          | Enable profiling via web interface `host:port/debug/pprof/`. |
 | --publish-service                    |`string`   | none                            | The namespaces and name of the Kubernetes Service fronting Kong Ingress Controller in the form of namespace/name. The controller will set the status of the Ingress resouces to match the endpoints of this service. In reference deployments, this is kong/kong-proxy.|
 | --publish-status-address             |`string`   | none                            | User customized address to be set in the status of ingress resources. The controller will set the endpoint records on the ingress using this address.|
@@ -65,9 +63,7 @@ Following table describes all the flags that are available:
 | --sync-rate-limit                    |`float32`  | `0.3`                           | Define the sync frequency upper limit. |
 | --update-status                      |`boolean`  | `true`                          | Indicates if the ingress controller should update the Ingress status IP/hostname.|
 | --update-status-on-shutdown          |`boolean`  | `true`                          | Indicates if the ingress controller should update the Ingress status IP/hostname when the controller is being stopped.|
-|  -v, --v Level                        `int`      | `0`                             | | Enable V-leveled logging at the specified level.|
 | --version                            |`boolean`  | `false`                         | Shows release information about the Kong Ingress controller.|
-| --vmodule moduleSpec                 |`string`   | none                            | The syntax of the argument is a comma-separated list of pattern=N, where pattern is a literal file name (minus the ".go" suffix) or "glob" pattern and N is a V level. For instance, -vmodule=gopher*=3 sets the V level to 3 in all Go files whose names begin "gopher".|
 | --watch-namespace                    |`string`   | none                            | Namespace to watch for Ingress and custom resources. The default value of an empty string results in the controller watching for resources in all namespaces and configuring Kong accordingly.|
 | --help                               |`boolean`  | `false`                         | Shows this documentation on the CLI and exit.|
 
diff --git a/docs/references/version-compatibility.md b/docs/references/version-compatibility.md
@@ -50,3 +50,15 @@ other enterprise functionality, built on top of the Open-Source Kong Gateway.
 | Kong Enterprise 1.3.x    | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :white_check_mark: | :white_check_mark: | :white_check_mark: | :x:                | :x:                |
 | Kong Enterprise 1.5.x    | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
 | Kong Enterprise 2.1.x    | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :x:                | :white_check_mark: | :white_check_mark: |
+
+## Kubernetes
+
+| Kong Ingress Controller  | 0.9.x              | 0.10.x             | 1.0.x              |
+|--------------------------|:------------------:|:------------------:|:------------------:|
+| Kubernetes 1.13          | :x:                | :x:                | :x:                |
+| Kubernetes 1.14          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Kubernetes 1.15          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Kubernetes 1.16          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Kubernetes 1.17          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Kubernetes 1.18          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
+| Kubernetes 1.19          | :white_check_mark: | :white_check_mark: | :white_check_mark: |
diff --git a/hack/dev/common/custom-types.yaml b/hack/dev/common/custom-types.yaml
@@ -192,44 +192,6 @@ spec:
   subresources:
     status: {}
 
----
-apiVersion: apiextensions.k8s.io/v1beta1
-kind: CustomResourceDefinition
-metadata:
-  name: kongcredentials.configuration.konghq.com
-spec:
-  group: configuration.konghq.com
-  version: v1
-  scope: Namespaced
-  names:
-    kind: KongCredential
-    plural: kongcredentials
-  additionalPrinterColumns:
-  - name: Credential-type
-    type: string
-    description: Type of credential
-    JSONPath: .type
-  - name: Age
-    type: date
-    description: Age
-    JSONPath: .metadata.creationTimestamp
-  - name: Consumer-Ref
-    type: string
-    description: Owner of the credential
-    JSONPath: .consumerRef
-  validation:
-    openAPIV3Schema:
-      required:
-      - consumerRef
-      - type
-      properties:
-        consumerRef:
-          type: string
-        type:
-          type: string
-  subresources:
-    status: {}
-
 ---
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
diff --git a/internal/ingress/controller/controller.go b/internal/ingress/controller/controller.go
@@ -125,9 +125,7 @@ func (n *KongController) syncIngress(interface{}) error {
 	return nil
 }
 
-// NewKongController creates a new NGINX Ingress controller.
-// If the environment variable NGINX_BINARY exists it will be used
-// as source for nginx commands
+// NewKongController creates a new Ingress controller.
 func NewKongController(ctx context.Context,
 	config *Configuration,
 	updateCh *channels.RingChannel,
@@ -235,7 +233,7 @@ type KongController struct {
 	Logger logrus.FieldLogger
 }
 
-// Start starts a new NGINX master process running in foreground, blocking until the next call to
+// Start starts a new master process running in foreground, blocking until the next call to
 // Stop.
 func (n *KongController) Start() {
 	n.Logger.Debugf("startin up controller")
@@ -286,7 +284,7 @@ func (n *KongController) Start() {
 	}
 }
 
-// Stop stops the NGINX master process gracefully.
+// Stop stops the master process gracefully.
 func (n *KongController) Stop() error {
 	atomic.StoreUint32(&n.isShuttingDown, 1)
 

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-* @hbagdi @rainest`
	`1`	`+* @hbagdi @rainest @mflendrich`