chore(jmx): remove jmx port (#140)

cryostatio · May 21, 2024 · 065060c · 065060c
1 parent 9a36228
commit 065060c
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 12 deletions.
diff --git a/charts/cryostat/README.md b/charts/cryostat/README.md
@@ -13,7 +13,6 @@ A Helm chart for deploying [Cryostat](https://cryostat.io/) on Kubernetes and Op
 | `core.image.tag`                               | Tag for the main Cryostat container image                                                                                                                                                                                                                                                                      | `3.0.0-snapshot`            |
 | `core.service.type`                            | Type of Service to create for the Cryostat application                                                                                                                                                                                                                                                         | `ClusterIP`                 |
 | `core.service.httpPort`                        | Port number to expose on the Service for Cryostat's HTTP server                                                                                                                                                                                                                                                | `8181`                      |
-| `core.service.jmxPort`                         | Port number to expose on the Service for remote JMX connections to Cryostat                                                                                                                                                                                                                                    | `9091`                      |
 | `core.sslProxied`                              | Enables SSL Proxied Environment Variables, useful when you are offloading SSL/TLS at External Loadbalancer instead of Ingress                                                                                                                                                                                  | `false`                     |
 | `core.ingress.enabled`                         | Whether to create an Ingress object for the Cryostat service                                                                                                                                                                                                                                                   | `false`                     |
 | `core.ingress.className`                       | Ingress class name for the Cryostat application Ingress                                                                                                                                                                                                                                                        | `""`                        |

diff --git a/charts/cryostat/templates/service.yaml b/charts/cryostat/templates/service.yaml
@@ -21,9 +21,5 @@ spec:
       targetPort: 8443
       protocol: TCP
       name: cryostat-https
-    - port: {{ .Values.core.service.jmxPort }}
-      targetPort: 9091
-      protocol: TCP
-      name: jfr-jmx
   selector:
     {{- include "cryostat.selectorLabels" $ | nindent 4 }}
diff --git a/charts/cryostat/values.schema.json b/charts/cryostat/values.schema.json
@@ -37,11 +37,6 @@
                             "type": "number",
                             "description": "Port number to expose on the Service for Cryostat's HTTP server",
                             "default": 8181
-                        },
-                        "jmxPort": {
-                            "type": "number",
-                            "description": "Port number to expose on the Service for remote JMX connections to Cryostat",
-                            "default": 9091
                         }
                     }
                 },

diff --git a/charts/cryostat/values.yaml b/charts/cryostat/values.yaml
@@ -13,8 +13,6 @@ core:
     type: ClusterIP
     ## @param core.service.httpPort Port number to expose on the Service for Cryostat's HTTP server
     httpPort: 8181
-    ## @param core.service.jmxPort Port number to expose on the Service for remote JMX connections to Cryostat
-    jmxPort: 9091
   ## @param core.sslProxied Enables SSL Proxied Environment Variables, useful when you are offloading SSL/TLS at External Loadbalancer instead of Ingress
   sslProxied: false
   ingress: