Merge "refactor fake_pymongo.py to eliminate repeated codes"
authorMorgan Richomme <morgan.richomme@orange.com>
Tue, 31 May 2016 07:23:44 +0000 (07:23 +0000)
committerGerrit Code Review <gerrit@172.30.200.206>
Tue, 31 May 2016 07:23:44 +0000 (07:23 +0000)
jjb/yardstick/yardstick-daily.sh
utils/jenkins-jnlp-connect.sh
utils/test/result_collection_api/resources/pod_models.py
utils/test/result_collection_api/resources/project_models.py
utils/test/result_collection_api/resources/result_models.py
utils/test/result_collection_api/resources/testcase_models.py
utils/test/result_collection_api/tests/unit/test_dashboard.py
utils/test/result_collection_api/tests/unit/test_result.py

index 3ad6b7c..51fc9c8 100755 (executable)
@@ -6,7 +6,7 @@ set -e
 labconfig=""
 sshkey=""
 if [[ ${INSTALLER_TYPE} == 'apex' ]]; then
-    instack_mac=$(sudo virsh domiflist instack | grep default | \
+    instack_mac=$(sudo virsh domiflist undercloud | grep default | \
                   grep -Eo "[0-9a-f]+:[0-9a-f]+:[0-9a-f]+:[0-9a-f]+:[0-9a-f]+:[0-9a-f]+")
     INSTALLER_IP=$(/usr/sbin/arp -e | grep ${instack_mac} | awk {'print $1'})
     sshkey="-v /root/.ssh/id_rsa:/root/.ssh/id_rsa"
index 10ac50b..8c41620 100755 (executable)
 #Monit setup script for opnfv jnlp slave connections
 
 test_firewall() {
-echo "testing that the firewall is open for us at build.opnfv.org"
-test=$(echo "blah"| nc -w 4 build.opnfv.org 57387 > /dev/null 2>&1; echo $?)
-if [[ $test == 0 ]]; then
-  echo "Firewall is open for us at build.opnfv.org"
-  exit 0
-else
-cat << EOF
+    echo "testing that the firewall is open for us at build.opnfv.org"
+    test=$(echo "blah"| nc -w 4 build.opnfv.org 57387 > /dev/null 2>&1; echo $?)
+    if [[ $test == 0 ]]; then
+        echo "Firewall is open for us at build.opnfv.org"
+        exit 0
+    else
+        cat << EOF
 LF firewall not open, please send a report to helpdesk with your gpg key attached, or better yet upload it to the key servers. (I should be able to find it with gpg --search-keys your@company.email.com
 opnfv-helpdesk@rt.linuxfoundation.org
 Jenkins Home: $jenkinshome
@@ -25,126 +25,118 @@ Jenkins User: $jenkinsuser
 Slave Name: $slave_name
 IP Address: $(curl -s http://icanhazip.com)
 EOF
-exit 1
-fi
+        exit 1
+    fi
 }
 
 main () {
-dir=$(cd $(dirname $0); pwd)
-
-#tests
-if [[ -z $jenkinsuser || -z $jenkinshome ]]; then
-  echo "jenkinsuser or home not defined, please edit this file to define it"
-  exit 1
-fi
+    dir=$(cd $(dirname $0); pwd)
 
-if [[ -z $slave_name || -z $slave_secret ]]; then
-  echo "slave name or secret not defined, please edit this file to define it"
-  exit 1
-fi
-
-if [[ $(whoami) != "root" && $(whoami) != "$jenkinsuser"  ]]; then
-  echo "This script must be run as user root or jenkins user"
-  exit 1
-fi
+    #tests
+    if [[ -z $jenkinsuser || -z $jenkinshome ]]; then
+        echo "jenkinsuser or home not defined, please edit this file to define it"
+        exit 1
+    fi
 
-if [ -d /etc/monit/conf.d ]; then
-  monitconfdir="/etc/monit/conf.d/"
-elif [ -d /etc/monit.d ]; then
-  monitconfdir="/etc/monit.d"
-else
-  echo "Could not determine the location of the monit configuration file."
-  echo "Make sure monit is installed."
-  exit 1
-fi
+    if [[ -z $slave_name || -z $slave_secret ]]; then
+        echo "slave name or secret not defined, please edit this file to define it"
+        exit 1
+    fi
 
-#make pid dir
-pidfile="/var/run/$jenkinsuser/jenkins_jnlp_pid"
-if ! [ -d /var/run/$jenkinsuser/ ]; then
-  mkdir   /var/run/$jenkinsuser/
-  chown $jenkinsuser:$jenkinsuser /var/run/$jenkinsuser/
-fi
+    if [[ $(whoami) != "root" && $(whoami) != "$jenkinsuser"  ]]; then
+        echo "This script must be run as user root or jenkins user"
+        exit 1
+    fi
 
+    if [ -d /etc/monit/conf.d ]; then
+        monitconfdir="/etc/monit/conf.d/"
+    elif [ -d /etc/monit.d ]; then
+        monitconfdir="/etc/monit.d"
+    else
+        echo "Could not determine the location of the monit configuration file."
+        echo "Make sure monit is installed."
+        exit 1
+    fi
 
+    #make pid dir
+    pidfile="/var/run/$jenkinsuser/jenkins_jnlp_pid"
+    if ! [ -d /var/run/$jenkinsuser/ ]; then
+        mkdir /var/run/$jenkinsuser/
+        chown $jenkinsuser:$jenkinsuser /var/run/$jenkinsuser/
+    fi
 
-if [[ $skip_monit == true ]]; then
-  :
-else
-  #check for monit
-  if [ $(which monit) ]; then
-    echo "monit installed"
-  else
-    if [ -n "$(command -v yum)" ]; then
-      echo "please install monit; eg: yum -y install monit"
-      exit 1
-    elif [ -n "$(command -v apt-get)" ]; then
-      echo "please install monit  eg: apt-get install -y monit"
-      exit 1
-    else
-      echo "system not supported plese contact help desk"
-      exit 1
+    if [[ $skip_monit != true ]]; then
+        #check for monit
+        if [ $(which monit) ]; then
+            echo "monit installed"
+        else
+            if [ -n "$(command -v yum)" ]; then
+                echo "please install monit; eg: yum -y install monit"
+                exit 1
+            elif [ -n "$(command -v apt-get)" ]; then
+                echo "please install monit; eg: apt-get install -y monit"
+                exit 1
+            else
+                echo "system not supported plese contact help desk"
+                exit 1
+            fi
+        fi
     fi
-  fi
-fi
 
-makemonit () {
-echo "Writing the following as monit config:"
-cat << EOF | tee $monitconfdir/jenkins
+    makemonit () {
+        echo "Writing the following as monit config:"
+        cat << EOF | tee $monitconfdir/jenkins
 check process jenkins with pidfile /var/run/$jenkinsuser/jenkins_jnlp_pid
 start program = "/usr/bin/sudo -u $jenkinsuser /bin/bash -c 'cd $dir; export started_monit=true; $0 $@'"
 stop program = "/bin/bash -c '/bin/kill \$(/bin/cat /var/run/$jenkinsuser/jenkins_jnlp_pid)'"
 EOF
-}
+    }
 
-if [[ -f $monitconfdir/jenkins ]]; then
-  #test for diff
-  if [[ "$(diff $monitconfdir/jenkins <(echo "\
+    if [[ -f $monitconfdir/jenkins ]]; then
+        #test for diff
+        if [[ "$(diff $monitconfdir/jenkins <(echo "\
 check process jenkins with pidfile /var/run/$jenkinsuser/jenkins_jnlp_pid
 start program = \"/usr/bin/sudo -u $jenkinsuser /bin/bash -c 'cd $dir; export started_monit=true; $0 $@'\"
 stop program = \"/bin/bash -c '/bin/kill \$(/bin/cat /var/run/$jenkinsuser/jenkins_jnlp_pid)'\"\
 ") )" ]]; then
-    echo "Updating monit config..."
-    makemonit $@
-  fi
-else
-  makemonit $@
-fi
-
+            echo "Updating monit config..."
+            makemonit $@
+        fi
+    else
+        makemonit $@
+    fi
 
 if [[ $started_monit == "true" ]]; then
-wget --timestamping https://build.opnfv.org/ci/jnlpJars/slave.jar && true
-chown $jenkinsuser:$jenkinsuser slave.jar
+    wget --timestamping https://build.opnfv.org/ci/jnlpJars/slave.jar && true
+    chown $jenkinsuser:$jenkinsuser slave.jar
 
-if [[ -f /var/run/$jenkinsuser/jenkins_jnlp_pid ]];
-then echo "pid file found"
+    if [[ -f /var/run/$jenkinsuser/jenkins_jnlp_pid ]]; then
+        echo "pid file found"
         if ! kill -0 "$(/bin/cat /var/run/$jenkinsuser/jenkins_jnlp_pid)"; then
-                echo "no java process running cleaning up pid file"
-                rm -f /var/run/$jenkinsuser/jenkins_jnlp_pid;
+            echo "no java process running cleaning up pid file"
+            rm -f /var/run/$jenkinsuser/jenkins_jnlp_pid;
         else
-                echo "java connection process found and running already running quitting."
-                exit 1
-
+            echo "java connection process found and running already running quitting."
+            exit 1
         fi
-fi
-
-if [[ $run_in_foreground == true ]]; then
-  $connectionstring
-else
-  exec $connectionstring &
-  echo $! > /var/run/$jenkinsuser/jenkins_jnlp_pid
-fi
+    fi
 
+    if [[ $run_in_foreground == true ]]; then
+        $connectionstring
+    else
+        exec $connectionstring &
+        echo $! > /var/run/$jenkinsuser/jenkins_jnlp_pid
+    fi
 else
-  echo "you are ready to start monit"
-  echo "eg: service monit start"
-  echo "example debug mode if you are having problems:  /usr/bin/monit -Ivv -c /etc/monit.conf "
-  exit 0
+    echo "you are ready to start monit"
+    echo "eg: service monit start"
+    echo "example debug mode if you are having problems:  /usr/bin/monit -Ivv -c /etc/monit.conf "
+    exit 0
 fi
-
 }
 
 usage() {
-cat << EOF
+    cat << EOF
 
 jenkins-jnlp-connect.sh configures monit to keep slave connection up
 Checks for new versions of slave.jar
@@ -162,31 +154,29 @@ usage: $0 [OPTIONS]
  -f  test firewall
 
 Example: $0 -j /home/jenkins/ -u jenkins -n lab1 -s 727fdefoofoofoofoofoofoofof800
-
 EOF
 
-exit 1
-
+    exit 1
 }
 
-if [[ -z "$@" ]]; then usage
+if [[ -z "$@" ]]; then
+    usage
 fi
 
-
 while getopts "j:u:n:s:htf" OPTION
 do
-        case $OPTION in
-                j ) jenkinshome="$OPTARG" ;;
-                u ) jenkinsuser="$OPTARG" ;;
-                n ) slave_name="$OPTARG" ;;
-                s ) slave_secret="$OPTARG";;
-                h ) usage; exit;;
-                t ) started_monit=true
-                    skip_monit=true
-                    run_in_foreground=true ;;
-                f ) test_firewall ;;
-                \? ) echo "Unknown option: -$OPTARG" >&2; exit 1;;
-        esac
+    case $OPTION in
+        j ) jenkinshome="$OPTARG" ;;
+        u ) jenkinsuser="$OPTARG" ;;
+        n ) slave_name="$OPTARG" ;;
+        s ) slave_secret="$OPTARG";;
+        h ) usage ;;
+        t ) started_monit=true
+            skip_monit=true
+            run_in_foreground=true ;;
+        f ) test_firewall ;;
+        \? ) echo "Unknown option: -$OPTARG" >&2; exit 1;;
+    esac
 done
 
 connectionstring="java -jar slave.jar -jnlpUrl https://build.opnfv.org/ci/computer/"$slave_name"/slave-agent.jnlp -secret "$slave_secret" -noCertificateCheck "
index cc98c90..0ccb661 100644 (file)
@@ -62,10 +62,11 @@ class Pod(PodCreateRequest):
 @swagger.model()
 class Pods(object):
     """
+        @property pods:
         @ptype pods: C{list} of L{Pod}
     """
-    def __init__(self, pods=list()):
-        self.pods = pods
+    def __init__(self):
+        self.pods = list()
 
     @staticmethod
     def from_dict(res_dict):
index a1592c3..1dc0c40 100644 (file)
@@ -71,10 +71,11 @@ class Project(object):
 @swagger.model()
 class Projects(object):
     """
+        @property projects:
         @ptype projects: C{list} of L{Project}
     """
-    def __init__(self, projects=list()):
-        self.projects = projects
+    def __init__(self):
+        self.projects = list()
 
     @staticmethod
     def from_dict(res_dict):
index 7faac16..27c3542 100644 (file)
@@ -48,20 +48,24 @@ class ResultCreateRequest(object):
 
 @swagger.model()
 class TestResult(object):
-    def __init__(self):
-        self._id = None
-        self.case_name = None
-        self.project_name = None
-        self.pod_name = None
-        self.installer = None
-        self.version = None
-        self.start_date = None
-        self.stop_date = None
-        self.details = None
-        self.build_tag = None
-        self.scenario = None
-        self.criteria = None
-        self.trust_indicator = None
+    def __init__(self, _id=None, case_name=None, project_name=None,
+                 pod_name=None, installer=None, version=None,
+                 start_date=None, stop_date=None, details=None,
+                 build_tag=None, scenario=None, criteria=None,
+                 trust_indicator=None):
+        self._id = _id
+        self.case_name = case_name
+        self.project_name = project_name
+        self.pod_name = pod_name
+        self.installer = installer
+        self.version = version
+        self.start_date = start_date
+        self.stop_date = stop_date
+        self.details = details
+        self.build_tag = build_tag
+        self.scenario = scenario
+        self.criteria = criteria
+        self.trust_indicator = trust_indicator
 
     @staticmethod
     def from_dict(a_dict):
@@ -137,10 +141,11 @@ class TestResult(object):
 @swagger.model()
 class TestResults(object):
     """
-        @ptype testcases: C{list} of L{TestResult}
+        @property results:
+        @ptype results: C{list} of L{TestResult}
     """
-    def __init__(self, results=list()):
-        self.results = results
+    def __init__(self):
+        self.results = list()
 
     @staticmethod
     def from_dict(a_dict):
index 90b3f75..a69f2f7 100644 (file)
@@ -82,10 +82,11 @@ class Testcase(object):
 @swagger.model()
 class Testcases(object):
     """
+        @property testcases:
         @ptype testcases: C{list} of L{Testcase}
     """
-    def __init__(self, testcases=list()):
-        self.testcases = testcases
+    def __init__(self):
+        self.testcases = list()
 
     @staticmethod
     def from_dict(res_dict):
index 3e71333..5c3f924 100644 (file)
@@ -44,7 +44,6 @@ class TestDashboardQuery(TestDashboardBase):
     def test_success(self):
         code, body = self.query(self._set_query())
         self.assertEqual(code, HTTP_OK)
-        print(body)
         self.assertIn('{"description": "vPing results for Dashboard"}', body)
 
     def test_caseIsStatus(self):
index f08ccea..7829d7c 100644 (file)
@@ -234,6 +234,7 @@ class TestResultGet(TestResultBase):
             self.assertEqual(0, len(body.results))
         else:
             for result in body.results:
+                self.assertEqual(1, len(body.results))
                 self.assert_res(code, result)
 
     def _set_query(self, *args):