-
Notifications
You must be signed in to change notification settings - Fork 28
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
Release 1.4.2.Beta9 #1069
Release 1.4.2.Beta9 #1069
Conversation
run tests |
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.
I need to get a PR in and make a release, so we might as well do it together. Please wait till the end of the day I'll try to create fix ASAP. Thx
74a7d68
to
d41dcf3
Compare
run tests |
@gtroitsk can you analyse OCP failures, please? |
Yes, I will |
@michalvavrik Failures in management module in OC JVM are weird, I need more time to investigate them properly. In Native |
I had mentioned that we still have next warning |
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.
LGTM, let's be careful with merging this into TS. We will need OCP tests.
run tests |
I rerun and analyzed failures, I think most of them goes down to the quay.io network issues. |
My conclusion is there are failures in the examples that we need to fix, but I'm not yet aware of failure that is problem with this release, so I suggest to merge this and run OCP TS before bumping version in FW. |
Summary
Release contains:
Please check the relevant options
run tests
phrase in comment)Checklist: