-
Notifications
You must be signed in to change notification settings - Fork 272
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
feat(operator): Support for JAVA_OPTS_APPEND when creating app deployment #5933
base: main
Are you sure you want to change the base?
Conversation
989f56c
to
e2ec14a
Compare
Note: Rebase onto #5971 |
7f6b14b
to
98b34bf
Compare
I've seen some condition-related errors as well as a problem with resource discriminators. I'll try to address this in my PR and see if it helps. Otherwise I'm not sure what causes the test failures, but it might be unrelated to this PR. |
yeah we should tune the default timeouts better https://github.com/Apicurio/apicurio-registry/blob/main/operator/controller/src/test/java/io/apicurio/registry/operator/it/ITBase.java#L322 because it seems one size does not fit all. Maybe we should define two different short wait and long wait options... |
Note: Rebase onto #5979 to see if the wf passes. |
42a32ca
to
ff63e3b
Compare
No description provided.