FoundationDB / fdb-kubernetes-operator

A kubernetes operator for FoundationDB
Apache License 2.0
241 stars 82 forks source link

Process Group ID should be increasing until a upper limit is reached #2071

Closed johscheuer closed 3 months ago

johscheuer commented 3 months ago

What would you like to be added/changed?

The current logic will take the first free Process Group ID. The intention was that the process group ID space was used efficiently. The issue with that approach is that if a process group fails to come up, we can see cases where a new process group gets the same name as the old one, even thought those are actually different process groups. This can be confusing when debugging issues. A better approach would be to increase the process group IDs until we hit an upper limit and then start from the beginning.