Python crontab try except ValueE

Question

我有这个脚本比我从命令行运行它工作得很好。。。在

#!/usr/bin/python
# -*- coding: utf-8 -*-

import time
from time import sleep
import os
import logging
import glob
import sys
import subprocess
import pymssql
import shutil
import StringIO
from StringIO import StringIO

#Check share mounted
if os.path.ismount("/home/marco/cas01-share/") == 0:

    #logging.info('share not mounted')
    print "share not mounted"
    #mount share
    #logging.info('share now mounted')
    #print "share now mounted"
else:
    print "share mounted"

# File paths
fullpath2Newfile = "/home/marco/cas01-share/New/"
fullpathfiles2Process = "/home/marco/cas01-share/Process/"
fullpathProcessedfiles = "/home/marco/cas01-share/Processed/"

if not os.listdir('/home/marco/cas01-share/New'):
    print "directory empty"
else:
    print "directory not empty"

    # Directory more than one file?
    num_files = len([f for f in os.listdir('/home/marco/cas01-share/New')
                if os.path.isfile(os.path.join('/home/marco/cas01-share/New', f))])
    print "directory has " + str(num_files) + " files"



    num_files = 0

    for filename in os.listdir('/home/marco/cas01-share/New'):
        succesfullRun = 0
        num_files += 1

        print filename
        filename = str(filename)
        batchnumber = filename.strip("['SENSE-extract-ascii--.csv']")[:-9]
        print batchnumber

        newfullpath2Newfile = fullpath2Newfile + filename
        fullpathfiles2Process = "/home/marco/cas01-share/Process/SENSE-extract.csv"
        fullpathProcessedfiles = "/home/marco/cas01-share/Processed/SENSE-extract-" + batchnumber + ".csv"
        print newfullpath2Newfile
        print fullpathfiles2Process
        print fullpathProcessedfiles


        # Execute job
        ExceptionFlag = 0
        while (succesfullRun == 0):

            if ExceptionFlag == 0:
                shutil.move(newfullpath2Newfile, fullpathfiles2Process)

            try:

                # Move to sense check folder
                # Connect to DB
                conn = pymssql.connect(server='IP', user='User', password='password', tds_version='8.0')
                cursor = conn.cursor()
                cursor.execute("EXEC msdb.dbo.sp_start_job @job_name = 'Marco_test'")

            except Exception as e:
                (error_code, error_message) = e

                ExceptionFlag = 1

                #print error_message

                if "does not exist" in error_message:
                    print "ERROR: Job name not found"

                if "job is already running" in error_message:
                    print "ERROR: a Job is already running"

                if "job already has a pending request" in error_message:
                    print "ERROR: pending request"

                time.sleep(0.5)

            else:
                print "Job executed succesfully"
                # Copy file to processed
                shutil.copy(fullpathfiles2Process, fullpathProcessedfiles)

                succesfullRun = 1

            conn.close()
            time.sleep(0.5)

exit()

但是当我试图在crontab上进行计划时，我得到了如下错误

^{pr2}$

我得到这个错误：

Traceback (most recent call last): File "...", line 239, in (error_code, error_message) = e ValueError: need more than 1 value to unpack

我认为这是因为“cronos schedule executed job”和它自己的脚本之间没有管道，我不知道这是否有意义，就像它不知道从何处获取信息和viceversa。。。在

本质上是在试图捕捉执行MSSQL作业命令的异常。在

Answer 1

你的问题是你正在解包一个异常。不应假定异常是ITerable，也不应假定它们包含参数。在

In [2]: try:
   ...:     assert False
   ...: except Exception as e:
   ...:     c, m = e
   ...:     
                                     -
ValueError                                Traceback (most recent call last)
<ipython-input-2-2e0dd0160822> in <module>()
      2     assert False
      3 except Exception as e:
  > 4     c, m = e
      5 

ValueError: need more than 0 values to unpack

如果您将消息传递给异常类，则它的值为1：

^{pr2}$

错误处理固有的会在错误捕获后产生错误。学习使用回溯或Exception.args代替：

回溯示例

In [10]: try:
   ....:         assert False, "This is a message"
   ....: except Exception as error:
   ....:     trace = traceback.format_exc()
   ....:     print(trace, file=sys.stderr)
   ....:     
Traceback (most recent call last):
  File "<ipython-input-10-5a56c0f9112a>", line 2, in <module>
    assert False, "This is a message"
AssertionError: This is a message

异常.args示例

In [11]: from __future__ import print_function

In [12]: import sys

In [13]: try:
   ....:        assert False, "This is a message"
   ....: except Exception as error:
   ....:        print(error.args, file=sys.stderr)
   ....:     
('This is a message',)

Answer 2

更新和工作代码：

#!/usr/bin/python
# -*- coding: utf-8 -*-

import time
from time import sleep
import os
import logging
import glob
import sys
import subprocess
import pymssql
import shutil
import StringIO
from StringIO import StringIO
import mechanize
from BeautifulSoup import BeautifulSoup
import urllib2

#Configure logging
if os.path.isdir("/home/administrator/Share/scripts/logs/") == 0:
    os.makedirs("/home/administrator/Share/scripts/logs/")

if os.path.isfile("/home/administrator/Share/scripts/logs/sense.log") == 0:
    open("/home/administrator/Share/scripts/logs/sense.log", 'w')

logging.basicConfig(filename='/home/administrator/Share/scripts/logs/sense.log',level=logging.DEBUG,format='%(asctime)s - %(message)s')
logging.info('RUNNING SCRIPT')

#Check share mounted
if os.path.ismount("/home/administrator/-share/") == 0:

    logging.info('share not mounted')
    #print "share not mounted"
    #mount share
    os.popen("sudo mount -t cifs \"//path to network share -o username=username,domain=domain,sec=ntlmssp,password=password")
    logging.info('share now mounted')
    #print "share now mounted"

# start Automated browsing
browser = mechanize.Browser()
browser.open("page")
browser.select_form(nr = 0)
browser.form['email'] = "user"
browser.form['password'] = "pass"
browser.submit()

#Open Page (Data)
browser.follow_link(text="Data")

#Find generate button
i = 0
batch = 0
for form in browser.forms():

    browser.form = list(browser.forms())[0]  # use when form is unnamed

    for control in browser.form.controls:
        if control.name == "generate":
            batch = 1
            logging.info('generate detected')
            #print "generate detected"
            #Click button
            browser.submit(nr=i)

    if batch == 0:
        logging.info('nothing to generate')
        #print "nothing to generate"
        #print "*******************"

    i =+ 1
#END Find generate button

#Back to "admin/Dowload" page (Data)
browser.follow_link(text="Data")

# Data Table parsing
html = browser.response().read()
soup = BeautifulSoup(html)

table = soup.find("table")

i = 0
for row in table.findAll("tr")[1:]:

    col = row.findAll("td")

    if not col:
        break

    batch = str(col[0])
    records = col[1]
    generated = col[2]
    downloaded = col[3]
    links = col[4]
    photos = col[5]
    sheet = str(col[6])
    confirmed = col[7]

    if "Not yet downloaded" in confirmed:
        i += 1

        stripbatch = batch.strip("<td></td>")
        stripsheet = sheet.strip("<td><a href=\"download/xls/\">Excel</a></td>")

        url = "website" + stripsheet
        time.strftime("%d/%m/%Y")
        file = "/home/administrator/-share/New/SENSE-extract-ascii-" + stripsheet + "-" + time.strftime("%Y%m%d") + ".csv"

        #Dowload file
        logging.info('File %s Downloaded', stripsheet)
        #print "File " + stripsheet + " Downloaded"
        browser.retrieve(url, file)[0]

    if i < 1:
        logging.info('nothing to dowload')
        #print "nothing to dowload"
        #print "******************"
        break

#END Table parsing

#Reparse table to confirm
browser.follow_link(text="Data") #reload page to see changes

html = browser.response().read()
soup = BeautifulSoup(html)

table = soup.find("table")

browser.select_form(nr = 0)

i=1
while i >= 1:

    browser.follow_link(text="Data") #reload page to see changes

    html = browser.response().read()
    soup = BeautifulSoup(html)
    table = soup.find("table")
    browser.select_form(nr = 0)

    for form in browser.forms():

        for control in browser.form.controls:
            if control.type == "submit":
                i += 1
            else:
                i = 0

    if i == 0:
        logging.info('nothing to confirm')
        #print "nothing to confirm"
        #print "******************"
    else:
        logging.info('%s confirmed', control.name[0])
        #print control.name[0] + "confirmed"

    browser.submit(nr = 0)

logging.info('*** Batch generated and CSV file downloaded from Sense ***')
#print "*** done ***"

#END reparsin table to confirm

#Change mount for access to test, for NEW, processed ...
# Declase File paths

# Scrap file information for naming Naming

# Move and replace from New to sense processed folder

#Execute Job start by job_name

# Copy file to processed


#END Sense file downloading


# File paths
fullpath2Newfile = "/home/administrator/-share/New/"
fullpathfiles2Process = "/home/administrator/-share/"
fullpathProcessedfiles = "/home/administrator/-share/Processed/"

if not os.listdir('/home/administrator/cas01-share/New'):
    # print "directory empty"
        logging.info('No New files to upload')

else:
    #print "directory not empty"

    # Directory more than one file?
    num_files = len([f for f in os.listdir('/home/administrator/-share/New')
                if os.path.isfile(os.path.join('/home/administrator/-share/New', f))])
    #print "directory has " + str(num_files) + " files"

    # logging.info('More than one file in New Directory')
    logging.info('Directory has: %s Files', str(num_files))



    num_files = 0

    for filename in os.listdir('/home/administrator/-share/New'):
        succesfullRun = 0
        num_files += 1

        # print filename
        logging.info('File: %s', filename)

        filename = str(filename)
        batchnumber = filename.strip("['SENSE-extract-ascii .csv']")[:-9]
        print batchnumber

        newfullpath2Newfile = fullpath2Newfile + filename
        fullpathfiles2Process = "/home/administrator/-share/SENSE-extract.csv"
        fullpathProcessedfiles = "/home/administrator/-share/Processed/SENSE-extract-" + batchnumber + ".csv"
        print newfullpath2Newfile
        # print fullpathfiles2Process
        # print fullpathProcessedfiles


        # Execute job
        ExceptionFlag = 0
        while (succesfullRun == 0):

            if ExceptionFlag == 0:
                shutil.move(newfullpath2Newfile, fullpathfiles2Process)

            try:

                # Move to sense check folder
                # Connect to DB
                conn = pymssql.connect(server='server', user='domain\user', password='pwd', tds_version='8.0')
                cursor = conn.cursor()
                cursor.execute("EXEC msdb.dbo.sp_start_job @job_name = 'jobname'")

            except Exception as e:
                (error_message) = e
                #print(e)

                ExceptionFlag = 1

                print error_message
                #print error_message


                if "does not exist" in error_message:
                    # print "ERROR: Job name not found"
                    logging.info('ERROR: Job name not found')

                if "job is already running" in error_message:
                     #print "ERROR: a Job is already running"
                    logging.info('ERROR: a Job is already running')

                if "job already has a pending request" in error_message:
                    # print "ERROR: pending request"
                    logging.info('ERROR: pending request')

                if "Unable to connect" in error_message:
                    # print "ERROR: Server unavailable"
                    logging.info('ERROR: Server unavailable')

                time.sleep(0.5)


            else:
                # print "Job executed succesfully"
                logging.info('Job %s Executed', batchnumber)
                # Copy file to processed
                shutil.copy(fullpathfiles2Process, fullpathProcessedfiles)

                succesfullRun = 1

            #conn.close()
            time.sleep(0.5)


logging.info('Finished uploading jobs')
logging.info('END')
exit()

#END script