Skip to content

Commit de862c5

Browse files
committed
style: formatting codes.
Signed-off-by: Yuki Ito <yuki462@ibm.com>
1 parent 4989fb3 commit de862c5

File tree

2 files changed

+22
-8
lines changed

2 files changed

+22
-8
lines changed

oper8/controller.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,9 @@ def finalize_components(self, session: Session): # noqa: B027
152152
The current session containing the per-event configs
153153
"""
154154

155-
def after_deploy(self, session: Session, deploy_completion_state: CompletionState) -> bool:
155+
def after_deploy(
156+
self, session: Session, deploy_completion_state: CompletionState
157+
) -> bool:
156158
"""This allows children to inject logic that will run when the
157159
controller has finished deploying all components, but not necessarily
158160
verifying all of them. The default behavior is a no-op.
@@ -170,7 +172,9 @@ def after_deploy(self, session: Session, deploy_completion_state: CompletionStat
170172
"""
171173
return True
172174

173-
def after_deploy_unsuccessful(self, session: Session, failed: bool, deploy_completion_state: CompletionState) -> bool:
175+
def after_deploy_unsuccessful(
176+
self, session: Session, failed: bool, deploy_completion_state: CompletionState
177+
) -> bool:
174178
"""This allows children to inject logic that will run when the
175179
controller has failed or could not finish deploying all components.
176180
The default behavior is a no-op.
@@ -193,7 +197,7 @@ def after_deploy_unsuccessful(self, session: Session, failed: bool, deploy_compl
193197
def after_verify(
194198
self,
195199
session: Session, # pylint: disable=unused-argument
196-
verify_completion_state: CompletionState
200+
verify_completion_state: CompletionState,
197201
) -> bool:
198202
"""This allows children to inject logic that will run when the
199203
controller has finished verifying all components. The default behavior
@@ -212,7 +216,9 @@ def after_verify(
212216
"""
213217
return True
214218

215-
def after_verify_unsuccessful(self, session: Session, failed: bool, verify_completion_state: CompletionState) -> bool:
219+
def after_verify_unsuccessful(
220+
self, session: Session, failed: bool, verify_completion_state: CompletionState
221+
) -> bool:
216222
"""This allows children to inject logic that will run when the
217223
controller has finished deploying all components but failed to verify.
218224
The default behavior is a no-op.

oper8/rollout_manager.py

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,9 @@ def rollout( # pylint: disable=too-many-locals,too-many-statements
326326
log.debug2("Running after-deploy-unsuccessful")
327327
try:
328328
is_after_deploy_unsuccessful_completed = (
329-
self._after_deploy_unsuccessful(self._session, phase1_failed, deploy_completion_state)
329+
self._after_deploy_unsuccessful(
330+
self._session, phase1_failed, deploy_completion_state
331+
)
330332
)
331333
if not is_after_deploy_unsuccessful_completed:
332334
phase2_exception = VerificationError(
@@ -344,7 +346,9 @@ def rollout( # pylint: disable=too-many-locals,too-many-statements
344346
if phase1_complete and self._after_deploy:
345347
log.debug2("Running after-deploy")
346348
try:
347-
phase2_complete = self._after_deploy(self._session, deploy_completion_state)
349+
phase2_complete = self._after_deploy(
350+
self._session, deploy_completion_state
351+
)
348352
if not phase2_complete:
349353
phase2_exception = VerificationError(
350354
"After-deploy verification failed"
@@ -436,7 +440,9 @@ def rollout( # pylint: disable=too-many-locals,too-many-statements
436440
log.debug("Running after-verify-unsuccessful")
437441
try:
438442
is_after_verify_unsuccessful_completed = (
439-
self._after_verify_unsuccessful(self._session, phase3_failed, verify_completion_state)
443+
self._after_verify_unsuccessful(
444+
self._session, phase3_failed, verify_completion_state
445+
)
440446
)
441447
if not is_after_verify_unsuccessful_completed:
442448
phase4_exception = VerificationError(
@@ -450,7 +456,9 @@ def rollout( # pylint: disable=too-many-locals,too-many-statements
450456
if phase3_complete and self._after_verify:
451457
log.debug("Running after-verify")
452458
try:
453-
phase4_complete = self._after_verify(self._session, verify_completion_state)
459+
phase4_complete = self._after_verify(
460+
self._session, verify_completion_state
461+
)
454462
if not phase4_complete:
455463
phase4_exception = VerificationError("After-verify failed")
456464
except Exception as err: # pylint: disable=broad-except

0 commit comments

Comments
 (0)