Description of problem: --upload feature does not provide much (verbose) logging, esp. when something goes wrong (or for uploading via sftp protocol). Let enhance that logging after some agreement when&what shall be logged but is currently missing. Version-Release number of selected component (if applicable): sos-4.2-2 How reproducible: 100% Steps to Reproduce: sos report -vvv --upload [--upload-protocol=sftp] .. and various scenarios wrt. success/failure/cause-of-failure Actual results: minimal (debug) logs Expected results: more (debug) logs Additional info:
This is going to be difficult to really do well, as our (file) logging is closed by the time we get to the upload attempt: https://github.com/sosreport/sos/blob/main/sos/report/__init__.py#L1409 We can build out some console messaging, but what specifically are you looking for more information on to report? Currently if anything in `Policy.upload_archive()` fails we raise an appropriately (hopefully, anyways) descriptive exception whose message is then printed to console.
(In reply to Jake Hunsaker from comment #2) > This is going to be difficult to really do well, as our (file) logging is > closed by the time we get to the upload attempt: > > https://github.com/sosreport/sos/blob/main/sos/report/__init__.py#L1409 .. and that file logging must be closed as the logfile is contained in the tarball we attempt to upload. > > We can build out some console messaging, but what specifically are you > looking for more information on to report? Currently if anything in > `Policy.upload_archive()` fails we raise an appropriately (hopefully, > anyways) descriptive exception whose message is then printed to console. This went out from discussion with Miroslav, raising needinfo on him when&what to more debug-log.
After evaluating this issue, there are no plans to address it further or fix it in an upcoming release. Therefore, it is being closed. If plans change such that this issue will be fixed in an upcoming release, then the bug can be reopened.
Reopening the auto-closed BZ since this is being implemented / reviewed in upstream, with soon plans to have it in RHEL downstream.
Hello, this bug is assumed to be fixed in errata https://access.redhat.com/errata/RHBA-2023:3801 so I am closing this BZ as fixed in current release. If you feel otherwise, please reopen the BZ and provide details what is missing.