Skip to content

Commit

Permalink
Remove the organization content for the schedules (#678)
Browse files Browse the repository at this point in the history
* remove the organization content for the schedules as it is not really needed. Add a reminder to check for the existance of 'ToDo:' in the generated files

* use fqcn

* completely remove the organization empty field

---------

Co-authored-by: silvinux <[email protected]>
  • Loading branch information
ivarmu and silvinux authored Aug 25, 2023
1 parent 589f57d commit 01ddd19
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
4 changes: 4 additions & 0 deletions roles/filetree_create/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,4 +51,8 @@
apply:
tags: "{{ input_tag | to_yaml }}"
tags: "{{ valid_tags }}"

- name: "Remind to check all the 'ToDo:' entries in the output files"
ansible.builtin.debug:
msg: "Please, check the existance of 'ToDo: ' entries in the files generated at '{{ output_path }}' with the following command: grep -R 'ToDo: ' '{{ output_path }}'"
...
1 change: 0 additions & 1 deletion roles/filetree_create/templates/current_schedules.j2
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ controller_schedules:
{% if current_schedules_asset_value.description is defined %}
description: "{{ current_schedules_asset_value.description }}"
{% endif %}
organization: "ToDo: The schedule '{{ current_schedules_asset_value.name }}' must have an organization"
enabled: {{ current_schedules_asset_value.enabled }}
unified_job_template: "{{ current_schedules_asset_value.summary_fields.unified_job_template.name }}"
{% if current_schedules_asset_value.summary_fields.inventory is defined %}
Expand Down

0 comments on commit 01ddd19

Please sign in to comment.