Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for python 3.6 (bionic) #10

Closed
zxhdaze opened this issue Apr 25, 2024 · 0 comments
Closed

Add support for python 3.6 (bionic) #10

zxhdaze opened this issue Apr 25, 2024 · 0 comments

Comments

@zxhdaze
Copy link
Contributor

zxhdaze commented Apr 25, 2024

The module uses python features that require a newer python version than provided in Ubuntu Bionic (18.04), which is 3.6.9 - In particular dataclasses, which became available in python 3.7. The stacktrace below is an example of the unit test suite failing due to this issue.

It would be convenient if the jujubackupall module had support for the python version in Ubuntu Bionic, as it is still used widely.

======================================================================
ERROR: test_process (unittest.loader._FailedTest)

ImportError: Failed to import test module: test_process
Traceback (most recent call last):
File "/home/jguedez/.pyenv/versions/3.6.9/lib/python3.6/unittest/loader.py", line 428, in _find_test_path
module = self._get_module_from_name(name)
File "/home/jguedez/.pyenv/versions/3.6.9/lib/python3.6/unittest/loader.py", line 369, in _get_module_from_name
import(name)
File "/home/jguedez/canonical/repos/bootstack/juju-backup-all/tests/unit/test_process.py", line 10, in
from jujubackupall.process import BackupProcessor, ControllerProcessor, JujuModel
File "/home/jguedez/canonical/repos/bootstack/juju-backup-all/jujubackupall/process.py", line 30, in
from jujubackupall.backup import (
File "/home/jguedez/canonical/repos/bootstack/juju-backup-all/jujubackupall/backup.py", line 20, in
import dataclasses
ModuleNotFoundError: No module named 'dataclasses'


Ran 19 tests in 0.011s

FAILED (errors=3)


Imported from Launchpad using lp2gh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant