-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #11191: Add a post-node-acceptation hook #1708
Fixes #11191: Add a post-node-acceptation hook #1708
Conversation
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.
Some typo to correct.
Wouldn't it be great to have that in branch 4.1 as well ?
postHooks <- RunHooks.getHooks(HOOKS_D + "/node-post-acceptance", HOOKS_IGNORE_SUFFIXES) | ||
runPostHook = RunHooks.syncRun(postHooks, hookEnv, systemEnv) | ||
timePostHooks = (System.currentTimeMillis - postHooksTime) | ||
_ = logger.debug(s"Node-post-deletion scripts hooks ran in ${timePostHooks} ms") |
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.
Node-post-deletion
@@ -0,0 +1,16 @@ | |||
= policy-generation-node-finished |
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.
shouldn't it be node-post-acceptance ?
I didn't wanted to put it in 4.1 because the feature is half-backed (no pre-node acceptance, not sure about the parameters passed to scripts) and I would prefer to have a real relection done on that subject before considering adding it to 4.1. |
3fb86dc
to
1372d62
Compare
PR rebased |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/11191