-
Notifications
You must be signed in to change notification settings - Fork 93
Restart WPE on KME restart #653
base: main
Are you sure you want to change the base?
Conversation
self._kv_helper.csv_prepend("wo-worker-scheduled", | ||
self._worker_id, wo_id) | ||
logger.info("Reinstating work order {} to ".format(wo_id) | ||
+ "wo-worker-scheduled as this worker will retart.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change retart to restart
a1d5bb6
to
1f6098a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved after one typo is fixed
1f6098a
to
8e74c00
Compare
self._worker_id, wo_id) | ||
logger.info("Reinstating work order {} to ".format(wo_id) | ||
+ "wo-worker-scheduled as this worker will restart.") | ||
raise |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is error message and code client will as a response for the WorkerRestartException?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not exposed to a client. It is an internal exception used as a signal to trigger restart.
Signed-off-by: Rajeev Ranjan <[email protected]>
8e74c00
to
ff441ed
Compare
Signed-off-by: Rajeev Ranjan [email protected]