Steven Hawkins
Steven Hawkins
@ahus1 I'm fine with removing the defaults around spread constraints and moving that to a topic for the scaling guide.
> No, please add a default best-effort spread. I'm just against making the default settings dependent on the cache configuration. The default spread is just an affinity for the same...
@vmuzikar added the docs, should be fully ready for review now.
@ryanemerson @ahus1 @vmuzikar in the docs should we call out specifically recommending an affinity for the same zone as the database?
> @shawkins Just last minor thing about the example formatting, otherwise LGTM Should be good now.
@vmuzikar @Pepo48 @mabartos here are some follow-up thoughts / questions based upon the implementation so far: - assuming that we are not initially addressing patch support, will the client secret...
Thank you for the feedback @vmuzikar > Secret is not required for public Clients, so the field needs to be optional. Ok, I'll leave it as a non-required SecretKeySelector that...
> So from my perspective yes, let's use the Service directly. If tls is configured do we need to initially bother with obtaining the server cert, or can that be...
@mposolda @vmuzikar I'm looking at setting up the client the operator will use for accessing each cluster. The recent change from keycloak-admin-client to keycloak-admin-client-tests seems to imply that keycloak-admin-client is...
> I also wonder if we should add a full integration test to cover https://github.com/keycloak/keycloak/issues/34095 I'll leave that up to you. I would certainly put something like that in place...