feat: pass verify_completion_state to verify_deploy controller callbacks.

Signed-off-by: Yuki Ito <yuki462@ibm.com>

Author: yuki462-b

oper8/controller.py

@@ -190,10 +190,10 @@ def after_deploy_unsuccessful(self, session: Session, failed: bool, deploy_compl
         """
         return True
 
-    # TODO get verify_completion_state
     def after_verify(
         self,
         session: Session,  # pylint: disable=unused-argument
+        verify_completion_state: CompletionState
     ) -> bool:
         """This allows children to inject logic that will run when the
         controller has finished verifying all components. The default behavior
@@ -202,6 +202,8 @@ def after_verify(
         Args:
             session:  Session
                 The current reconciliation session
+            verify_completion_state: CompletionState
+                Completion state of the last verification
 
         Returns:
             success:  bool
@@ -210,7 +212,7 @@ def after_verify(
         """
         return True
 
-    def after_verify_unsuccessful(self, session: Session, failed: bool) -> bool:
+    def after_verify_unsuccessful(self, session: Session, failed: bool, verify_completion_state: CompletionState) -> bool:
         """This allows children to inject logic that will run when the
         controller has finished deploying all components but failed to verify.
         The default behavior is a no-op.
@@ -220,6 +222,8 @@ def after_verify_unsuccessful(self, session: Session, failed: bool) -> bool:
                 The current reconciliation session
             failed:  bool
                 Indicator of whether or not the termination was a failure
+            verify_completion_state: CompletionState
+                Completion state of the last verification
 
         Returns:
             success:  bool

oper8/rollout_manager.py

@@ -436,7 +436,7 @@ def rollout(  # pylint: disable=too-many-locals,too-many-statements
             log.debug("Running after-verify-unsuccessful")
             try:
                 is_after_verify_unsuccessful_completed = (
-                    self._after_verify_unsuccessful(self._session, phase3_failed)
+                    self._after_verify_unsuccessful(self._session, phase3_failed, verify_completion_state)
                 )
                 if not is_after_verify_unsuccessful_completed:
                     phase4_exception = VerificationError(
@@ -450,7 +450,7 @@ def rollout(  # pylint: disable=too-many-locals,too-many-statements
         if phase3_complete and self._after_verify:
             log.debug("Running after-verify")
             try:
-                phase4_complete = self._after_verify(self._session)
+                phase4_complete = self._after_verify(self._session, verify_completion_state)
                 if not phase4_complete:
                     phase4_exception = VerificationError("After-verify failed")
             except Exception as err:  # pylint: disable=broad-except