Project

General

Profile

Actions
Copy link

Bug #19425

closed

run-playbook errors in ceph-ansible.py

Added by Anonymous about 7 years ago. Updated over 4 years ago.

Status:
Can't reproduce
Priority:
Normal
Assignee:
-
Category:
ansible
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

The following text in ceph_ansible looks wrong:

            run.Raw(';'),
            'pip',
            'install',
            'setuptools>=11.3',
            run.Raw(ansible_ver),
            run.Raw(';'),

It ends up redirecting output to a file named =11.3

Actions
Copy link
 #1

Updated by Anonymous about 7 years ago

Both the setuptools line and the ansible_ver line below it need to be quoted, when all is said and done.

Actions
Copy link
 #2

Updated by Anonymous about 7 years ago 

                run.Raw('"setuptools>=11.3"'),
                run.Raw('"%s"' % ansible_ver),
Actions
Copy link
 #3

Updated by Anonymous about 7 years ago

It looks like there are two more problems here.

One is that an ansisble command 'ansible -vv ...' is included in this list. It should be broken into 'ansible', '-vv', ...

The second is that this code appears to be hit by: https://bugzilla.redhat.com/show_bug.cgi?id=1354701

Actions described in: https://access.redhat.com/solutions/2943461 seem to fix this problem.

Actions
Copy link
 #4

Updated by Anonymous about 7 years ago

  • Subject changed from missing run.Raw() in ceph-ansible.py to run-playbook errors in ceph-ansible.py

These problems probably also exist in run_rh_playbook (the ansible command being one line, and the potential problem with bugzilla 1354701).

I think that this code could be merged with run_playbook with some not super onerous modifications.

Actions
Copy link
 #5

Updated by Tamilarasi muthamizhan over 6 years ago

Warren, please paste the error seen with run-playbook.

Actions
Copy link
 #6

Updated by Anonymous over 6 years ago

This problem just bit me again. I think that the code as written is definitely broken. When I ran it, the command failed and afterwards, I got a file in ~/ceph-ansible named "=11.3"

Actions
Copy link
 #7

Updated by Anonymous over 6 years ago

There may be more to this. Right now, the ceph-ansible directory that I am using does not have the ceph-ansible yml files in it (just venv). Still investigating.

Actions
Copy link
 #8

Updated by Anonymous over 6 years ago

The problem that I am running into is caused by ceph-ansible not being installed. It it probably okay for the previous code to be as is (although values are still hard-coded).

Actions
Copy link
 #9

Updated by Kyrylo Shatskyy over 4 years ago

Looks like the issue should not be reproducible any further, since I can see a fix for that:

> grep -A 2 -B 2 setuptools teuthology/task/ceph_ansible.py 
            'pip',
            'install',
            run.Raw('setuptools>=11.3'),
            run.Raw('notario>=0.0.13'), # FIXME: use requirements.txt
            run.Raw('netaddr'),
Actions
Copy link
 #10

Updated by Kyrylo Shatskyy over 4 years ago

  • Status changed from New to Can't reproduce
Actions
Copy link

Also available in: Atom PDF