Fix spawner behaviour on failing controller activation or deactivation

[saikishor]

Before the fix proposed in this PR, the spawner reports the controller activation as successful, if the activation is failed at the last stage after calling activate_controllers. This PR addresses the issue by proposing a fix and this should now properly fail at the spawner end as well

Before the fix:

1: [INFO] [1734338012.820800005] [test_controller_manager]: Activating controllers: [ ctrl_with_joint1_and_joint2_command_interfaces ]
1: [ERROR] [1734338012.829138606] [test_controller_manager]: Resource conflict for controller 'ctrl_with_joint1_and_joint2_command_interfaces'. Command interface 'joint1/position' is already claimed.
1: [INFO] [1734338012.830469222] [spawner_ctrl_with_joint1_and_joint2_command_interfaces]: Configured and activated ctrl_with_joint1_and_joint2_command_interfaces

With the proposed fix:

1: [INFO] [1734338233.806072093] [test_controller_manager]: Activating controllers: [ ctrl_with_joint1_and_joint2_command_interfaces ]
1: [ERROR] [1734338233.814172942] [test_controller_manager]: Resource conflict for controller 'ctrl_with_joint1_and_joint2_command_interfaces'. Command interface 'joint1/position' is already claimed.
1: [ERROR] [1734338233.814291503] [test_controller_manager]: Could not activate controller : 'ctrl_with_joint1_and_joint2_command_interfaces'
1: [ERROR] [1734338233.817168509] [spawner_ctrl_with_joint1_and_joint2_command_interfaces]: Failed to activate controller : ctrl_with_joint1_and_joint2_command_interfaces

[codecov]

Codecov Report

Attention: Patch coverage is 80.35714% with 11 lines in your changes missing coverage. Please review.

Project coverage is 88.20%. Comparing base (612b30b) to head (b436a4e).

Files with missing lines	Patch %	Lines
controller_manager/src/controller_manager.cpp	63.15%	5 Missing and 2 partials ⚠️
...r_manager/test/test_controller/test_controller.cpp	75.00%	1 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1941      +/-   ##
==========================================
+ Coverage   88.13%   88.20%   +0.06%     
==========================================
  Files         121      121              
  Lines       13163    13215      +52     
  Branches     1183     1189       +6     
==========================================
+ Hits        11601    11656      +55     
+ Misses       1129     1126       -3     
  Partials      433      433              

Flag	Coverage Δ
unittests	88.20% <80.35%> (+0.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
controller_manager/controller_manager/spawner.py	74.81% <ø> (+2.22%)	⬆️
...ontroller_manager/test/test_release_interfaces.cpp	87.50% <ø> (ø)
controller_manager/test/test_spawner_unspawner.cpp	96.72% <100.00%> (+0.16%)	⬆️
...r_manager/test/test_controller/test_controller.cpp	88.05% <75.00%> (-4.10%)	⬇️
controller_manager/src/controller_manager.cpp	76.20% <63.15%> (-0.13%)	⬇️

... and 2 files with indirect coverage changes

[christophfroehlich]

LGTM (At least the test you've written makes totally sense.)

[bmagyar]

Thank you!!