-
Notifications
You must be signed in to change notification settings - Fork 76
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
python3 [Ansible] #2019
python3 [Ansible] #2019
Conversation
not a sample now |
This looks like it might be ready? What OS's has this been tested on? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
Let's offer more clarifying explanation as to what
ansible -m setup ...
is doing? -
Looks like a bug in
runrole
?
My intentition for '-m setup | grep python' to be an in your face feedback as to which python ansible is running while demo'ing/debugging ansible running under python3. Would also make a great question as to deduce the age of the user's install. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jvonau can we remove line 154 from iiab-install here?
It seems to be a duplicate of the line above it.
(Or if it serves a purpose, please explain, thanks.)
FYI this cannot be merged as it fails every time when running
In contrast, the same Ubuntu 18.04.3 VM with the same Ansible 2.9.0rc5 runs to completion when on Python 2.7 (i.e. iiab/iiab master branch). Either way,
|
Think you either forgot to manually install python3-pymysql or didn't test the other part of the PR that adds python3-pymysql to scripts/ansble. |
I did run
And yet |
Don't think ansible --version can be relied upon, on rpi:
yet we have
And continues to runs to completion. 19.10 does work with python-3.7 out of the box once and python3-distutils and python3-pymysql is added with ansible version 2.8.3 supplied by Ubuntu so 2.9 should work there. |
This works on Ubuntu 19.10 with Ansible 2.9.0rc5 on Python 3.7.5rc1 for whatever reason:
Context:
|
I'm seeing a random flip in python usage with
Can you edit ansible.cfg to be and test: I think -e on the cmdline can go away if the above works, 18.04 being the problem child at the moment. |
Uh-oh to clarify the above post from 1hr ago
|
Results from the new commit (aff014a) added to this PR ~40min ago:
|
passlib is new.. I have python3-passlib installed |
Results from the new commit (7035bb3) added to this PR ~30min ago:
|
Can you check if python*-mysqldb is installed please. Suggests to fix Ubuntu I've added a commit here to ease your testing but should not really be part of this PR. |
No ansible is generating the traceback due to the missing library, looking to keep all ansible's dependencies in one place. |
Looks like the problem child has:
From the above link
I'd prefer one approach for the role.. |
FYI the very latest version of this PR still fails on Ubuntu 18.04.3:
(Is this expected?) Output if it helps...
|
In the past ansible's optional modules that need external libraries were intermixed with the roles' dependencies making finding all of ansible's full dependencies disjointed, but be much clearer now. |
If a little heavier now forcing everyone to install all 7 packages {python3-distutils python3-mysqldb python3-passlib python3-pip python3-setuptools python3-virtualenv python3-psycopg2}. Making it harder to see which of the 7 above dependencies relates to which playbook (example: if we decide to remove I'm now trying this PR as a fresh install on Debian 10.1 where Ansible 2.9.0rc5 is using Python 3.7.3 and running PS This PR obviously doesn't modify https://github.com/iiab/iiab-admin-console/blob/master/install but are there any considerations there to keep in mind for when that happens? (Similar to this PR's 1-lin [optional?] mods to iiab-install, iiab-network, iiab-support, runrole ?) PPS this PR has been tested with |
FYI a fresh install of this PR (#2019, BIG-sized) failed on Debian 10.1 as follows:
Context upon re-running
|
Moodle is rarely used but part of Medium so it's going to be installed anyway. Having ansible's needs intermixed with what is being installed for the role's needs doesn't really help. You can do your thing and add the notes in scripts/ansible.
They would only end up being installed at a later point anyway, now we pay upfront instead of over time during the install. Looking to isolate python2 related actions for future porting/removal.
With no ansible.cfg file to enable exclusive use of python3 ansible will run python2 there, and should be fine as an upgrade because at somepoint all the dependencies are met. Fresh installs should be ok also as python-pip is being installed if needed in those plays. |
That was sort of anticipated on what was a python3 only machine, the fix is in #2024 with the venv updates contained there. |
sample for #1810