-
Notifications
You must be signed in to change notification settings - Fork 38
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 #11245: Add constraint on hash_type #624
Fixes #11245: Add constraint on hash_type #624
Conversation
@@ -21,7 +21,8 @@ | |||
# | |||
# @parameter source Source file (path relative to Rudder shared folder) | |||
# @parameter destination Destination file (absolute path on the target node) | |||
# @parameter hash_type Hash algorithm used to check if file is updated | |||
# @parameter hash_type Hash algorithm used to check if file is updated (md5, sha1, sha256) |
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.
You should document that this parameter is always treated as md5 on cfengine.
Commit modified |
40ce705
to
7db7ac2
Compare
@@ -21,7 +21,8 @@ | |||
# | |||
# @parameter source Source file (path relative to Rudder shared folder) | |||
# @parameter destination Destination file (absolute path on the target node) | |||
# @parameter hash_type Hash algorithm used to check if file is updated | |||
# @parameter hash_type Hash algorithm used to check if file is updated (md5, sha1, sha256), only md5 can be used on cfengine agent |
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.
it's not used on cfengine, so why not simply state that it is not used for cfengine agent ?
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.
I will write that on cfengine it will always be treated as md5 for now
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.
ping ?
Commit modified |
7db7ac2
to
ed9ad0c
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/11245