Namespacing causes sigterm on exit

[el3ment]

gazebo_ros_pkgs/gazebo_ros/src/gazebo_ros_api_plugin.cpp

Line 1985 in f03ddc2

physics_reconfigure_set_client_ = nh_->serviceClient<gazebo_msgs::SetPhysicsProperties>("/gazebo/set_physics_properties");

I think this line (and the next) are supposed to use the gazebo_namespace, not just be hardcoded to /gazebo

The issue is that when using a namespace, we get this

waitForService: Service [/gazebo/set_physics_properties] has not been advertised, waiting...

Because it will never be advertised. When killing the agent, gazebo waits for a long time, and eventually raises a sigterm and dies

[kev-the-dev]

Good catch. This actually would break the physics reconfigure feature when in a namespace. Fixed in #732