Skip to content

Better zk cleanup for removed requests#1477

Merged
ssalinas merged 4 commits intomasterfrom
zk_cleanup
Apr 5, 2017
Merged

Better zk cleanup for removed requests#1477
ssalinas merged 4 commits intomasterfrom
zk_cleanup

Conversation

@ssalinas
Copy link
Copy Markdown
Contributor

Cleans up a number of stale nodes created for requests.

On request delete:

  • remove from /deploys/requests
  • remove from /tasks/history

In purger (x -> purgeStaleRequestIdsAfterDays setting in config, 7 days by default)

  • In /deploys/requests delete if not in active request ids + hasn't been updated in x days + no STATE key present
  • In /tasks/history delete if not in active request ids + hasn't been updated in x days + no child nodes
  • In /metadata/mails delete if not in active request ids + hasn't been updated in x days

/cc @wsorenson

@ssalinas ssalinas modified the milestone: 0.15.0 Mar 30, 2017
@ssalinas ssalinas merged commit 4c16ce9 into master Apr 5, 2017
@ssalinas ssalinas deleted the zk_cleanup branch April 5, 2017 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant