This is an automated email from the ASF dual-hosted git repository.

tokers pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git


The following commit(s) were added to refs/heads/master by this push:
     new cc67f0f  test: add api-breaker plugin e2e test cases (#393)
cc67f0f is described below

commit cc67f0f61fad0d4b7b6c2560d1440f37fc32fcec
Author: Alex Zhang <[email protected]>
AuthorDate: Wed Apr 21 10:11:30 2021 +0800

    test: add api-breaker plugin e2e test cases (#393)
---
 test/e2e/plugins/api_breaker.go         | 149 ++++++++++++++++++++++++++++++++
 test/e2e/testdata/apisix-gw-config.yaml |   1 +
 2 files changed, 150 insertions(+)

diff --git a/test/e2e/plugins/api_breaker.go b/test/e2e/plugins/api_breaker.go
new file mode 100644
index 0000000..f7a09f4
--- /dev/null
+++ b/test/e2e/plugins/api_breaker.go
@@ -0,0 +1,149 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
+// contributor license agreements.  See the NOTICE file distributed with
+// this work for additional information regarding copyright ownership.
+// The ASF licenses this file to You under the Apache License, Version 2.0
+// (the "License"); you may not use this file except in compliance with
+// the License.  You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+package plugins
+
+import (
+       "fmt"
+       "net/http"
+       "time"
+
+       "github.com/stretchr/testify/assert"
+
+       "github.com/apache/apisix-ingress-controller/test/e2e/scaffold"
+       "github.com/onsi/ginkgo"
+)
+
+var _ = ginkgo.Describe("api-breaker plugin", func() {
+       opts := &scaffold.Options{
+               Name:                    "default",
+               Kubeconfig:              scaffold.GetKubeconfig(),
+               APISIXConfigPath:        "testdata/apisix-gw-config.yaml",
+               APISIXDefaultConfigPath: 
"testdata/apisix-gw-config-default.yaml",
+               IngressAPISIXReplicas:   1,
+               HTTPBinServicePort:      80,
+               APISIXRouteVersion:      "apisix.apache.org/v2alpha1",
+       }
+       s := scaffold.NewScaffold(opts)
+       ginkgo.It("sanity", func() {
+               backendSvc, backendPorts := s.DefaultHTTPBackend()
+               ar := fmt.Sprintf(`
+apiVersion: apisix.apache.org/v2alpha1
+kind: ApisixRoute
+metadata:
+ name: httpbin-route
+spec:
+ http:
+ - name: rule1
+   match:
+     hosts:
+     - httpbin.org
+     paths:
+       - /status/*
+   backends:
+   - serviceName: %s
+     servicePort: %d
+     resolveGranularity: service
+   plugins:
+   - name: api-breaker
+     enable: true
+     config:
+       break_response_code: 502
+       unhealthy:
+         http_statuses:
+         - 505
+         failures: 2
+       max_breaker_sec: 3
+       healthy:
+         http_statuses:
+         - 200
+         successes: 2
+`, backendSvc, backendPorts[0])
+
+               assert.Nil(ginkgo.GinkgoT(), s.CreateResourceFromString(ar))
+               err := s.EnsureNumApisixUpstreamsCreated(1)
+               assert.Nil(ginkgo.GinkgoT(), err, "Checking number of 
upstreams")
+               err = s.EnsureNumApisixRoutesCreated(1)
+               assert.Nil(ginkgo.GinkgoT(), err, "Checking number of routes")
+
+               resp := 
s.NewAPISIXClient().GET("/status/200").WithHeader("Host", 
"httpbin.org").Expect()
+               resp.Status(http.StatusOK)
+
+               for i := 0; i < 2; i++ {
+                       resp = 
s.NewAPISIXClient().GET("/status/505").WithHeader("Host", 
"httpbin.org").Expect()
+                       resp.Status(505)
+               }
+
+               // Trigger the api-breaker threshold
+               resp = 
s.NewAPISIXClient().GET("/status/505").WithHeader("Host", 
"httpbin.org").Expect()
+               resp.Status(http.StatusBadGateway)
+
+               time.Sleep(3500 * time.Millisecond)
+               resp = 
s.NewAPISIXClient().GET("/status/200").WithHeader("Host", 
"httpbin.org").Expect()
+               resp.Status(http.StatusOK)
+       })
+       ginkgo.It("disable plugin", func() {
+               backendSvc, backendPorts := s.DefaultHTTPBackend()
+               ar := fmt.Sprintf(`
+apiVersion: apisix.apache.org/v2alpha1
+kind: ApisixRoute
+metadata:
+ name: httpbin-route
+spec:
+ http:
+ - name: rule1
+   match:
+     hosts:
+     - httpbin.org
+     paths:
+       - /status/*
+   backends:
+   - serviceName: %s
+     servicePort: %d
+     resolveGranularity: service
+   plugins:
+   - name: api-breaker
+     enable: false
+     config:
+       break_response_code: 502
+       unhealthy:
+         http_statuses:
+         - 505
+         failures: 2
+       max_breaker_sec: 3
+       healthy:
+         http_statuses:
+         - 200
+         successes: 2
+`, backendSvc, backendPorts[0])
+
+               assert.Nil(ginkgo.GinkgoT(), s.CreateResourceFromString(ar))
+               err := s.EnsureNumApisixUpstreamsCreated(1)
+               assert.Nil(ginkgo.GinkgoT(), err, "Checking number of 
upstreams")
+               err = s.EnsureNumApisixRoutesCreated(1)
+               assert.Nil(ginkgo.GinkgoT(), err, "Checking number of routes")
+
+               resp := 
s.NewAPISIXClient().GET("/status/200").WithHeader("Host", 
"httpbin.org").Expect()
+               resp.Status(http.StatusOK)
+
+               for i := 0; i < 2; i++ {
+                       resp = 
s.NewAPISIXClient().GET("/status/505").WithHeader("Host", 
"httpbin.org").Expect()
+                       resp.Status(505)
+               }
+
+               // Trigger the api-breaker threshold
+               resp = 
s.NewAPISIXClient().GET("/status/505").WithHeader("Host", 
"httpbin.org").Expect()
+               resp.Status(505)
+       })
+})
diff --git a/test/e2e/testdata/apisix-gw-config.yaml 
b/test/e2e/testdata/apisix-gw-config.yaml
index 25e39d1..a0ba901 100644
--- a/test/e2e/testdata/apisix-gw-config.yaml
+++ b/test/e2e/testdata/apisix-gw-config.yaml
@@ -123,6 +123,7 @@ etcd:
 #    read: 5000                    # default 5000ms
 
 plugins:                          # plugin list
+  - api-breaker
   - example-plugin
   - limit-req
   - limit-count

Reply via email to