Travis CI lint failed on [testing_master] nightly PR #3432 with the following errors:
echo "Pylint on /usr/bin/python3 is running, please wait ..."; \ PYTHONPATH=. /usr/bin/python3 -m pylint \ --rcfile=./pylintrc \ --load-plugins pylint_plugins \ ${FILES} Pylint on /usr/bin/python3 is running, please wait ... ************* Module ipaclient.plugins.misc ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipaclient.plugins.rpcclient -> ipalib -> ipaclient.remote_plugins)) ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipatests.pytest_ipa.integration.config -> ipatests.pytest_ipa.integration.env_config)) ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipatests.pytest_ipa.integration.config -> ipatests.pytest_ipa.integration.host -> ipatests.pytest_ipa.integration.env_config)) ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipaclient.plugins.rpcclient -> ipalib.rpc -> ipalib -> ipaclient.remote_plugins)) ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipaclient.frontend -> ipalib -> ipaclient.remote_plugins -> ipaclient.remote_plugins.schema)) ipaclient/plugins/misc.py:1: [R0401(cyclic-import), ] Cyclic import (ipaclient.frontend -> ipalib -> ipaclient.remote_plugins -> ipaclient.remote_plugins.compat)) ------------------------------------ Your code has been rated at 10.00/10
In this run, python3-pylint-2.3.1-1.fc30.noarch is used.
I propose to add cyclic-import to the ignore list in pylintrc. Cyclic dependencies can cause issues and are not elegant. But so far they haven't caused any serious issues.
cyclic-import
pylintrc
We can look into the issue another time.
Was this resolved by the changes in https://pagure.io/freeipa/issue/8102 ?
That changes (https://pagure.io/freeipa/issue/8102) were not addressed to cyclic-import. The 'issue' still presents and could be reproduced in single-process mode of Pylint (pylintrc: jobs=1).
jobs=1
Travis-CI is not used anymore, and on azure the tests don't show this issue (they are currently using pylint 2.8.3).
Closing as fixed.
Metadata Update from @frenaud: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
Login to comment on this ticket.