Bug 2115462
| Summary: | extend error message when hitting execnet exception on closed I/O | ||
|---|---|---|---|
| Product: | [Red Hat Storage] Red Hat Ceph Storage | Reporter: | Michaela Lang <milang> |
| Component: | Cephadm | Assignee: | Adam King <adking> |
| Status: | NEW --- | QA Contact: | Manasa <mgowri> |
| Severity: | low | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 5.1 | CC: | cephqe-warriors, gjose, saraut |
| Target Milestone: | --- | ||
| Target Release: | 7.0 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | Type: | Bug | |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
I've checked out the posted PR. It looks good other than some stuff with the commit message. |
Description of problem: Exception reporting to generic Version-Release number of selected component (if applicable): 5.1 How reproducible: everytime Steps to Reproduce: 1. setup a Ceph cluster 2. prepare a host to be added to the cluster 3. do not configure "NOPASSWD" in sudoers file for the cephorch (cephadm configured) user 3.1 echo "${CEPHUSER} ALL=(ALL) ALL" > /etc/sudoers.d/cephorch 4. cephadm shell ceph orch host add node1 127.0.0.1 _admin Actual results: Can't communicate with remote host `127.0.0.1`, possibly because python3 is not installed there: cannot send (already closed?) Expected results: Added host 'node1' with addr '127.0.0.1' Additional info: Reason for the issue is comming from sudo with password closes the I/O and execnet fails which is catched by cephadm/ssh.py only with a generic error message. This leads to difficult debugging even though, the procedure and documentation for setting up is correct and mentioning NOPASSWD in sudoers as only supported way. I additionally created a pull request upstream as well https://github.com/ceph/ceph/pull/47464.