-
Notifications
You must be signed in to change notification settings - Fork 14
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
[issue 114] - Adding the PostgreSql template provisioner (for legacy … #115
Conversation
dcb4b23
to
c459236
Compare
/retest |
1 similar comment
/retest |
Hi @marekkopecky - the CI checks can be taken as successful, and the PR is ready for review. |
...a/org/jboss/intersmash/tools/provision/openshift/PostgreSQLTemplateOpenShiftProvisioner.java
Show resolved
Hide resolved
...boss/intersmash/tools/provision/openshift/PostgreSQLTemplateOpenShiftProvisionerFactory.java
Show resolved
Hide resolved
Hi @marekkopecky - this CI check run is good. The failure in the prod job is expected and the one in the community job is a randomly intermittent failure. |
5037f6b
to
3316aae
Compare
@fabiobrz: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
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.
Thank you, LGTM.
It seems that there is no option to merge this PR, I see:
Can you please double-check? |
Thanks @marekkopecky - I checked and will try to update the settings in the future. For now I'll merge this since you approved. Thanks! |
Description
Fix #114
Type of change
test, version modification, documentation, etc.)
Checklist