michaeljmarshall opened a new issue, #19866: URL: https://github.com/apache/pulsar/issues/19866
### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Motivation With https://github.com/apache/pulsar/pull/19151 and https://github.com/apache/pulsar/pull/19762, we verify that cluster urls are valid. It seems like we could also verify that there is at least one `serviceUrl`/`serviceUrlTls` and at least one `brokerServiceUrl`/`brokerServiceUrlTls` set. ### Solution Update the following code to add a check that some of those variables are set: https://github.com/apache/pulsar/blob/e829672b84c15a596574da25205d32ae7b1abf4b/pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java#L412-L430 ### Alternatives _No response_ ### Anything else? _No response_ ### Are you willing to submit a PR? - [ ] I'm willing to submit a PR! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
