-
Notifications
You must be signed in to change notification settings - Fork 23
filetree_create fixes to export workflow job templates #81
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
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.
Thanks a lot for your PR and your effort, I appreciate it a lot. That being said, there's missing some information, as what errors do you get without these changes, what versions are you using...
Please check the individual review comments.
roles/filetree_create/tasks/all.yml
Outdated
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.
We are not going back with these names. For AAP2.5, aap_*
are the correct ones and are aligned with the main collection:
So, please adapt your input variables to the new names for this collection.
roles/filetree_create/tasks/main.yml
Outdated
@@ -50,7 +50,6 @@ | |||
args: | |||
apply: | |||
tags: "{{ input_tag | to_yaml }}" | |||
tags: "{{ valid_tags }}" |
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.
What's the matter with this? This is required to run correctly when you only need to export one or more objet types, but not all of them.
2. Add double quotes for `*_nodes` lines that may contain special characters (like #) 3. Fix Truthy value for `survey_enabled` coming in as `True` or `False` instead of `true` or `false`
* misc: inventory name unification * fix: missing change * fix: missing change after merge
…hat-cop/infra.controller_configuration#22 (redhat-cop#85) * Update controller_workflow_job_templates.j2 to fix issue found at redhat-cop/infra.controller_configuration#22 * add changelog
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.
LGTM. It adds an extra check to be sure there always will be at least one item in the list. Good job!
What does this PR do?
When running a fairly straight-forward playbook (as alluded to in the docs), I get the following failures:
Of note, I still get failures when attempting to export Applications, I didn't need this functionality but perhaps we should look into fixing downstream resources exports.
How should this be tested?
Include filetre_create role.
Is there a relevant Issue open for this?
No that I'm aware of, but as mentioned above filetree_create role didn't work for me without these changes.