-
Notifications
You must be signed in to change notification settings - Fork 24
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 #10425: Add migration script for hooks ignore suffixes #1294
Fixes #10425: Add migration script for hooks ignore suffixes #1294
Conversation
PR rebased |
5747ae5
to
519a437
Compare
rudder-webapp/SOURCES/rudder-upgrade
Outdated
# before running a hook under /opt/rudder/etc/hooks.d. | ||
# If an executable file has one of the following extensions, it | ||
# will be IGNORED and the corresponding hook skipped. Non executable files are | ||
# always ignored, with or without any of these extensions. |
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.
Can we please consistently use "suffixes" instead of "extensions"? Or if you think it should be "extension", then rename the property, but don't mix and match.
rudder-webapp/SOURCES/rudder-upgrade
Outdated
" | ||
|
||
|
||
>>>>>>> Stashed changes |
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 looks like a git merge issue
PR rebased |
519a437
to
6e81d9b
Compare
PR rebased |
6e81d9b
to
5a8761d
Compare
rudder-webapp/SOURCES/rudder-upgrade
Outdated
@@ -1,4 +1,4 @@ | |||
#!/bin/bash | |||
rudder.hooks.ignore-suffixes#!/bin/bash |
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.
oops :)
PR rebased |
5a8761d
to
d314239
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10425