Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
hzy46 committed Feb 5, 2021
1 parent 7008caa commit c2e34ba
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/database-controller/src/write-merger/handler.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,10 @@ async function postWatchEvents(req, res, next) {
if (snapshot.getState() === 'Completed') {
// if event is DELETED and the state is Completed, mark apiServerDeleted = true
internalUpdate.apiServerDeleted = true;
// If the job is completed and deleted, we could reset its sensitive fields.
internalUpdate.configSecretDef = '';
internalUpdate.dockerSecretDef = '';
internalUpdate.tokenSecretDef = '';
} else {
// Event is DELETED and the state is not Completed.
// This case could occur when someone deletes the framework in API server directly.
Expand Down

0 comments on commit c2e34ba

Please sign in to comment.