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 #18384: Create a file from remote template generic method #1249
Fixes #18384: Create a file from remote template generic method #1249
Conversation
this is not tested yet |
f9de958
to
02fcf7b
Compare
Commit modified |
PR updated with a new commit |
1 similar comment
PR updated with a new commit |
"full_class_prefix" string => canonify("file_from_template_${report_param}"); | ||
"class_prefix" string => string_head("${full_class_prefix}", "1000"); | ||
|
||
"template_folder" string => "${sys.workdir}/templates/file_from_remote_template"; |
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.
there is a /var/rudder/tmp that already exists and the training tells user to use it for templates
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.
interesting, because File from template with type recommends the templates folder
It is common to use a specific folder to store those templates after copy, for example in
${sys.workdir}/templates/
.
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.
/var/rudder/templates is not provided by the package, i think we should avoid multiplying things in /var/rudder
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.
so we should change the documentation on the others
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.
yes we should
PR updated with a new commit |
1 similar comment
PR updated with a new commit |
OK, squash merging this PR |
355cd71
to
e7b1470
Compare
https://issues.rudder.io/issues/18384