Always stop rbe and collect logs

Existing logic stops rbe only when build succeeds. This CL ensures we shut down rbe and collects logs all the time.

One example this cl will help:
https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20Engine%20Drone/631801/overview

Change-Id: Ic3d5b049b01e92fa1166656ac50c9f537171d9da
Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/54100
Reviewed-by: Ricardo Amador <ricardoamador@google.com>
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Keyong Han <keyonghan@google.com>

recipe_modules/rbe/api.py

diff --git a/recipe_modules/rbe/api.py b/recipe_modules/rbe/api.py
index cfeb5b9..280f900 100644
--- a/recipe_modules/rbe/api.py
+++ b/recipe_modules/rbe/api.py
@@ -73,8 +73,7 @@
         with self.m.context(infra_steps=is_infra_step):
           yield
       finally:
-        if not self.m.runtime.in_global_shutdown:
-          self._stop(working_dir=working_dir, config_path=config_path)
+        self._stop(working_dir=working_dir, config_path=config_path)
 
   @property
   def _ensure_reclient_path(self):