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

Python 3.8: ansible, ansible-lint #48645

Closed
wants to merge 2 commits into from

Conversation

bayandin
Copy link
Member

@bayandin bayandin commented Jan 7, 2020

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Ref #47274

@bayandin bayandin added the python Python use is a significant feature of the PR or issue label Jan 7, 2020
@fxcoudert fxcoudert closed this in be6eb40 Jan 7, 2020
@bayandin bayandin deleted the python-3.8-p5 branch January 7, 2020 09:40
@oschrenk
Copy link
Contributor

Not sure where to report this but I see a bunch of errors when working with ansible 2.9.2_1 and python@3.8

PLAY [apply common configuration to all nodes] **********************************************
/usr/local/Cellar/ansible/2.9.2_1/libexec/lib/python3.8/site-packages/Crypto/Random/Fortuna/FortunaGenerator.py:28: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if sys.version_info[0] is 2 and  sys.version_info[1] is 1:
/usr/local/Cellar/ansible/2.9.2_1/libexec/lib/python3.8/site-packages/Crypto/Random/Fortuna/FortunaGenerator.py:28: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if sys.version_info[0] is 2 and  sys.version_info[1] is 1:

TASK [Gathering Facts] **********************************************************************
/usr/local/Cellar/python@3.8/3.8.1/Frameworks/Python.framework/Versions/3.8/lib/python3.8/multiprocessing/resource_tracker.py:96: UserWarning: resource_tracker: process died unexpectedly, relaunching.  Some resources might leak.
  warnings.warn('resource_tracker: process died unexpectedly, '
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'multiprocessing.resource_tracker'

Is there a way to still use ansible with 3.7 ?

@bayandin
Copy link
Member Author

bayandin commented Jan 16, 2020

/usr/local/Cellar/ansible/2.9.2_1/libexec/lib/python3.8/site-packages/Crypto/Random/Fortuna/FortunaGenerator.py:28: SyntaxWarning: "is" with a literal. Did you mean "=="?
if sys.version_info[0] is 2 and sys.version_info[1] is 1:
/usr/local/Cellar/ansible/2.9.2_1/libexec/lib/python3.8/site-packages/Crypto/Random/Fortuna/FortunaGenerator.py:28: SyntaxWarning: "is" with a literal. Did you mean "=="?
if sys.version_info[0] is 2 and sys.version_info[1] is 1:

These warnings definetely came from python@3.8 and I will fix them

/usr/local/Cellar/python@3.8/3.8.1/Frameworks/Python.framework/Versions/3.8/lib/python3.8/multiprocessing/resource_tracker.py:96: UserWarning: resource_tracker: process died unexpectedly, relaunching. Some resources might leak.
warnings.warn('resource_tracker: process died unexpectedly, '
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'multiprocessing.resource_tracker'

Regarding this one, It's unclear for me right now, could you run ansible with -vvvv flag (to get verbose output)

@bayandin bayandin mentioned this pull request Jan 16, 2020
5 tasks
@oschrenk
Copy link
Contributor

@oschrenk
Copy link
Contributor

I also found this: spack/spack#13256

Let me know if I can do anything to help debug

@lock lock bot added the outdated PR was locked due to age label Feb 15, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age python Python use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants