continue if 'subprocess.call' failes

[tester_V]

Hello!
I'm pulling out logs from remote hosts to my server for processing.
The server has a faceless account and all the Hosts have it added as an admin(admin group).
Sometimes, a new Host has a bad configuration and it asks for UserName...
I thought I can use try/except to keep going if an error or cannot log in...
for some reason, the code just stops with the message:
"The password is invalid for..."
I thought "try/except' will make it keep going but it is not:

try :
	subprocess.call("net use U: /delete /yes")
	time.sleep(1)
	subprocess.call("net use U: "+"\\\\"+itm+"\\c$\\SMTS\\TDR_Logs")
	tr = time.localtime()
	cutr = time.strftime("%Y-%m-%d %I:%M:%S", tr)
	print(f" Dir mounted -->> {cutr} -- {host_path}")            
except OSError as st :
	print(f" Error --> {st}")
	pass

Any idea on how to ignore the message ""The password is invalid for...""?
Thank you!

[Gribouillis]

I'm surprised your subprocess call works at all. I think you should write it as

subprocess.call(["net", "use", "U:",  "/delete",  "/yes"])

[tester_V]

I'm not sure if it is the right way to write the code but It does work.
I tried your suggestion:

subprocess.call(["net", "use", "U:",  "/delete",  "/yes"])

It works too but I still have the same message:
"The password is invalid for..."
"Enter the user name for ...."
thank you

[bowlofred]

I would probably not try to parse the output, but would just try a timeout. Give the command a timeout value and see if it will kill it after some time.

subprocess.call(["net",  "use",  "U:", f"\\\\{itm}\\c$\\SMTS\\TDR_Logs"], shell=True, timeout=10)

Then be able to handle the subprocess.TimeoutExpired exception and continue or whatever you want to do when the command doesn't succeed.

Your other choice is to not use run() but Popen.communicate() directly (since you want to see what is returned before the process exits). For this particular case, I don't think that's necessary (but would be faster since you don't have to wait for an arbitrary timeout).

[tester_V]

I just tried:

subprocess.call("net use U: "+"\\\\"+itm+"\\c$\\SMTS\\TDR_Logs", shell=True, timeout=10)

it still asks for the "The password is invalid for"

Then produces errors and exits.
Last error:

AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 1473, in _wait
raise TimeoutExpired(self.args, timeout)
subprocess.TimeoutExpired: Command 'net use U: \\SomeITEM\c$\SMTS\TDR_Logs' timed out after 5 seconds

Thank you.

[bowlofred]

Yes, it won't stop the process from asking. But you don't have to actually answer. If you don't want to see it ask, add capture_output=True to the options.

So something like:

import subprocess

try:
    subprocess.call("net use U: "+"\\\\"+itm+"\\c$\\SMTS\\TDR_Logs", shell=True, timeout=10, capture_output=True)
except subprocess.TimeoutExpired:
    print(f"Server {itm} took too long to mount.  Skipping...")
    continue #or break or return or whatever appropriate to go to the next server....

[tester_V]

Now it is a different error and it is complaining about the "capture_output)"

Error:
AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 349, in call
with Popen(*popenargs, **kwargs) as p:
TypeError: __init__() got an unexpected keyword argument 'capture_output'

Thak you!

[bowlofred]

Changed in version 3.7: Added the text parameter, as a more understandable alias of universal_newlines. Added the capture_output parameter.

Looks like capture_output becomes available in 3.7. You should consider using a more current release.

For earlier versions I think you can just open stdout/stderr directly....

import subprocess
 
try:
    subprocess.call("net use U: "+"\\\\"+itm+"\\c$\\SMTS\\TDR_Logs", shell=True, timeout=10, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
except subprocess.TimeoutExpired:
    print(f"Server {itm} took too long to mount.  Skipping...")
    continue #or break or return or whatever appropriate to go to the next server....

[snippsat]

Looks like capture_output becomes available in 3.7. You should consider using a more current release.

I think the problem is that he us call() then there is no parameter capture_output .
capture_output only in run().

[bowlofred]

I think the problem is that he us call() then there is no parameter capture_output .
capture_output only in run().

Ah, thank you. To avoid changing the command string I used for testing, I just modified the string in the thread and didn't notice that I was testing with run() instead of call().

I don't think there's a benefit to using call, so changing it to run should work (depending on the version available).

import subprocess
 
try:
    subprocess.run("net use U: "+"\\\\"+itm+"\\c$\\SMTS\\TDR_Logs", shell=True, timeout=10, capture_output=True)
except subprocess.TimeoutExpired:
    print(f"Server {itm} took too long to mount.  Skipping...")
    continue #or break or return or whatever appropriate to go to the next server....