Skip to content
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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

EricWittmann
Copy link
Member

No description provided.

jsenko
jsenko previously approved these changes Feb 12, 2025
@jsenko
Copy link
Member

jsenko commented Feb 12, 2025

Note: Rebase onto #5971

@jsenko jsenko force-pushed the feat/operator-java-opts branch from 7f6b14b to 98b34bf Compare February 13, 2025 11:46
@jsenko
Copy link
Member

jsenko commented Feb 13, 2025

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.

@jsenko
Copy link
Member

jsenko commented Feb 13, 2025

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...

@jsenko
Copy link
Member

jsenko commented Feb 13, 2025

Note: Rebase onto #5979 to see if the wf passes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants