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
systemvm: remove config in /etc/pam.d/systemd-user to fix user@0.service #8048
systemvm: remove config in /etc/pam.d/systemd-user to fix user@0.service #8048
Conversation
the service `user@0.service` fails in system vms and virtual routers This PR removes a change to fix memory leak of SSH connections in the systemvm templates with old linux kernel.
@blueorangutan package |
@weizhouapache a [SF] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 7250 |
Codecov Report
@@ Coverage Diff @@
## 4.18 #8048 +/- ##
============================================
- Coverage 13.07% 13.06% -0.01%
+ Complexity 9107 9106 -1
============================================
Files 2720 2720
Lines 257513 257513
Branches 40151 40151
============================================
- Hits 33658 33649 -9
- Misses 219624 219634 +10
+ Partials 4231 4230 -1 see 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
looks good, the original bug for which this was introduced is fixed. I see no reference to any problems, so it should be good. Not sure if we can test anything other than the SVMs running fine.
@weizhouapache this will require new systemvm template? |
@shwstppr |
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 - didn't test it but changes make sense, esp that the upstream systemd is fixed
@weizhouapache @DaanHoogland @harikrishna-patnala I'm assuming this doesn't require a systemvmtemplate build (since changes are not in the cloud-early-config ?) |
@rohityadavcloud This may require new systemvm template build. #8048 (comment) |
Thanks @shwstppr @weizhouapache - that seems to be correct, but the other question is - is this fixing any issue? Is the failure of the service causing issues with systemvms and vrouters? |
@rohityadavcloud |
Thanks for checking @weizhouapache cc @shwstppr - if necessary let's build systemvmtemplate for 4.19.0 ? |
…ice (apache#8048) the service `user@0.service` fails in system vms and virtual routers This PR removes a change to fix memory leak of SSH connections in the systemvm templates with old linux kernel. (cherry picked from commit e290ac5) Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Description
the service
user@0.service
fails in system vms and virtual routersThis PR removes a change to fix memory leak of SSH connections in the systemvm templates with old linux kernel.
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?