Run pre-commit hooks to fix python test imports (#3221)

* Run pre-commit hooks to fix python test imports

* Add context.minimum_version

Author: lucacome

internal/certmanager/cm_controller.go

@@ -127,16 +127,16 @@ func (c *CmController) processItem(ctx context.Context, key string) error {
 // example, the following Certificate "cert-1" is controlled by the VirtualServer
 // "vs-1":
 //
-//     kind: Certificate
-//     metadata:                                           Note that the owner
-//       namespace: cert-1                                 reference does not
-//       ownerReferences:                                  have a namespace,
-//       - controller: true                                since owner refs
-//         apiVersion: networking.x-k8s.io/v1alpha1        only work inside
-//         kind: VirtualServer                             the same namespace.
-//         name: vs-1
-//         blockOwnerDeletion: true
-//         uid: 7d3897c2-ce27-4144-883a-e1b5f89bd65a
+//	kind: Certificate
+//	metadata:                                           Note that the owner
+//	  namespace: cert-1                                 reference does not
+//	  ownerReferences:                                  have a namespace,
+//	  - controller: true                                since owner refs
+//	    apiVersion: networking.x-k8s.io/v1alpha1        only work inside
+//	    kind: VirtualServer                             the same namespace.
+//	    name: vs-1
+//	    blockOwnerDeletion: true
+//	    uid: 7d3897c2-ce27-4144-883a-e1b5f89bd65a
 func certificateHandler(queue workqueue.RateLimitingInterface) func(obj interface{}) {
 	return func(obj interface{}) {
 		crt, ok := obj.(*cmapi.Certificate)

internal/certmanager/helper.go

@@ -49,25 +49,26 @@ var (
 // apiVersion: k8s.nginx.org/v1
 // kind: VirtualServer
 // spec:
-//   <...>
-//   tls:
-//     <...>
-//     cert-manager:
-//       common-name: example.com
-//       duration: 2160h
-//       renew-before: 1440h
-//       usages: "digital signature,key encipherment"
+//
+//	<...>
+//	tls:
+//	  <...>
+//	  cert-manager:
+//	    common-name: example.com
+//	    duration: 2160h
+//	    renew-before: 1440h
+//	    usages: "digital signature,key encipherment"
 //
 // is mapped to the following Certificate:
 //
-//   kind: Certificate
-//   spec:
-//     commonName: example.com
-//     duration: 2160h
-//     renewBefore: 1440h
-//     usages:
-//       - digital signature
-//       - key encipherment
+//	kind: Certificate
+//	spec:
+//	  commonName: example.com
+//	  duration: 2160h
+//	  renewBefore: 1440h
+//	  usages:
+//	    - digital signature
+//	    - key encipherment
 func translateVsSpec(crt *cmapi.Certificate, vsCmSpec *vsapi.CertManager) error {
 	var errs []string
 	if crt == nil {

internal/certmanager/sync.go

@@ -274,10 +274,10 @@ func certNeedsUpdate(a, b *cmapi.Certificate) bool {
 // Certificate created for the given VirtualServer resource. We look up the following
 // VS TLS Cert-Manager fields:
 //
-//   cluster-issuer
-//   issuer
-//   issuer-kind
-//   issuer-group
+//	cluster-issuer
+//	issuer
+//	issuer-kind
+//	issuer-group
 func issuerForVirtualServer(vs *vsapi.VirtualServer) (name, kind, group string, err error) {
 	var errs []string
 	vsCmSpec := vs.Spec.TLS.CertManager

internal/configs/version1/nginx.ingress.tmpl

@@ -96,7 +96,7 @@ server {
 
 	{{range $location := $server.Locations}}
 	location {{$location.Path}} {
-		set $service "{{$location.ServiceName}}"; 
+		set $service "{{$location.ServiceName}}";
 		{{with $location.MinionIngress}}
 		# location for minion {{$location.MinionIngress.Namespace}}/{{$location.MinionIngress.Name}}
 		set $resource_name "{{$location.MinionIngress.Name}}";

internal/configs/version2/nginx.transportserver.tmpl

@@ -47,4 +47,4 @@ server {
     proxy_next_upstream_timeout {{ $s.ProxyNextUpstreamTimeout }};
     proxy_next_upstream_tries {{ $s.ProxyNextUpstreamTries }};
     {{ end }}
-}
+}

perf-tests/requirements.txt

@@ -1,10 +1,10 @@
-PyYAML==6.0
-requests==2.28.1
+certifi==2022.9.24
+cffi==1.15.1
 kubernetes==25.3.0
+locust==2.13.0
 pytest==7.2.0
-cffi==1.15.1
-certifi==2022.9.24
-urllib3==1.26.12
 pytest-html==3.2.0
 pytest-repeat==0.9.1
-locust==2.13.0
+PyYAML==6.0
+requests==2.28.1
+urllib3==1.26.12

pyproject.toml

@@ -17,6 +17,8 @@ pythonpath = [
     ".",
     "tests/suite",
     "tests/suite/grpc",
+    "tests/suite/fixtures",
+    "tests/suite/utils",
 ]
 addopts = "--tb=native -ra --disable-warnings -x -l --profile -v"
 log_cli = true

tests/conftest.py

@@ -16,7 +16,7 @@
     NS_COUNT,
     NUM_REPLICAS,
 )
-from suite.resources_utils import get_first_pod_name
+from suite.utils.resources_utils import get_first_pod_name
 
 
 def pytest_addoption(parser) -> None:
@@ -111,7 +111,7 @@ def pytest_addoption(parser) -> None:
 
 
 # import fixtures into pytest global namespace
-pytest_plugins = ["suite.fixtures", "suite.ic_fixtures", "suite.custom_resource_fixtures"]
+pytest_plugins = ["suite.fixtures.fixtures", "suite.fixtures.ic_fixtures", "suite.fixtures.custom_resource_fixtures"]
 
 
 def pytest_collection_modifyitems(config, items) -> None:

tests/kind/kind-config-template.yaml

@@ -1,16 +1,16 @@
-PyYAML==6.0
-requests==2.28.1
+certifi==2022.9.24
+cffi==1.15.1
+flaky==3.7.0
 forcediphttpsadapter==1.0.2
+grpcio==1.50.0
+grpcio-tools==1.50.0
 kubernetes==25.3.0
-pytest==7.2.0
-cffi==1.15.1
+mock==4.0.3
+more-itertools==9.0.0
 pyOpenSSL==22.1.0
-certifi==2022.9.24
-urllib3==1.26.12
+pytest==7.2.0
 pytest-html==3.2.0
 pytest-profiling==1.7.0
-more-itertools==9.0.0
-mock==4.0.3
-grpcio==1.50.0
-grpcio-tools==1.50.0
-flaky==3.7.0
+PyYAML==6.0
+requests==2.28.1
+urllib3==1.26.12