standard-test-roles

#360 Cleanup drive file handling
Merged by astepano. Opened by till.
till/standard-test-roles tempfiles  into  master

Download 360.patch

This should take care of tracebacks like:

Traceback (most recent call last):
  File "/home/till/scm/standard-test-roles/inventory/standard-inventory-qcow2", line 122, in cleanup
    fullname = os.path.join(tempfile['path'], tempfile['file'].name)
  File "/usr/lib64/python3.7/posixpath.py", line 80, in join
    a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not NoneType

Seems good, name will be absolute path, because of :

tempfile._mkstemp_inner(dir, pre, suf, flags):
            return (fd, _os.path.abspath(file))
NamedTemporaryFile:
    (fd, name) = _mkstemp_inner(dir, prefix, suffix, flags)

Commit de57fd18 fixes this pull-request

Pull-Request has been merged by astepano

Pull-Request has been merged by astepano
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.