-
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
VR: fix password server exception when no password is found #9699
VR: fix password server exception when no password is found #9699
Conversation
see errors in /var/log/daemon.log below ``` Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: ---------------------------------------- Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: Exception occurred during processing of request from ('192.168.20.8', 51108) Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: Traceback (most recent call last): Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/socketserver.py", line 691, in process_request_thread Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self.finish_request(request, client_address) Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/socketserver.py", line 361, in finish_request Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self.RequestHandlerClass(request, client_address, self) Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/socketserver.py", line 755, in __init__ Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self.handle() Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/http/server.py", line 432, in handle Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self.handle_one_request() Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/http/server.py", line 420, in handle_one_request Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: method() Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/opt/cloud/bin/passwd_server_ip.py", line 117, in do_GET Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self.wfile.write('saved_password') Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: File "/usr/lib/python3.11/socketserver.py", line 834, in write Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: self._sock.sendall(b) Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: TypeError: a bytes-like object is required, not 'str' Sep 13 12:36:58 systemvm passwd_server_ip.py[2154]: ---------------------------------------- ```
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9699 +/- ##
============================================
- Coverage 15.81% 15.81% -0.01%
Complexity 12554 12554
============================================
Files 5629 5629
Lines 492031 492031
Branches 62921 61224 -1697
============================================
- Hits 77823 77813 -10
- Misses 405888 405895 +7
- Partials 8320 8323 +3
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.
clgtm
@blueorangutan package |
@weizhouapache a [SL] 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]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11138 |
@blueorangutan test |
@weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
[SF] Trillian test result (tid-11509)
|
LGTM, passing to @JoaoJandre to review and consider merging. |
The code LGTM. Could you add a description of the tests @weizhouapache ? |
@JoaoJandre |
Description
This PR fixes errors appeared in /var/log/daemon.log below
steps to reproduce the issue
/var/log/daemon.log
in cloudstack VRThis is a minor issue caused by python3 upgrade #8497 . There is no real issue with the user vms.
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?