Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Execute Shell Script from Python with multiple pipes

I want to execute the following Shell Command in a python script:

dom=myserver    
cat /etc/xen/$myserver.cfg | grep limited | cut -d= -f2 | tr -d \"

I have this:

dom = myserver

limit = subprocess.call(["cat /etc/xen/%s.cfg | grep limited | cut -d= -f2", str(dom)])
subprocess.call(['/root/bin/xen-limit', str(dom), str(limit)])

It does not work, but I don't know why..

Update:

c1 = ['cat /etc/xen/%s.cfg']
p1 = subprocess.Popen(c1, stdout=subprocess.PIPE)

c2 = ['grep limited']
p2 = subprocess.Popen(c2, stdin=p1.stdout, stdout=subprocess.PIPE)

c3 = ['cut -d= -f2']
p3 = subprocess.Popen(c3, stdin=p2.stdout, stdout=subprocess.PIPE)

c4 = ['tr -d \"']
p4 = subprocess.Popen(c4, stdin=p3.stdout, stdout=subprocess.PIPE)

result = p4.stdout.read()

limit = subprocess.call([result])
subprocess.call(['/root/bin/xen-limit', str(dom), str(limit)])
like image 437
Vince Avatar asked Oct 11 '13 13:10

Vince


2 Answers

Made it! :D Thanks

dom = myserver    
c1 = ['/bin/cat', '/etc/xen/%s.cfg' % (str(dom))]
p1 = subprocess.Popen(c1, stdout=subprocess.PIPE)

c2 = ['grep', 'limited']
p2 = subprocess.Popen(c2, stdin=p1.stdout,
                  stdout=subprocess.PIPE)

c3 = ['cut', '-d=', '-f2']
p3 = subprocess.Popen(c3, stdin=p2.stdout,
                  stdout=subprocess.PIPE)

c4 = ['tr', '-d', '\"']
p4 = subprocess.Popen(c4, stdin=p3.stdout,
                  stdout=subprocess.PIPE)

result = p4.stdout.read()
subprocess.call(['/root/bin/xen-limit', str(dom), str(result)])
like image 23
Vince Avatar answered Nov 15 '22 21:11

Vince


You can glue several subprocesses together:

c1 = ['ls']
p1 = subprocess.Popen(c1, stdout=subprocess.PIPE)

c2 = ['wc']
p2 = subprocess.Popen(c2, stdin=p1.stdout,
                      stdout=subprocess.PIPE)

result = p2.stdout.read()

Notice how we've set the stdin of p2 to be the stdout of p1.

EDIT: simplified the example

like image 197
Aidan Kane Avatar answered Nov 15 '22 21:11

Aidan Kane