summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorRyota MIBU <r-mibu@cq.jp.nec.com>2017-03-03 06:58:42 +0000
committerRyota MIBU <r-mibu@cq.jp.nec.com>2017-03-04 23:00:32 +0000
commit8f72e695538c2281f923bdbf7a6d7b1d1763c70c (patch)
tree6e5f9791ab340f0008cda575c605c47e64f97958 /tests
parentf093b7f19e8cd6f76b26117b18b3c9e552ad5bcc (diff)
urgent bug fixes for danube (1)
- create "doctor_net" and use it in VM creation in order to avoid virtual network issues - make sample inspector use keystone v3 - use "openstack compute service set" for unset force_down - drop nova_force_down.py as keystone returns nova v2.1 - use '--os-project-name' except for ceilometer cli - avoid ping check (wait_ping) with empty "COMPUTE_IP" - add missing '$' in quota update args - move ping check (wait_ping) into unset_forced_down_hosts() and perform ping check only to downed compute hosts - use wait_until() instead of wait_ping() - put out unset_forced_down_hosts from collect_logs() JIRA: DOCTOR-95 Co-Authored-By: Carlos Goncalves <carlos.goncalves@neclab.eu> Change-Id: I3275ff5dd993b82029dac6a58087096baa251022 Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/inspector.py31
-rw-r--r--tests/lib/inspectors/sample4
-rw-r--r--tests/monitor.py8
-rw-r--r--tests/nova_force_down.py60
-rwxr-xr-xtests/run.sh95
5 files changed, 74 insertions, 124 deletions
diff --git a/tests/inspector.py b/tests/inspector.py
index 7195969a..27c969e5 100644
--- a/tests/inspector.py
+++ b/tests/inspector.py
@@ -17,10 +17,10 @@ import os
import threading
import time
+from keystoneauth1.identity import v3
+from keystoneauth1 import session
import novaclient.client as novaclient
-import nova_force_down
-
LOG = doctor_log.Logger('doctor_inspector').getLogger()
@@ -39,7 +39,7 @@ class ThreadedResetState(threading.Thread):
class DoctorInspectorSample(object):
- NOVA_API_VERSION = '2.11'
+ NOVA_API_VERSION = '2.34'
NUMBER_OF_CLIENTS = 50
# TODO(tojuvone): This could be enhanced in future with dynamic
# reuse of self.novaclients when all threads in use and
@@ -49,14 +49,18 @@ class DoctorInspectorSample(object):
def __init__(self):
self.servers = collections.defaultdict(list)
self.novaclients = list()
+ auth = v3.Password(auth_url=os.environ['OS_AUTH_URL'],
+ username=os.environ['OS_USERNAME'],
+ password=os.environ['OS_PASSWORD'],
+ user_domain_name=os.environ['OS_USER_DOMAIN_NAME'],
+ project_name=os.environ['OS_PROJECT_NAME'],
+ project_domain_name=os.environ['OS_PROJECT_DOMAIN_NAME'])
+ sess=session.Session(auth=auth)
# Pool of novaclients for redundant usage
for i in range(self.NUMBER_OF_CLIENTS):
- self.novaclients.append(novaclient.Client(self.NOVA_API_VERSION,
- os.environ['OS_USERNAME'],
- os.environ['OS_PASSWORD'],
- os.environ['OS_TENANT_NAME'],
- os.environ['OS_AUTH_URL'],
- connection_pool=True))
+ self.novaclients.append(
+ novaclient.Client(self.NOVA_API_VERSION, session=sess,
+ connection_pool=True))
# Normally we use this client for non redundant API calls
self.nova=self.novaclients[0]
self.nova.servers.list(detailed=False)
@@ -87,14 +91,7 @@ class DoctorInspectorSample(object):
threads.append(t)
for t in threads:
t.join()
- # NOTE: We use our own client here instead of this novaclient for a
- # workaround. Once keystone provides v2.1 nova api endpoint
- # in the service catalog which is configured by OpenStack
- # installer, we can use this:
- #
- # self.nova.services.force_down(hostname, 'nova-compute', True)
- #
- nova_force_down.force_down(hostname)
+ self.nova.services.force_down(hostname, 'nova-compute', True)
LOG.info('doctor mark host(%s) down at %s' % (hostname, time.time()))
diff --git a/tests/lib/inspectors/sample b/tests/lib/inspectors/sample
index d09f7ad0..75d5c022 100644
--- a/tests/lib/inspectors/sample
+++ b/tests/lib/inspectors/sample
@@ -1,9 +1,9 @@
#!/bin/bash
+INSPECTOR_PORT=${INSPECTOR_PORT:-12345}
+
function start_inspector_sample {
pgrep -f "python inspector.py" && return 0
- # TODO (r-mibu): make sample inspector use keystone v3 api
- OS_AUTH_URL=${OS_AUTH_URL/v3/v2.0} \
python inspector.py "$INSPECTOR_PORT" > inspector.log 2>&1 &
}
diff --git a/tests/monitor.py b/tests/monitor.py
index 75d82392..8d29dceb 100644
--- a/tests/monitor.py
+++ b/tests/monitor.py
@@ -40,11 +40,12 @@ class DoctorMonitorSample(object):
raise Exception("Inspector type '%s' not supported", args.inspector_type)
self.hostname = args.hostname
- self.inspector_url = args.inspector_url
self.inspector_type = args.inspector_type
self.ip_addr = args.ip or socket.gethostbyname(self.hostname)
- if self.inspector_type == 'congress':
+ if self.inspector_type == 'sample':
+ self.inspector_url = 'http://127.0.0.1:12345/events'
+ elif self.inspector_type == 'congress':
auth = v2.Password(auth_url=os.environ['OS_AUTH_URL'],
username=os.environ['OS_USERNAME'],
password=os.environ['OS_PASSWORD'],
@@ -114,9 +115,6 @@ def get_args():
parser.add_argument('inspector_type', metavar='INSPECTOR_TYPE', type=str, nargs='?',
help='inspector to report',
default='sample')
- parser.add_argument('inspector_url', metavar='INSPECTOR_URL', type=str, nargs='?',
- help='inspector url to report error',
- default='http://127.0.0.1:12345/events')
return parser.parse_args()
diff --git a/tests/nova_force_down.py b/tests/nova_force_down.py
deleted file mode 100644
index abea5671..00000000
--- a/tests/nova_force_down.py
+++ /dev/null
@@ -1,60 +0,0 @@
-##############################################################################
-# Copyright (c) 2016 NEC Corporation and others.
-#
-# All rights reserved. This program and the accompanying materials
-# are made available under the terms of the Apache License, Version 2.0
-# which accompanies this distribution, and is available at
-# http://www.apache.org/licenses/LICENSE-2.0
-##############################################################################
-
-import argparse
-import json
-import os
-
-from keystoneclient.v2_0 import client
-import requests
-
-
-def force_down(hostname, force_down=True):
- keystone = client.Client(username=os.environ['OS_USERNAME'],
- password=os.environ['OS_PASSWORD'],
- tenant_name=os.environ['OS_TENANT_NAME'],
- auth_url=os.environ['OS_AUTH_URL'])
-
- for service in keystone.auth_ref['serviceCatalog']:
- if service['type'] == 'compute':
- base_url = service['endpoints'][0]['internalURL']
- break
-
- url = '%s/os-services/force-down' % base_url.replace('/v2/', '/v2.1/')
- data = {
- 'forced_down': force_down,
- 'binary': 'nova-compute',
- 'host': hostname,
- }
- headers = {
- 'Content-Type': 'application/json',
- 'Accept': 'application/json',
- 'X-Auth-Token': keystone.auth_ref['token']['id'],
- 'X-OpenStack-Nova-API-Version': '2.11',
- }
-
- print requests.put(url, data=json.dumps(data), headers=headers)
-
-
-def get_args():
- parser = argparse.ArgumentParser(description='Doctor Test Cleaner')
- parser.add_argument('hostname', metavar='HOSTNAME', type=str, nargs='?',
- help='a nova-compute hostname to force down')
- parser.add_argument('--unset', action='store_true', default=False,
- help='unset force_down flag')
- return parser.parse_args()
-
-
-def main():
- args = get_args()
- force_down(args.hostname, not(args.unset))
-
-
-if __name__ == '__main__':
- main()
diff --git a/tests/run.sh b/tests/run.sh
index 37dbaf4a..d97a5c9c 100755
--- a/tests/run.sh
+++ b/tests/run.sh
@@ -21,8 +21,9 @@ VM_BASENAME=doctor_vm
VM_FLAVOR=m1.tiny
#if VM_COUNT set, use that instead
VM_COUNT=${VM_COUNT:-1}
+NET_NAME=doctor_net
+NET_CIDR=192.168.168.0/24
ALARM_BASENAME=doctor_alarm
-INSPECTOR_PORT=12345
CONSUMER_PORT=12346
DOCTOR_USER=doctor
DOCTOR_PW=doctor
@@ -34,7 +35,10 @@ PROFILER_TYPE=${PROFILER_TYPE:-none}
TOP_DIR=$(cd $(dirname "$0") && pwd)
as_doctor_user="--os-username $DOCTOR_USER --os-password $DOCTOR_PW
- --os-tenant-name $DOCTOR_PROJECT"
+ --os-project-name $DOCTOR_PROJECT --os-tenant-name $DOCTOR_PROJECT"
+# NOTE: ceilometer command still requires '--os-tenant-name'.
+#ceilometer="ceilometer ${as_doctor_user/--os-project-name/--os-tenant-name}"
+ceilometer="ceilometer $as_doctor_user"
# Functions
@@ -110,17 +114,17 @@ create_test_user() {
openstack user create "$DOCTOR_USER" --password "$DOCTOR_PW" \
--project "$DOCTOR_PROJECT"
}
- openstack user role list "$DOCTOR_USER" --project "$DOCTOR_PROJECT" \
- | grep -q " $DOCTOR_ROLE " || {
- openstack role add "$DOCTOR_ROLE" --user "$DOCTOR_USER" \
- --project "$DOCTOR_PROJECT"
+ openstack role show "$DOCTOR_ROLE" || {
+ openstack role create "$DOCTOR_ROLE"
}
+ openstack role add "$DOCTOR_ROLE" --user "$DOCTOR_USER" \
+ --project "$DOCTOR_PROJECT"
# tojuvone: openstack quota show is broken and have to use nova
# https://bugs.launchpad.net/manila/+bug/1652118
# Note! while it is encouraged to use openstack client it has proven
# quite buggy.
# QUOTA=$(openstack quota show $DOCTOR_PROJECT)
- DOCTOR_QUOTA=$(nova quota-show --tenant DOCTOR_PROJECT)
+ DOCTOR_QUOTA=$(nova quota-show --tenant $DOCTOR_PROJECT)
# We make sure that quota allows number of instances and cores
OLD_INSTANCE_QUOTA=$(echo "${DOCTOR_QUOTA}" | grep " instances " | \
awk '{print $4}')
@@ -138,26 +142,35 @@ create_test_user() {
boot_vm() {
# test VM done with test user, so can test non-admin
+
+ if ! openstack $as_doctor_user network show $NET_NAME; then
+ openstack $as_doctor_user network create $NET_NAME
+ fi
+ if ! openstack $as_doctor_user subnet show $NET_NAME; then
+ openstack $as_doctor_user subnet create $NET_NAME \
+ --network $NET_NAME --subnet-range $NET_CIDR --no-dhcp
+ fi
+ net_id=$(openstack $as_doctor_user network show $NET_NAME -f value -c id)
+
servers=$(openstack $as_doctor_user server list)
for i in `seq $VM_COUNT`; do
echo "${servers}" | grep -q " $VM_BASENAME$i " && continue
openstack $as_doctor_user server create --flavor "$VM_FLAVOR" \
- --image "$IMAGE_NAME" \
- "$VM_BASENAME$i"
+ --image "$IMAGE_NAME" --nic net-id=$net_id "$VM_BASENAME$i"
done
sleep 1
}
create_alarm() {
# get vm_id as test user
- alarm_list=$(ceilometer $as_doctor_user alarm-list)
+ alarm_list=$($ceilometer alarm-list)
vms=$(openstack $as_doctor_user server list)
for i in `seq $VM_COUNT`; do
echo "${alarm_list}" | grep -q " $ALARM_BASENAME$i " || {
vm_id=$(echo "${vms}" | grep " $VM_BASENAME$i " | awk '{print $2}')
# TODO(r-mibu): change notification endpoint from localhost to the
# consumer. IP address (functest container).
- ceilometer $as_doctor_user alarm-event-create \
+ $ceilometer alarm-event-create \
--name "$ALARM_BASENAME$i" \
--alarm-action "http://localhost:$CONSUMER_PORT/failure" \
--description "VM failure" \
@@ -174,7 +187,7 @@ create_alarm() {
start_monitor() {
pgrep -f "python monitor.py" && return 0
sudo -E python monitor.py "$COMPUTE_HOST" "$COMPUTE_IP" "$INSPECTOR_TYPE" \
- "http://127.0.0.1:$INSPECTOR_PORT/events" > monitor.log 2>&1 &
+ > monitor.log 2>&1 &
}
stop_monitor() {
@@ -300,19 +313,6 @@ calculate_notification_time() {
}'
}
-wait_ping() {
- local interval=5
- local rounds=$(($1 / $interval))
- for i in `seq $rounds`; do
- ping -c 1 "$COMPUTE_IP"
- if [[ $? -ne 0 ]] ; then
- sleep $interval
- continue
- fi
- return 0
- done
-}
-
check_host_status() {
# Check host related to first Doctor VM is in wanted state
# $1 Expected state
@@ -340,25 +340,36 @@ check_host_status() {
}
unset_forced_down_hosts() {
- for host in $(openstack compute service list --service nova-compute \
- -f value -c Host -c State | sed -n -e '/down$/s/ *down$//p')
+ # for debug
+ openstack compute service list --service nova-compute
+
+ downed_computes=$(openstack compute service list --service nova-compute \
+ -f value -c Host -c State | grep ' down$' \
+ | sed -e 's/ *down$//')
+ echo "downed_computes: $downed_computes"
+ for host in $downed_computes
do
- # TODO (r-mibu): make sample inspector use keystone v3 api
- OS_AUTH_URL=${OS_AUTH_URL/v3/v2.0} \
- python ./nova_force_down.py $host --unset
+ # TODO(r-mibu): use openstack client
+ #openstack compute service set --up $host nova-compute
+ nova service-force-down --unset $host nova-compute
done
echo "waiting disabled compute host back to be enabled..."
wait_until 'openstack compute service list --service nova-compute
-f value -c State | grep -q down' 240 5
+
+ for host in $downed_computes
+ do
+ # TODO(r-mibu): improve 'get_compute_ip_from_hostname'
+ get_compute_ip_from_hostname $host
+ wait_until "! ping -c 1 $COMPUTE_IP" 120 5
+ done
}
collect_logs() {
- unset_forced_down_hosts
- # TODO: We need to make sure the target compute host is back to IP
- # reachable. wait_ping() will be added by tojuvone .
- sleep 110
- scp $ssh_opts_cpu "$COMPUTE_USER@$COMPUTE_IP:disable_network.log" .
+ if [[ -n "$COMPUTE_IP" ]];then
+ scp $ssh_opts_cpu "$COMPUTE_USER@$COMPUTE_IP:disable_network.log" .
+ fi
# TODO(yujunz) collect other logs, e.g. nova, aodh
}
@@ -398,10 +409,8 @@ cleanup() {
stop_consumer
unset_forced_down_hosts
+ collect_logs
- wait_ping 120
-
- scp $ssh_opts_cpu "$COMPUTE_USER@$COMPUTE_IP:disable_network.log" .
vms=$(openstack $as_doctor_user server list)
vmstodel=""
for i in `seq $VM_COUNT`; do
@@ -409,12 +418,15 @@ cleanup() {
vmstodel+=" $VM_BASENAME$i"
done
[[ $vmstodel ]] && openstack $as_doctor_user server delete $vmstodel
- alarm_list=$(ceilometer $as_doctor_user alarm-list)
+ alarm_list=$($ceilometer alarm-list)
for i in `seq $VM_COUNT`; do
alarm_id=$(echo "${alarm_list}" | grep " $ALARM_BASENAME$i " |
awk '{print $2}')
- [ -n "$alarm_id" ] && ceilometer $as_doctor_user alarm-delete "$alarm_id"
+ [ -n "$alarm_id" ] && $ceilometer alarm-delete "$alarm_id"
done
+ openstack $as_doctor_user subnet delete $NET_NAME
+ sleep 1
+ openstack $as_doctor_user network delete $NET_NAME
sleep 1
image_id=$(openstack image list | grep " $IMAGE_NAME " | awk '{print $2}')
@@ -427,6 +439,8 @@ cleanup() {
--project "$DOCTOR_PROJECT"
openstack project delete "$DOCTOR_PROJECT"
openstack user delete "$DOCTOR_USER"
+ # NOTE: remove role only for doctor test.
+ #openstack role delete "$DOCTOR_ROLE"
cleanup_installer
cleanup_inspector
@@ -481,6 +495,7 @@ inject_failure
check_host_status "(DOWN|UNKNOWN)" 60
calculate_notification_time
+unset_forced_down_hosts
collect_logs
run_profiler