Closed johndelcastillo closed 1 year ago
Hi, nice to see some progress. When we can expect release?:)
Largely looks good to me, however I think a bunch of python cache files where checked in and should get cleaned up unless there is a good reason to have them in repo?
Removed the python guff, not needed.
Thanks for the hard work! It could really have been a 0.10.0 release.
HI @johndelcastillo Thank you for your hard work. I get the following error and 0.9.12 does not seem to work for Cassandra 4.1
WARN [cassandra-exporter-harvester-defer-0] 2023-05-26 11:51:37,289 Harvester.java:191 - Failed to register collector for MBean org.apache.cassandra .metrics:type=Connection,scope=xx.xx.xx.xx_7001,name=Timeouts. java.lang.IllegalArgumentException: 'xx.xx.xx.xx_7001' is not an IP string literal. at com.google.common.net.InetAddresses.formatIllegalArgumentException(InetAddresses.java:987) at com.google.common.net.InetAddresses.forString(InetAddresses.java:143) at com.zegelin.cassandra.exporter.MetadataFactory.endpointLabels(MetadataFactory.java:71) at com.zegelin.cassandra.exporter.FactoriesSupplier.lambda$connectionMetric$17(FactoriesSupplier.java:474) at com.zegelin.cassandra.exporter.FactoriesSupplier$FactoryBuilder.lambda$withLabelMaker$0(FactoriesSupplier.java:73) at com.zegelin.cassandra.exporter.FactoriesSupplier$FactoryBuilder.lambda$build$1(FactoriesSupplier.java:99) at com.zegelin.cassandra.exporter.Harvester.lambda$registerMBean$0(Harvester.java:174) at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
I am grateful for any help
Add support for Cassandra 4.0+ clusters
Tested agent and standalone, locally using CCM running Cassandra 4.1.0 and Prometheus 2.42.0.
Includes a WIP integration test framework developed by @zegelin
Cassandra 4 changes largely inspired from https://github.com/instaclustr/cassandra-exporter/pull/102