Temporary directory is not cleaned up in cdist-web #146

New issue

Closed

opened 2021-11-20 15:20:12 +00:00 by ungleich-gitea · 4 comments

ungleich-gitea commented

2021-11-20 15:20:12 +00:00

Owner

Created by: pcoder

from cdist.integration import configure_hosts_simple
import cdist
import tempfile

def my_configure_host(hosts):
    return_value = True
    with tempfile.NamedTemporaryFile() as tmp_manifest:
        tmp_manifest.writelines([b'__cdist_marker'])
        try:
            configure_hosts_simple(hosts,
                                   tmp_manifest.name,
                                   verbose=cdist.argparse.VERBOSE_TRACE)
        except Exception as cdist_exception:
            logger.error(cdist_exception)
            return_value = False
    return return_value
    
my_configure_host(['my_host'])

Everytime I run my_configure_host one or more new temporary directories are created in /tmp and they don't get cleaned up on completion of the script.

However, if I run the same manifest using cdist config -vv my_host, the temporary directory is cleaned up perfectly.

*Created by: pcoder* ``` from cdist.integration import configure_hosts_simple import cdist import tempfile def my_configure_host(hosts): return_value = True with tempfile.NamedTemporaryFile() as tmp_manifest: tmp_manifest.writelines([b'__cdist_marker']) try: configure_hosts_simple(hosts, tmp_manifest.name, verbose=cdist.argparse.VERBOSE_TRACE) except Exception as cdist_exception: logger.error(cdist_exception) return_value = False return return_value my_configure_host(['my_host']) ``` Everytime I run `my_configure_host` one or more new temporary directories are created in /tmp and they don't get cleaned up on completion of the script. However, if I run the same manifest using `cdist config -vv my_host`, the temporary directory is cleaned up perfectly.

ungleich-gitea closed this issue

2021-11-20 15:20:12 +00:00

ungleich-gitea commented

2021-11-20 15:43:12 +00:00

Author

Owner

Created by: pcoder

Thanks @darko-poljak for the quick fix. That works like a charm 👍

*Created by: pcoder* Thanks @darko-poljak for the quick fix. That works like a charm 👍

ungleich-gitea commented

2021-11-20 15:43:14 +00:00

Author