[ST] Remove unused methods from the KafkaNodePoolResource class

[im-konge]

Type of change

• Refactoring

Description

Inside KafkaNodePoolResource are methods that are not used anywhere in STs. Plus, in case of getLabelSelector(final String poolName, final ProcessRoles processRoles) it's usage little bit confuse anyone when watching rolling update of the Pods, as we can see something similar to this in the log of the test execution:

2024-02-06 12:29:39 [main] INFO  [RollingUpdateUtils:79] Waiting for component matching LabelSelector(matchExpressions=[], matchLabels={strimzi.io/pool-name=kafka, strimzi.io/kind=Kafka, strimzi.io/broker-role=true}, additionalProperties={}) -> co-namespace/null-kafka rolling update
2024-02-06 12:30:58 [main] INFO  [RollingUpdateUtils:90] Component matching LabelSelector(matchExpressions=[], matchLabels={strimzi.io/pool-name=kafka, strimzi.io/kind=Kafka, strimzi.io/broker-role=true}, additionalProperties={}) -> co-namespace/null-kafka has been successfully rolled
2024-02-06 12:30:59 [main] INFO  [RollingUpdateUtils:102] Waiting for 3 Pod(s) of co-namespace/null-kafka to be ready

Checklist

• Make sure all tests pass

[henryZrncik]

Make sense. Thanks for PR!

[im-konge]

/azp run upgrade

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).