Update parallel-processing-expansion.md#30346
Conversation
removed unwanted jinja tags
|
Welcome @sureshdsk! |
|
✔️ Deploy Preview for kubernetes-io-main-staging ready! 🔨 Explore the source changes: 2b61e46 🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/61829ac2183441000847087d 😎 Browse the preview: https://deploy-preview-30346--kubernetes-io-main-staging.netlify.app |
RinkiyaKeDad
left a comment
There was a problem hiding this comment.
Deployed preview at: https://deploy-preview-30346--kubernetes-io-main-staging.netlify.app/docs/tasks/job/parallel-processing-expansion/#use-advanced-template-parameters
@sureshdsk I was testing this out and got the following error after running cat job.yaml.jinja2 | render_template > jobs.yaml
Traceback (most recent call last):
File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'jinja2'
Were you able to test it out following the written instructions?
|
@RinkiyaKeDad I've tried this out and its working correctly. command to install jinja2 |
|
I think we could add a note about that too since it might not be obvious to someone not familiar with python 🤔 |
|
my bad. It is mentioned in before-you-begin section -> |
|
/lgtm |
|
LGTM label has been added. DetailsGit tree hash: 7e45e650a7b8f900588e24633c45c703f2a4104a |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: divya-mohan0209 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
https://kubernetes.io/docs/tasks/job/parallel-processing-expansion/#use-advanced-template-parameters
Made the jinja example look clean by removing the unwanted hyphen.