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

Updating python-passlib assert #4277

Merged
merged 1 commit into from
May 26, 2017
Merged

Conversation

ewolinetz
Copy link
Contributor

@ewolinetz ewolinetz commented May 24, 2017

@ewolinetz ewolinetz requested a review from sdodson May 24, 2017 19:28
@metal3d
Copy link

metal3d commented May 25, 2017

I guess my suggestion at #4285 is probably better. I'm not using rpm but a python command. Because passlib can be installed with "pip" for example, so RPM will not find the package. Am I wrong ?

@sdodson
Copy link
Member

sdodson commented May 25, 2017

Yeah, while control host being other platforms is not something we explicitly support I'd prefer that anything that happens on the control host be as portable as possible. So I think attempting a python import is probably a better solution than relying on rpm.

@ewolinetz
Copy link
Contributor Author

@sdodson updated to use approach used by @metal3d

@metal3d
Copy link

metal3d commented May 25, 2017

So I will close my PR and vote for yours

@sdodson
Copy link
Member

sdodson commented May 25, 2017

aos-ci-test

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for 219db68 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_NOT_containerized, aos-ci-jenkins/OS_3.6_NOT_containerized_e2e_tests" for 219db68 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for 219db68 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 219db68 (logs)

@sdodson
Copy link
Member

sdodson commented May 25, 2017

[merge]

@openshift-bot
Copy link

[test]ing while waiting on the merge queue

@openshift-bot
Copy link

Evaluated for openshift ansible test up to 219db68

@openshift-bot
Copy link

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/156/) (Base Commit: a353d4d)

@sdodson
Copy link
Member

sdodson commented May 25, 2017

[merge][severity: bug]

@sdodson
Copy link
Member

sdodson commented May 26, 2017

flake openshift/origin#13980
[merge][severity:bug]

@openshift-bot
Copy link

Evaluated for openshift ansible merge up to 219db68

@openshift-bot
Copy link

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/467/) (Base Commit: 109a54e) (Extended Tests: bug)

@sdodson sdodson merged commit daed7a1 into openshift:master May 26, 2017
@isaldarriaga
Copy link

@sdodson still happenning.

Client's output:

$ git describe
openshift-ansible-3.10.29-1-22-gdec10a6d6
$ cat /etc/*-release
Fedora release 28 (Twenty Eight)
...
$ ansible-playbook -i "/path/to/inventory.yaml" "playbooks/openshift-metrics/config.yml"
...
TASK [openshift_metrics : Check that python-passlib is available on the control host] ********************************************
task path: /home/ivan/convergence.cloud/dev-ops/kube-prod/origin/openshift-ansible-v3.10/roles/openshift_metrics/tasks/main.yaml:6
fatal: [prod-master-01.convergence.cloud]: FAILED! => {
    "assertion": "'not installed' not in passlib_result.stdout", 
    "changed": false, 
    "evaluated_to": false, 
    "msg": "python-passlib rpm must be installed on control host"
}
...
prod-master-01.example.com : ok=55   changed=2    unreachable=0    failed=1
...

Server's output:

[root@prod-master-01 ~]# cat /etc/*-release
CentOS Linux release 7.5.1804 (Core)
...

[root@prod-master-01 ~]# rpm -qa | grep passlib
python-passlib-1.6.5-2.el7.noarch

[root@prod-master-01 ~]# rpm -q python2-passlib
package python2-passlib is not installed

[root@prod-master-01 ~]# yum info python-passlib
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: mirror.scalabledns.com
 * extras: mirror.hmc.edu
 * updates: centos.sonn.com
Installed Packages
Name        : python-passlib
Arch        : noarch
Version     : 1.6.5
Release     : 2.el7
Size        : 2.7 M
Repo        : installed
From repo   : extras
Summary     : Comprehensive password hashing framework supporting over 20
            : schemes
URL         : https://bitbucket.org/ecollins/passlib
License     : BSD and Beerware and Copyright only
Description : Passlib is a password hashing library for Python 2 & 3, which
            : provides cross-platform implementations of over 20 password
            : hashing algorithms, as well as a framework for managing existing
            : password hashes. It's designed to be useful for a wide range of
            : tasks, from verifying a hash found in /etc/shadow, to providing
            : full-strength password hashing for multi-user application.

this problem is fixed in ansible 2.40 and 2.3.2

$ ansible --version

ansible 2.6.2
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/ivan/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.15 (default, May 16 2018, 17:50:09) [GCC 8.1.1 20180502 (Red Hat 8.1.1-1)]

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

Successfully merging this pull request may close these issues.

5 participants