- 05 Jan, 2021 2 commits
-
-
Geoff Simmons authored
I believe this became necessary because the meaning of the state and temperature fields in vcl.list has changed in recent Varnish versions (which we are now using).
-
Geoff Simmons authored
This makes backend names more readily recognizable in VCL sources, and as used in tools such as varnishlog and varnishstat. If the Pod ns/name is not available, fall back to the previous scheme of generating names from the Endpoint network addresses.
-
- 03 Jan, 2021 1 commit
-
-
Geoff Simmons authored
This makes the naming for Varnish and IngressBackend instances more legible, and more readily associated with the Pods on which they run, than by using only network addresses. First use the names in the controller logs.
-
- 01 Jan, 2021 1 commit
-
-
Geoff Simmons authored
This is possible now that we are using Varnish versions that support return(error(404)) from the backend side. It eliminates unnecessary computation when a request leads to a cache hit or synthetic response. It also simplifies the generated VCL code, since the evaluation happens in one place, rather than in each of hit, miss, pass and pipe.
-
- 31 Dec, 2020 3 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
- 30 Dec, 2020 8 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
Cached responses were interfering with other tests when the e2e test script was run.
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
Type declarations in struct literals are unnecessary where the type is unambiguous.
-
Geoff Simmons authored
-
- 07 Dec, 2020 1 commit
-
-
Geoff Simmons authored
Forgot it in the previous commit.
-
- 06 Dec, 2020 3 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
- 04 Dec, 2020 1 commit
-
-
Geoff Simmons authored
-
- 26 Nov, 2020 5 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
- 25 Nov, 2020 2 commits
-
-
Geoff Simmons authored
Also set up the tests so that the external IP and port for both the NodePort and LoadBalancer tests. For this it is necessary to pass in the external cluster IP, and/or set up an external IP for the LoadBalancer Service. If neither of these are available, just use port-forwarding as for the other tests. That doesn't actually test the Service types, since connections are forwarded to the clusterIP. But the tests can be run in automation when the external networking is not available.
-
Geoff Simmons authored
-
- 24 Nov, 2020 3 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
The helm charts already do this, it is a change for the kubectl/yaml examples, which have been using NodePort. NodePort has been unnecessary since we've automated testing by using port-forward to the Service. This brings the two different deployment methods more in line. We will add tests for LoadBalancer and NodePort Services. The docs are now even more out of sync with the actual example/test code, since they still reference the NodePort type.
-
Geoff Simmons authored
Previously the status was only updated when the sync to implement an Ingress was successful. This will update the status due to changes in addresses (IPs or hosts) assigned to the Service.
-
- 20 Nov, 2020 1 commit
-
-
Geoff Simmons authored
Also, the Service info sources for Ingress loadBalancer status depend on the Service type, mostly following the nginx ingress implementation.
-
- 15 Oct, 2020 2 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
The addresses for this array are taken from the public names and/or IPs in the spec for Service(s) that expose the Ingress. These are identified as: - in the same namespace as the admin Service - have the label viking.uplex.de/svc=public - have the same selectors as the admin Service - type is one of ClusterIP, NodePort or LoadBalancer The label viking.uplex.de/svc is only required if the Ingress status update is required. For example to use a tool like ArgoCD, or if the cloud provider requires it. Set the label in the Service template for the viking-service chart.
-
- 14 Oct, 2020 1 commit
-
-
Geoff Simmons authored
No longer needed since we've set the Pods to ready regardless of whether an Ingress has been configured.
-
- 13 Oct, 2020 6 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-