Jira (PDB-4804) Don't defer shutdown for cases intended to be immediate

6 views
Skip to first unread message

Rob Browning (Jira)

unread,
Jul 7, 2020, 5:33:03 PM7/7/20
to puppe...@googlegroups.com
Rob Browning created an issue
 
PuppetDB / Bug PDB-4804
Don't defer shutdown for cases intended to be immediate
Issue Type: Bug Bug
Affects Versions: PDB 6.10.0
Assignee: Unassigned
Created: 2020/07/07 2:32 PM
Fix Versions: PDB 6.11.2
Priority: Normal Normal
Reporter: Rob Browning

Our previous adjustments to shutdown in a friendlier way for appropriate errors (PDB-4627) ecb22d3a0fc4579d5e94075d57634bdf6df880fa, we ended up deferring some shutdowns that should have been immediate (i.e. if the config were broken). because request-shutdown only initiates the shutdown after all services have finished starting (if none of them throw an exception).  This was a problem because the PuppetDBServer was not necessarily left in a consistent state for other dependent services like the command dispatcher.

Since TK doesn't have a way to initiate an immediate friendly shutdown yet, for now just revert to throwing an exception.

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.5.2#805002-sha1:a66f935)
Atlassian logo

Rob Browning (Jira)

unread,
Jul 7, 2020, 5:34:03 PM7/7/20
to puppe...@googlegroups.com

Rob Browning (Jira)

unread,
Jul 8, 2020, 1:59:03 PM7/8/20
to puppe...@googlegroups.com
Rob Browning updated an issue
Our previous adjustments to shutdown in a friendlier way for appropriate errors (PDB-4627) ecb22d3a0fc4579d5e94075d57634bdf6df880fa, we ended up deferring some shutdowns that should have been immediate (i.e. if the config were broken). because request-shutdown only initiates the shutdown after all services have finished starting (if none of them throw an exception).  This was is a problem because the PuppetDBServer was not necessarily left in a consistent state for other dependent services like the command dispatcher.

Since TK doesn't have a way to initiate an immediate friendly shutdown yet, for now
, as a mitigation, just revert to throwing an exception make sure we can't come out of maintenance mode during the (deferred) shutdown, until we resolve PDB-4805 .
Reply all
Reply to author
Forward
0 new messages