Update Submariner dependencies to v0.17.2

Signed-off-by: Automated Release <release@submariner.io>

.github/dependabot.yml

@@ -9,6 +9,15 @@ updates:
       github-actions:
         patterns:
           - "*"
+  - package-ecosystem: github-actions
+    directory: '/'
+    target-branch: "release-0.14"
+    schedule:
+      interval: monthly
+    groups:
+      github-actions:
+        patterns:
+          - "*"
   - package-ecosystem: github-actions
     directory: '/'
     target-branch: "release-0.15"
@@ -27,21 +36,3 @@ updates:
       github-actions:
         patterns:
           - "*"
-  - package-ecosystem: github-actions
-    directory: '/'
-    target-branch: "release-0.17"
-    schedule:
-      interval: monthly
-    groups:
-      github-actions:
-        patterns:
-          - "*"
-  - package-ecosystem: github-actions
-    directory: '/'
-    target-branch: "release-0.18"
-    schedule:
-      interval: monthly
-    groups:
-      github-actions:
-        patterns:
-          - "*"

.github/workflows/branch.yml

@@ -11,6 +11,6 @@ jobs:
     name: PR targets branch
     runs-on: ubuntu-latest
     steps:
-      - name: Check that the PR targets release-0.19
+      - name: Check that the PR targets release-0.17
-        if: ${{ github.base_ref != 'release-0.19' }}
+        if: ${{ github.base_ref != 'release-0.17' }}
         run: exit 1

.github/workflows/e2e-full.yml

@@ -29,11 +29,11 @@ jobs:
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
 
       - name: Run E2E deployment and tests
-        uses: submariner-io/shipyard/gh-actions/e2e@release-0.19
+        uses: submariner-io/shipyard/gh-actions/e2e@release-0.17
         with:
           k8s_version: ${{ matrix.k8s_version }}
           using: ${{ matrix.cable_driver }} ${{ matrix.globalnet }} ${{ matrix.lighthouse }}
 
       - name: Post mortem
         if: failure()
-        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.19
+        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.17

.github/workflows/e2e.yml

@@ -16,8 +16,8 @@ jobs:
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
 
       - name: Run E2E deployment and tests
-        uses: submariner-io/shipyard/gh-actions/e2e@release-0.19
+        uses: submariner-io/shipyard/gh-actions/e2e@release-0.17
 
       - name: Post mortem
         if: failure()
-        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.19
+        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.17

.github/workflows/flake_finder.yml

@@ -24,10 +24,10 @@ jobs:
         uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332
 
       - name: Run E2E deployment and tests
-        uses: submariner-io/shipyard/gh-actions/e2e@release-0.19
+        uses: submariner-io/shipyard/gh-actions/e2e@release-0.17
         with:
           using: ${{ matrix.cable_driver }} ${{ matrix.globalnet }} ${{ matrix.lighthouse }}
 
       - name: Post mortem
         if: failure()
-        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.19
+        uses: submariner-io/shipyard/gh-actions/post-mortem@release-0.17

.github/workflows/linting.yml

@@ -46,7 +46,7 @@ jobs:
           version: v3.6.0
 
       - name: Set up Python
-        uses: actions/setup-python@f677139bbe7f9c59b41e40162b753c062f5d49a3
+        uses: actions/setup-python@82c7e631bb3cdc910f68e0081d67478d79c6982d
         with:
           python-version: '3.x'
 

.github/workflows/release.yml

@@ -4,7 +4,7 @@ name: Release Charts
 on:
   push:
     branches:
-      - release-0.19
+      - release-0.17
 
 permissions:
   contents: write

Makefile

@@ -1,4 +1,4 @@
-BASE_BRANCH ?= release-0.19
+BASE_BRANCH ?= release-0.17
 export BASE_BRANCH
 export HELM_REPO_LOCATION=./helm_repo
 
@@ -17,7 +17,7 @@ endif
 export DEPLOYTOOL = helm
 GH_URL=https://submariner-io.github.io/submariner-charts/charts
 CHARTS_DIR=charts
-CHARTS_VERSION=0.19.2
+CHARTS_VERSION=0.17.2
 HELM_DOCS_VERSION=0.15.0
 REPO_URL=$(shell git config remote.origin.url)
 

submariner-operator/README.md

@@ -58,8 +58,6 @@ Submariner enables direct networking between Pods and Services in different Kube
 | submariner.coreDNSCustomConfig | object | `{}` |  |
 | submariner.debug | bool | `false` |  |
 | submariner.globalCidr | string | `""` |  |
-| submariner.clustersetIpCidr | string | `""` |  |
-| submariner.clustersetIpEnabled | bool | `false` |  |
 | submariner.healthcheckEnabled | bool | `true` |  |
 | submariner.images.repository | string | `"quay.io/submariner"` |  |
 | submariner.images.tag | string | `"0.14.0"` |  |

submariner-operator/crds/crd.yaml

@@ -3,7 +3,8 @@ apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
 metadata:
   annotations:
-    controller-gen.kubebuilder.io/version: v0.12.1
+    controller-gen.kubebuilder.io/version: v0.4.1
+  creationTimestamp: null
   name: submariners.submariner.io
 spec:
   group: submariner.io
@@ -17,7 +18,7 @@ spec:
   - name: v1alpha1
     schema:
       openAPIV3Schema:
-          description: Submariner is the Schema for the submariners API.
+        description: Submariner is the Schema for the submariners API
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -32,72 +33,43 @@ spec:
           metadata:
             type: object
           spec:
-              description: SubmarinerSpec defines the desired state of Submariner.
+            description: SubmarinerSpec defines the desired state of Submariner
             properties:
-                airGappedDeployment:
-                  type: boolean
               broker:
-                  description: Type of broker (must be "k8s").
                 type: string
               brokerK8sApiServer:
-                  description: The broker API URL.
                 type: string
               brokerK8sApiServerToken:
-                  description: The broker API Token.
                 type: string
               brokerK8sCA:
-                  description: The broker certificate authority.
                 type: string
               brokerK8sInsecure:
                 type: boolean
               brokerK8sRemoteNamespace:
-                  description: The Broker namespace.
-                  type: string
-                brokerK8sSecret:
                 type: string
               cableDriver:
-                  description: Cable driver implementation - any of [libreswan, wireguard,
-                    vxlan].
                 type: string
               ceIPSecDebug:
-                  description: Enable logging IPsec debugging information.
                 type: boolean
               ceIPSecForceUDPEncaps:
-                  description: Force UDP encapsulation for IPsec.
                 type: boolean
               ceIPSecIKEPort:
-                  description: The IPsec IKE port (500 usually).
                 type: integer
               ceIPSecNATTPort:
-                  description: The IPsec NAT traversal port (4500 usually).
                 type: integer
               ceIPSecPSK:
-                  description: The IPsec Pre-Shared Key which must be identical in all
-                    route agents across the cluster.
-                  type: string
-                ceIPSecPSKSecret:
                 type: string
               ceIPSecPreferredServer:
-                  description: Enable this cluster as a preferred server for data-plane
-                    connections.
                 type: boolean
               clusterCIDR:
-                  description: The cluster CIDR.
                 type: string
               clusterID:
-                  description: The cluster ID used to identify the tunnels.
-                  type: string
-                clustersetIPCIDR:
-                  description:  ClustersetIP CIDR for allocating ClustersetIPs to exported
-                    services.
                 type: string
               colorCodes:
                 type: string
               connectionHealthCheck:
-                  description: The gateway connection health check.
                 properties:
                   enabled:
-                      description: Enable the connection health check.
                     type: boolean
                   intervalSeconds:
                     description: The interval at which health check pings are sent.
@@ -110,111 +82,47 @@ spec:
                     type: integer
                 type: object
               coreDNSCustomConfig:
-                  description: Name of the custom CoreDNS configmap to configure forwarding
-                    to Lighthouse. It should be in <namespace>/<name> format where <namespace>
-                    is optional and defaults to kube-system.
                 properties:
                   configMapName:
-                      description: Name of the custom CoreDNS configmap.
                     type: string
                   namespace:
-                      description: Namespace of the custom CoreDNS configmap.
                     type: string
                 type: object
               customDomains:
-                  description: List of domains to use for multi-cluster service discovery.
                 items:
                   type: string
                 type: array
                 x-kubernetes-list-type: set
               debug:
-                  description: Enable operator debugging.
                 type: boolean
               globalCIDR:
-                  description: The Global CIDR super-net range for allocating GlobalCIDRs
-                    to each cluster.
                 type: string
-                haltOnCertificateError:
-                  description: Halt on certificate error (so the pod gets restarted).
-                  type: boolean
               imageOverrides:
                 additionalProperties:
                   type: string
-                  description: Override component images.
                 type: object
               loadBalancerEnabled:
-                  description: Enable automatic Load Balancer in front of the gateways.
                 type: boolean
               namespace:
-                  description: The namespace in which to deploy the submariner operator.
                 type: string
               natEnabled:
-                  description: Enable NAT between clusters.
                 type: boolean
-                nodeSelector:
-                  additionalProperties:
-                    type: string
-                  type: object
               repository:
-                  description: The image repository.
                 type: string
               serviceCIDR:
-                  description: The service CIDR.
                 type: string
               serviceDiscoveryEnabled:
-                  description: Enable support for Service Discovery (Lighthouse).
                 type: boolean
-                clustersetIPEnabled:
-                  description: Enable ClustersetIP default for services exported on this
-                    cluster.
-                  type: boolean
-                tolerations:
-                  items:
-                    description: The pod this Toleration is attached to tolerates any
-                      taint that matches the triple <key,value,effect> using the matching
-                      operator <operator>.
-                    properties:
-                      effect:
-                        description: Effect indicates the taint effect to match. Empty
-                          means match all taint effects. When specified, allowed values
-                          are NoSchedule, PreferNoSchedule and NoExecute.
-                        type: string
-                      key:
-                        description: Key is the taint key that the toleration applies
-                          to. Empty means match all taint keys. If the key is empty,
-                          operator must be Exists; this combination means to match all
-                          values and all keys.
-                        type: string
-                      operator:
-                        description: Operator represents a key's relationship to the
-                          value. Valid operators are Exists and Equal. Defaults to Equal.
-                          Exists is equivalent to wildcard for value, so that a pod
-                          can tolerate all taints of a particular category.
-                        type: string
-                      tolerationSeconds:
-                        description: TolerationSeconds represents the period of time
-                          the toleration (which must be of effect NoExecute, otherwise
-                          this field is ignored) tolerates the taint. By default, it
-                          is not set, which means tolerate the taint forever (do not
-                          evict). Zero and negative values will be treated as 0 (evict
-                          immediately) by the system.
-                        format: int64
-                        type: integer
-                      value:
-                        description: Value is the taint value the toleration matches
-                          to. If the operator is Exists, the value should be empty,
-                          otherwise just a regular string.
-                        type: string
-                    type: object
-                  type: array
               version:
-                  description: The image tag.
                 type: string
             required:
             - broker
             - brokerK8sApiServer
+            - brokerK8sApiServerToken
+            - brokerK8sCA
             - brokerK8sRemoteNamespace
             - ceIPSecDebug
+            - ceIPSecPSK
             - clusterCIDR
             - clusterID
             - debug
@@ -223,23 +131,15 @@ spec:
             - serviceCIDR
             type: object
           status:
-              description: SubmarinerStatus defines the observed state of Submariner.
+            description: SubmarinerStatus defines the observed state of Submariner
             properties:
-                airGappedDeployment:
-                  type: boolean
               clusterCIDR:
-                  description: The current cluster CIDR.
                 type: string
               clusterID:
-                  description: The current cluster ID.
-                  type: string
-                clustersetIPCIDR:
-                  description: The current clustersetIP CIDR.
                 type: string
               colorCodes:
                 type: string
               deploymentInfo:
-                  description: Information about the deployment.
                 properties:
                   cloudProvider:
                     type: string
@@ -251,7 +151,6 @@ spec:
                     type: string
                 type: object
               gatewayDaemonSetStatus:
-                  description: The status of the gateway DaemonSet.
                 properties:
                   lastResourceVersion:
                     type: string
@@ -275,7 +174,7 @@ spec:
                           description: Details about a terminated container
                           properties:
                             containerID:
-                                description: Container's ID in the format '<type>://<container_id>'
+                              description: Container's ID in the format 'docker://<container_id>'
                               type: string
                             exitCode:
                               description: Exit status from the last termination of
@@ -388,9 +287,9 @@ spec:
                         format: int32
                         type: integer
                       numberReady:
-                          description: numberReady is the number of nodes that should
+                        description: The number of nodes that should be running the
-                            be running the daemon pod and have one or more of the daemon
+                          daemon pod and have one or more of the daemon pod running
-                            pod running with a Ready Condition.
+                          and ready.
                         format: int32
                         type: integer
                       numberUnavailable:
@@ -419,7 +318,6 @@ spec:
                 - mismatchedContainerImages
                 type: object
               gateways:
-                  description: Status of the gateways in the cluster.
                 items:
                   properties:
                     connections:
@@ -546,10 +444,8 @@ spec:
                   type: object
                 type: array
               globalCIDR:
-                  description: The current global CIDR.
                 type: string
               globalnetDaemonSetStatus:
-                  description: The status of the Globalnet DaemonSet.
                 properties:
                   lastResourceVersion:
                     type: string
@@ -573,7 +469,7 @@ spec:
                           description: Details about a terminated container
                           properties:
                             containerID:
-                                description: Container's ID in the format '<type>://<container_id>'
+                              description: Container's ID in the format 'docker://<container_id>'
                               type: string
                             exitCode:
                               description: Exit status from the last termination of
@@ -686,9 +582,9 @@ spec:
                         format: int32
                         type: integer
                       numberReady:
-                          description: numberReady is the number of nodes that should
+                        description: The number of nodes that should be running the
-                            be running the daemon pod and have one or more of the daemon
+                          daemon pod and have one or more of the daemon pod running
-                            pod running with a Ready Condition.
+                          and ready.
                         format: int32
                         type: integer
                       numberUnavailable:
@@ -717,7 +613,6 @@ spec:
                 - mismatchedContainerImages
                 type: object
               loadBalancerStatus:
-                  description: The status of the load balancer DaemonSet.
                 properties:
                   status:
                     description: LoadBalancerStatus represents the status of a load-balancer.
@@ -739,53 +634,15 @@ spec:
                               description: IP is set for load-balancer ingress points
                                 that are IP based (typically GCE or OpenStack load-balancers)
                               type: string
-                              ports:
-                                description: Ports is a list of records of service ports
-                                  If used, every port defined in the service should
-                                  have an entry in it
-                                items:
-                                  properties:
-                                    error:
-                                      description: 'Error is to record the problem with
-                                      the service port The format of the error shall
-                                      comply with the following rules: - built-in
-                                      error values shall be specified in this file
-                                      and those shall use CamelCase names - cloud
-                                      provider specific error values must have names
-                                      that comply with the format foo.example.com/CamelCase.
-                                      --- The regex it matches is (dns1123SubdomainFmt/)?(qualifiedNameFmt)'
-                                      maxLength: 316
-                                      pattern: ^([a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*/)?(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])$
-                                      type: string
-                                    port:
-                                      description: Port is the port number of the service
-                                        port of which status is recorded here
-                                      format: int32
-                                      type: integer
-                                    protocol:
-                                      default: TCP
-                                      description: 'Protocol is the protocol of the
-                                      service port of which status is recorded here
-                                      The supported values are: "TCP", "UDP", "SCTP"'
-                                      type: string
-                                  required:
-                                    - port
-                                    - protocol
-                                  type: object
-                                type: array
-                                x-kubernetes-list-type: atomic
                           type: object
                         type: array
                     type: object
                 type: object
               natEnabled:
-                  description: The current NAT status.
                 type: boolean
               networkPlugin:
-                  description: The current network plugin.
                 type: string
               routeAgentDaemonSetStatus:
-                  description: The status of the route agent DaemonSet.
                 properties:
                   lastResourceVersion:
                     type: string
@@ -809,7 +666,7 @@ spec:
                           description: Details about a terminated container
                           properties:
                             containerID:
-                                description: Container's ID in the format '<type>://<container_id>'
+                              description: Container's ID in the format 'docker://<container_id>'
                               type: string
                             exitCode:
                               description: Exit status from the last termination of
@@ -922,9 +779,9 @@ spec:
                         format: int32
                         type: integer
                       numberReady:
-                          description: numberReady is the number of nodes that should
+                        description: The number of nodes that should be running the
-                            be running the daemon pod and have one or more of the daemon
+                          daemon pod and have one or more of the daemon pod running
-                            pod running with a Ready Condition.
+                          and ready.
                         format: int32
                         type: integer
                       numberUnavailable:
@@ -953,11 +810,6 @@ spec:
                 - mismatchedContainerImages
                 type: object
               serviceCIDR:
-                  description: The current service CIDR.
-                  type: string
-                version:
-                  description: The image version in use by the various Submariner DaemonSets
-                    and Deployments.
                 type: string
             required:
             - clusterID
@@ -968,12 +820,19 @@ spec:
     storage: true
     subresources:
       status: {}
+status:
+  acceptedNames:
+    kind: ""
+    plural: ""
+  conditions: []
+  storedVersions: []
 ---
 apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
 metadata:
   annotations:
-    controller-gen.kubebuilder.io/version: v0.12.1
+    controller-gen.kubebuilder.io/version: v0.4.1
+  creationTimestamp: null
   name: servicediscoveries.submariner.io
 spec:
   group: submariner.io
@@ -987,7 +846,7 @@ spec:
   - name: v1alpha1
     schema:
       openAPIV3Schema:
-          description: ServiceDiscovery is the Schema for the servicediscoveries API.
+        description: ServiceDiscovery is the Schema for the servicediscoveries API
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -1002,7 +861,7 @@ spec:
           metadata:
             type: object
           spec:
-              description: ServiceDiscoverySpec defines the desired state of ServiceDiscovery.
+            description: ServiceDiscoverySpec defines the desired state of ServiceDiscovery
             properties:
               brokerK8sApiServer:
                 type: string
@@ -1014,19 +873,13 @@ spec:
                 type: boolean
               brokerK8sRemoteNamespace:
                 type: string
-                brokerK8sSecret:
-                  type: string
               clusterID:
                 type: string
-                clustersetIPCIDR:
-                  type: string
               coreDNSCustomConfig:
                 properties:
                   configMapName:
-                      description: Name of the custom CoreDNS configmap.
                     type: string
                   namespace:
-                      description: Namespace of the custom CoreDNS configmap.
                     type: string
                 type: object
               customDomains:
@@ -1038,72 +891,27 @@ spec:
                 type: boolean
               globalnetEnabled:
                 type: boolean
-                haltOnCertificateError:
-                  type: boolean
-                clustersetIPEnabled:
-                  type: boolean
               imageOverrides:
                 additionalProperties:
                   type: string
                 type: object
               namespace:
                 type: string
-                nodeSelector:
-                  additionalProperties:
-                    type: string
-                  type: object
               repository:
                 type: string
-                tolerations:
-                  items:
-                    description: The pod this Toleration is attached to tolerates any
-                      taint that matches the triple <key,value,effect> using the matching
-                      operator <operator>.
-                    properties:
-                      effect:
-                        description: Effect indicates the taint effect to match. Empty
-                          means match all taint effects. When specified, allowed values
-                          are NoSchedule, PreferNoSchedule and NoExecute.
-                        type: string
-                      key:
-                        description: Key is the taint key that the toleration applies
-                          to. Empty means match all taint keys. If the key is empty,
-                          operator must be Exists; this combination means to match all
-                          values and all keys.
-                        type: string
-                      operator:
-                        description: Operator represents a key's relationship to the
-                          value. Valid operators are Exists and Equal. Defaults to Equal.
-                          Exists is equivalent to wildcard for value, so that a pod
-                          can tolerate all taints of a particular category.
-                        type: string
-                      tolerationSeconds:
-                        description: TolerationSeconds represents the period of time
-                          the toleration (which must be of effect NoExecute, otherwise
-                          this field is ignored) tolerates the taint. By default, it
-                          is not set, which means tolerate the taint forever (do not
-                          evict). Zero and negative values will be treated as 0 (evict
-                          immediately) by the system.
-                        format: int64
-                        type: integer
-                      value:
-                        description: Value is the taint value the toleration matches
-                          to. If the operator is Exists, the value should be empty,
-                          otherwise just a regular string.
-                        type: string
-                    type: object
-                  type: array
               version:
                 type: string
             required:
             - brokerK8sApiServer
+            - brokerK8sApiServerToken
+            - brokerK8sCA
             - brokerK8sRemoteNamespace
             - clusterID
             - debug
             - namespace
             type: object
           status:
-              description: ServiceDiscoveryStatus defines the observed state of ServiceDiscovery.
+            description: ServiceDiscoveryStatus defines the observed state of ServiceDiscovery
             properties:
               deploymentInfo:
                 properties:
@@ -1122,6 +930,12 @@ spec:
     storage: true
     subresources:
       status: {}
+status:
+  acceptedNames:
+    kind: ""
+    plural: ""
+  conditions: []
+  storedVersions: []
 ---
 apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition

submariner-operator/templates/rbac.yaml

@@ -18,6 +18,16 @@ rules:
       - get
       - create
       - update
+  - apiGroups:
+      - ""
+    resources:
+      # For syncing Secrets from the broker
+      - secrets
+    verbs:
+      - get
+      - create
+      - update
+      - delete
   - apiGroups:
       - ""
     resources:
@@ -226,15 +236,6 @@ metadata:
     chart: {{ template "submariner.chart" . }}
     app: {{ template "submariner.name" . }}
 rules:
-  - apiGroups:
-      - submariner.io
-    resources:
-      - gateways
-    verbs:
-      - get
-      - list
-      - watch
-      - update
   - apiGroups:
       - coordination.k8s.io
     resources:
@@ -477,6 +478,7 @@ rules:
       - get
       - list
       - watch
+      - update
   - apiGroups:
       - projectcalico.org
     resources:
@@ -517,6 +519,15 @@ metadata:
     chart: {{ template "submariner.chart" . }}
     app: {{ template "submariner.name" . }}
 rules:
+  - apiGroups:
+      - ""
+    resources:
+      - nodes
+    verbs:
+      - get
+      - list
+      - watch
+      - update
   - apiGroups:
       - ""
     resources:

submariner-operator/templates/submariner.yaml

@@ -52,8 +52,6 @@ spec:
 {{- end }}
   serviceCIDR: "{{ .Values.submariner.serviceCidr }}"
   globalCIDR: "{{ .Values.submariner.globalCidr }}"
-  clustersetIPCIDR: "{{ .Values.submariner.clustersetIpCidr }}"
-  clustersetIPEnabled: {{ .Values.submariner.clustersetIpEnabled }}
   serviceDiscoveryEnabled: {{ .Values.submariner.serviceDiscovery }}
   cableDriver: {{ .Values.submariner.cableDriver }}
   connectionHealthCheck:

submariner-operator/values.yaml

@@ -5,8 +5,6 @@ submariner:
   clusterCidr: ""
   serviceCidr: ""
   globalCidr: ""
-  clustersetIpCidr: ""
-  clustersetIpEnabled: false
   loadBalancerEnabled: false
   natEnabled: false
   colorCodes: blue