prometheus / prometheus

The Prometheus monitoring system and time series database.
https://prometheus.io/
Apache License 2.0
55.06k stars 9.06k forks source link

IONOS cloud pagination #10722

Open roidelapluie opened 2 years ago

roidelapluie commented 2 years ago

The IONOS cloud integration does not use pagination, limiting to 1000 the number of servers discovered per datacenter.

We should integrate pagination in case a user has more than 1000 servers.

oceakun commented 1 month ago

Hi @roidelapluie! I'd like to take this up, if nobody is working on it.

Is there anything else I should know besides going through the IONOS docs?

bboreham commented 3 weeks ago

@oceakun go ahead! I suspect that making tests will be the hard part.

oceakun commented 1 week ago

Hi @bboreham, yes, updating the test took some time.

Though I have raised a PR for this, if you could please review it.

PS: The high diff is mainly due to the large test data image

Also, I'd appreciate your opinion on how I should go about passing the checks as I'm not sure

image