remove unused function parameter

Author: razvan

rust/operator-binary/src/crd/mod.rs

@@ -492,7 +492,7 @@ impl v1alpha1::HbaseCluster {
     /// Returns required port name and port number tuples depending on the role.
     /// Hbase versions 2.6.* will have two ports for each role. The metrics are available over the
     /// UI port.
-    pub fn ports(&self, role: &HbaseRole, _hbase_version: &str) -> Vec<(String, u16)> {
+    pub fn ports(&self, role: &HbaseRole) -> Vec<(String, u16)> {
         match role {
             HbaseRole::Master => vec![
                 ("master".to_string(), HBASE_MASTER_PORT),

rust/operator-binary/src/hbase_controller.rs

@@ -513,7 +513,7 @@ pub fn build_region_server_role_service(
         .server_role_service_name()
         .context(GlobalServiceNameNotFoundSnafu)?;
     let ports = hbase
-        .ports(&role, &resolved_product_image.product_version)
+        .ports(&role)
         .into_iter()
         .map(|(name, value)| ServicePort {
             name: Some(name),
@@ -706,7 +706,7 @@ fn build_rolegroup_service(
     resolved_product_image: &ResolvedProductImage,
 ) -> Result<Service> {
     let ports = hbase
-        .ports(hbase_role, &resolved_product_image.product_version)
+        .ports(hbase_role)
         .into_iter()
         .map(|(name, value)| ServicePort {
             name: Some(name),
@@ -772,7 +772,7 @@ fn build_rolegroup_statefulset(
     let hbase_version = &resolved_product_image.app_version_label;
 
     let ports = hbase
-        .ports(hbase_role, &resolved_product_image.product_version)
+        .ports(hbase_role)
         .into_iter()
         .map(|(name, value)| ContainerPort {
             name: Some(name),