TheadGroupManager.start_with_acquired_lock does not pass keyword arguments to the function being started

[Pavlo Shchelokovskyy]

It went unnoticed since we did not launch any functions requiring kwargs with it so far. I stumbled upon it when working on blueprint on cancelling of stack update. Also the ThreadGroupManager lacks unit tests, which also contributed to this mistake not being caught earlier.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/76251
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=0f8be1dbcf9e641d5192fc5ce85990fd580083a0
Submitter: Jenkins
Branch: master

commit 0f8be1dbcf9e641d5192fc5ce85990fd580083a0
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Tue Feb 25 15:52:47 2014 +0000

    Fix start_with_acquired_lock - pass kwargs to func

    This method of ThreadGroupManager was not passing kwargs
    to the function to be started.

    Closes-Bug: #1288627
    Change-Id: Icaaa012bcc65c67f3e8ecc2878dee2a78121e357

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/78286
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=16840f8585034f8fbe92b1420909da8d59e4fc2e
Submitter: Jenkins
Branch: master

commit 16840f8585034f8fbe92b1420909da8d59e4fc2e
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Wed Mar 5 18:15:09 2014 +0200

    Add more unit tests for ThreadGroupManager

    this patch adds unit tests for the rest of ThreadGroupManager methods.
    Related-Bug: #1288627
    Change-Id: I38a0864c649d1693ef4fb5b49d5aeb37632263ca