yahoo-eng-team team mailing list archive

[OpenStack Infra <1852869@xxxxxxxxxxxxxxxxxx>]

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

commit 728d8ee4b1b9fd6a99ebbd66a0e30e81a078a5b5
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Tue Nov 19 17:27:38 2019 +0000

    NetcatTester.stop_processes skip "No such process" exception
    
    If any of the processes, client or server, spawned by NetcatTester is
    not present during the stop command (kill signal sent), by default the
    method will not raise an exception.
    
    Change-Id: If8cf47a01dc353734ad07ca6cd4db7bec6c90fb6
    Closes-Bug: #1852869


** 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/1852869

Title:
  NetcatTester.stop_processes should handle, if needed, the case when
  the process does not exist

Status in neutron:
  Fix Released

Bug description:
  In [1], the process to be stopped does not exist. This situation
  raises an exception not handled in upper methods and the test case
  logic fails.

  In the related test case:

  ConnectionTester.assert_connection --> _test_transport_connectivity
  --> stop_processes

  The first method (assert_no_connection) catches
  ConnectionTesterException but not ProcessExecutionError.

  
  [1] https://8244a21336f288b0d527-4f28a4e2359819cc7c03faeac91c9897.ssl.cf5.rackcdn.com/683273/11/check/neutron-functional/1368ac6/testr_results.html.gz

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