diff options
8 files changed, 135 insertions, 137 deletions
diff --git a/jjb/yardstick/yardstick-daily.sh b/jjb/yardstick/yardstick-daily.sh index 3ad6b7c7c..51fc9c823 100755 --- a/jjb/yardstick/yardstick-daily.sh +++ b/jjb/yardstick/yardstick-daily.sh @@ -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" diff --git a/utils/jenkins-jnlp-connect.sh b/utils/jenkins-jnlp-connect.sh index 10ac50b0a..8c41620d6 100755 --- a/utils/jenkins-jnlp-connect.sh +++ b/utils/jenkins-jnlp-connect.sh @@ -11,13 +11,13 @@ #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 " diff --git a/utils/test/result_collection_api/resources/pod_models.py b/utils/test/result_collection_api/resources/pod_models.py index cc98c90ce..0ccb66162 100644 --- a/utils/test/result_collection_api/resources/pod_models.py +++ b/utils/test/result_collection_api/resources/pod_models.py @@ -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): diff --git a/utils/test/result_collection_api/resources/project_models.py b/utils/test/result_collection_api/resources/project_models.py index a1592c31d..1dc0c40e5 100644 --- a/utils/test/result_collection_api/resources/project_models.py +++ b/utils/test/result_collection_api/resources/project_models.py @@ -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): diff --git a/utils/test/result_collection_api/resources/result_models.py b/utils/test/result_collection_api/resources/result_models.py index 7faac16d4..27c354220 100644 --- a/utils/test/result_collection_api/resources/result_models.py +++ b/utils/test/result_collection_api/resources/result_models.py @@ -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): diff --git a/utils/test/result_collection_api/resources/testcase_models.py b/utils/test/result_collection_api/resources/testcase_models.py index 90b3f75e9..a69f2f751 100644 --- a/utils/test/result_collection_api/resources/testcase_models.py +++ b/utils/test/result_collection_api/resources/testcase_models.py @@ -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): diff --git a/utils/test/result_collection_api/tests/unit/test_dashboard.py b/utils/test/result_collection_api/tests/unit/test_dashboard.py index 3e713332c..5c3f92492 100644 --- a/utils/test/result_collection_api/tests/unit/test_dashboard.py +++ b/utils/test/result_collection_api/tests/unit/test_dashboard.py @@ -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): diff --git a/utils/test/result_collection_api/tests/unit/test_result.py b/utils/test/result_collection_api/tests/unit/test_result.py index f08ccea83..7829d7c6a 100644 --- a/utils/test/result_collection_api/tests/unit/test_result.py +++ b/utils/test/result_collection_api/tests/unit/test_result.py @@ -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): |