Remove MINIO_ENDPOINTS env var

[dvaldivia]

Removing the MINIO_ENDPOINTS to avoid having to do a statefulset rolling restart when adding new pools

Signed-off-by: Daniel Valdivia 18384552+dvaldivia@users.noreply.github.com

[harshavardhana]

Not sure what this change is @dvaldivia can you explain?

[dvaldivia]

yes, since we load the endpoints via webhook and ignore the presence of this environment variable then it's not needed, additionally when we add a new pool, we cause the existing pools to rolling restart since we made a change in this environment variable, if we remove it, we don't need to do that rolling restart, the current mc admin service restart will be enough to have the existing pools adopt the new one @harshavardhana

[harshavardhana]

yes, since we load the endpoints via webhook and ignore the presence of this environment variable then it's not needed, additionally when we add a new pool, we cause the existing pools to rolling restart since we made a change in this environment variable, if we remove it, we don't need to do that rolling restart, the current mc admin service restart will be enough to have the existing pools adopt the new one @harshavardhana

I see