← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1857035] Re: Add better description for "test_cleanup_network_namespaces_cleans_dhcp_and_l3_namespaces" in case of failure

 

Reviewed:  https://review.opendev.org/700791
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a0fbb3da9bb53a675705f4a04b760017684ca37b
Submitter: Zuul
Branch:    master

commit a0fbb3da9bb53a675705f4a04b760017684ca37b
Author: Choi-Sung-Hoon <tjdgns9246@xxxxxxxxx>
Date:   Mon Dec 30 21:33:16 2019 +0000

    Modify exception string to have more information
    
    The original exception string was "Didn't spawn expected number
    of processes" and it's not enough to understand why the test
    failed.
    
    Improve test_cleanup_network_namespaces_cleans_dhcp_and_l3_namespaces()
    error message for timeout.
    
    The improved error message now has expected process number, spawned
    process number and process information.
    
    Change-Id: Ie0c27316f0e26b3770724a8bc97e46adf7e32d4c
    Closes-Bug: #1857035


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1857035

Title:
  Add better description for
  "test_cleanup_network_namespaces_cleans_dhcp_and_l3_namespaces" in
  case of failure

Status in neutron:
  Fix Released

Bug description:
  In case of failing, "test_cleanup_network_namespaces_cleans_dhcp_and_l3_namespaces" should provide a more detailed description:
  - How many processes are running.
  - Print in the exception the "ps" lines of those processes.

  This information can provide a better description of why the test
  failed.

  Example:
  https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c68/698853/2/gate
  /neutron-functional/c685963/testr_results.html.gz

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1857035/+subscriptions


References