fuel-web tests leave running fakeui process

Bug #1335847 reported by Aleksandra Fedorova
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Dima Shulyak

Bug Description

See the log here https://fuel-jenkins.mirantis.com/job/verify-fuel-web/126/consoleFull

At the beginning of the test we have:

+ grep manage.py
jenkins 13130 0.0 0.0 11744 920 ? S 14:32 0:00 grep manage.py

And at the end:

+ grep manage.py
jenkins 18862 0.7 0.2 907476 42440 ? Sl 14:55 0:01 .tox/venv/bin/python manage.py run --port=5544 --config=/tmp/static_compressed/settings.yaml --fake-tasks --fake-tasks-tick-count=80 --fake-tasks-tick-interval=1
jenkins 18980 2.5 0.3 1054940 51036 ? Sl 14:55 0:06 .tox/venv/bin/python manage.py run --port=8003 --config= --fake-tasks --fake-tasks-tick-count=80 --fake-tasks-tick-interval=1
jenkins 23256 0.0 0.0 11748 924 ? S 14:59 0:00 grep manage.py

Running run_tests.sh with 'set -x' shows that first pid in a cycle is not catched by the grep filter.

Tags: fuel-ci
Revision history for this message
Evgeniy L (rustyrobot) wrote :

`run_server` function sometimes doesn't return pid of the process, because tox not always have time to run the sever [1], need to add retry mechanism when we retrieve pid of the process

[1] https://github.com/stackforge/fuel-web/blob/fd4203b1163705c19e6a8208f427893a5617790d/run_tests.sh#L482-L483

Changed in fuel:
status: New → Triaged
milestone: none → 5.1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/105409

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Dima Shulyak (dshulyak)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/105409
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=62226f26e58905a057d9462d958dfe19d1f4416a
Submitter: Jenkins
Branch: master

commit 62226f26e58905a057d9462d958dfe19d1f4416a
Author: Dima Shulyak <email address hidden>
Date: Tue Jul 8 12:57:40 2014 +0300

    Properly kill running server after tests

    - verify that server is up and running by http request,
      and not port scanning
    - look for process pid when verification is done for server
    - use lsof for server pid detection
    - fix return from function, it should be either echoi $pid in function,
      or echo $?

    Change-Id: I10b13eaec267dd7ccb779acc2d12200884074e1d
    Closes-Bug: #1335847

Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.