New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial bootstrap admin #10661
Initial bootstrap admin #10661
Conversation
3a49a20
to
fb81957
Compare
operator/src/main/java/org/keycloak/operator/v2alpha1/KeycloakAdminSecret.java
Outdated
Show resolved
Hide resolved
operator/src/main/java/org/keycloak/operator/v2alpha1/KeycloakAdminSecret.java
Outdated
Show resolved
Hide resolved
operator/src/main/java/org/keycloak/operator/v2alpha1/KeycloakAdminSecret.java
Outdated
Show resolved
Hide resolved
operator/src/test/java/org/keycloak/operator/KeycloakDeploymentE2EIT.java
Show resolved
Hide resolved
operator/src/test/java/org/keycloak/operator/KeycloakDeploymentE2EIT.java
Outdated
Show resolved
Hide resolved
786d67b
to
05c7880
Compare
@vmuzikar ready for next round! 🎉 |
05c7880
to
29c31fe
Compare
AtomicReference<String> adminUsername = new AtomicReference<>(); | ||
AtomicReference<String> adminPassword = new AtomicReference<>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the AtomicReference
really necessary? Awaitility doesn't use threads AFAIK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are using those values in lambda
s and the IDE screams if they are not final
or AtomicRef
s
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right. The "effectively final" rule...
Resolves #9843
NDR: This is a stop gap solution.