Bug 56162 - Join.log spammed with tracebacks
Join.log spammed with tracebacks
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: General
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0-4-errata
Assigned To: Philipp Hahn
Julia Bremer
https://git.knut.univention.de/univen...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2023-06-20 10:05 CEST by Julia Bremer
Modified: 2023-08-23 14:54 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.086
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments
Revert to inherit from object (1.04 KB, patch)
2023-06-20 10:07 CEST, Julia Bremer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julia Bremer univentionstaff 2023-06-20 10:05:26 CEST
Since commit aeb07b074d style(lib): Properly implement TextIOBase protocol

The join.log of our tests is spammed with this Traceback.
It doesn't seem to have any consequences but can mask real errors.

stderr: io.UnsupportedOperation: fileno
  Traceback (most recent call last):
    File "/usr/lib/python3/dist-packages/apt/progress/text.py", line 192, in pulse
      not os.isatty(self._file.fileno())):
  io.UnsupportedOperation: fileno
  Traceback (most recent call last):
    File "/usr/lib/python3/dist-packages/apt/progress/text.py", line 192, in pulse
      not os.isatty(self._file.fileno())):
  io.UnsupportedOperation: Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
  Error in function pulse
Comment 1 Julia Bremer univentionstaff 2023-06-20 10:07:38 CEST
Created attachment 11072 [details]
Revert to inherit from object

This patch fixes the error for me
Comment 2 Philipp Hahn univentionstaff 2023-06-27 14:55:12 CEST
Reproducer:
```python
import  logging
logging.basicConfig(level=logging.INFO)
from univention.lib.package_manager import PackageManager
from package_manager import PackageManager
pm = PackageManager()
pm.update()
```

The current implementation heavily depends on the APT text protocol. Instead the implementation should be based on the APT API and should directly translate those calls into the output expected by `univention.management.console.modules.setup.util.ProgressParser`. This is implemented in univention/ucs!816
Comment 3 Philipp Hahn univentionstaff 2023-08-19 14:39:55 CEST
https://git.knut.univention.de/univention/ucs/-/commit/711d7f1f277e247f81b662446ba265058fbb14d0

Package: univention-lib
Version: 9.0.16-4
Branch: ucs_5.0-0
Scope: errata5.0-4

OK: errata-announce -V --only univention-lib.yaml
QA: See <https://git.knut.univention.de/univention/ucs/-/merge_requests/816>
Comment 4 Julia Bremer univentionstaff 2023-08-22 19:45:58 CEST
OK: No more tracebacks during system setup
OK: YAML
OK: built
Verified