fuel-web UI tests fail due to missing nailgun api.log

[Igor Belikov]

Hi,

fuel-web ui tests running in https://ci.fuel-infra.org/job/verify-fuel-web-ui/ job depend on /var/log/nailgun/api.log existence, but this log is not created or modified in any way during the job run.
This job is able to finish successfully only because there's some leftover api.log file on the slaves, e.g.:

root@ci-slave21:~# stat /var/log/nailgun/api.log
  File: ‘/var/log/nailgun/api.log’
  Size: 85863 Blocks: 168 IO Block: 4096 regular file
Device: fc01h/64513d Inode: 9045329 Links: 1
Access: (0664/-rw-rw-r--) Uid: ( 1000/ jenkins) Gid: ( 1000/ jenkins)
Access: 2015-04-22 11:22:12.870532338 +0000
Modify: 2015-04-22 11:30:24.855498805 +0000
Change: 2015-04-22 11:30:24.855498805 +0000

During test runs on freshly deployed slaves job fails due to missing api.log, see https://ci.fuel-infra.org/job/verify-fuel-web-ui/5142/

There are also a lot of deadlock warnings in server_log files created during the test, see http://paste.openstack.org/show/479969/

[Vitaly Kramskikh (vkramskikh)]

It seems the problem is that run_tests.sh overrides log location to a temp file - probably we need to use sinon for proper testing without log files available

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/262973
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=7e3efeb6ee11951310551a952ecc3a2ba8976461
Submitter: Jenkins
Branch: master

commit 7e3efeb6ee11951310551a952ecc3a2ba8976461
Author: Vitaly Kramskikh <email address hidden>
Date: Sat Jan 2 16:20:07 2016 +0300

    Use sinon for mocking server response in UI func tests for logs

    Closes-Bug: #1519752

    Change-Id: I62859dca7fa5c138b012d4c5fd4e184c941fb992

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/314572

[Alexey Stupnikov (astupnikov)]

This bug is a core reason of failures described in bug #1566730.

[Alexey Stupnikov (astupnikov)]

Georgy, proposed patch is breaking another tests and should be re-written. Can you do it?

[Georgy Kibardin (gkibardin)]

Sure I can, it'll just take longer.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/314967

[OpenStack Infra (hudson-openstack)]

Change abandoned by Alexey Stupnikov (<email address hidden>) on branch: stable/8.0
Review: https://review.openstack.org/314572
Reason: Fixed by https://review.openstack.org/#/c/314967

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/314967
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=3f03555f88df88ee92a940210c4cbcb5b5a7e35f
Submitter: Jenkins
Branch: stable/8.0

commit 3f03555f88df88ee92a940210c4cbcb5b5a7e35f
Author: Georgy Kibardin <email address hidden>
Date: Wed May 11 13:28:10 2016 +0300

    Log files paths in test configuration

    Test configuration now defines paths for logs so now log api works in
    test environment.

    Change-Id: I85b446ecca933feb44b1397f59e990452c67ab0f
    Closes-Bug: #1519752