-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
4.20.merge #10013
4.20.merge #10013
Conversation
@vishesh92 @JoaoJandre @weizhouapache @rohityadavcloud @GutoVeronezi @rp- @nvazquez @Pearl1594 @bernardodemarco @sureshanaparti @FelipeM525 @RodrigoDLopez @winterhazel this is a days worth of merging and fixing test failures. I think it is ok, and usually I just push but in this case I think a bunch of review is in order. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 4.20 #10013 +/- ##
============================================
+ Coverage 15.80% 15.82% +0.02%
- Complexity 12586 12623 +37
============================================
Files 5627 5630 +3
Lines 492343 492778 +435
Branches 59694 59745 +51
============================================
+ Hits 77828 77996 +168
- Misses 405992 406248 +256
- Partials 8523 8534 +11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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 did only go trough kvm and linstor changes
@DaanHoogland |
I am not sure but I think we have to redo once we found all the issues. (already newer merges on 4.20 |
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.
Changes related to #9966, #9759, and #9927 look good. @DaanHoogland, I couldn’t spot the changes for #9949, though.
I will redo this to see if I can do better, if need be there will be some cherry-picking involved... |
c71c0b6
to
52e7b41
Compare
* 4.20: VR: apply iptables rules when add/remove static routes (#10064) Certificate and VM hostname validation improvements (#10051) set ulimit for server according to redhat spec (#10040) kvm-storage: provide isVMMigrate information to storage plugins (#10093) Allow config drive deletion of migrated VM, on host maintenance (#10045) linstor: improve heartbeat check with also asking linstor (#10105) server: simplify role change validation (#9173) UI: create VPC network offering with conserve mode (#10082) server: fix typo removeaccessvpn in VirtualRouterElement (#10086) UI: remove duplicated Instance Name in Public IP details page (#10087) UI: Fixes in the Usage UI (#10000) SAML2: add cookie with HttpOnly too #10013 (#10047) ui: Allow font-awesome icon usage and optimise icon size inconsistency (#9744)
Description
This PR...
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?