Skip to content

Replace removed cron task include with include_tasks - #76

Closed
93jun wants to merge 1 commit into
devopshobbies:masterfrom
93jun:fix-cron-include-tasks
Closed

Replace removed cron task include with include_tasks#76
93jun wants to merge 1 commit into
devopshobbies:masterfrom
93jun:fix-cron-include-tasks

Conversation

@93jun

@93jun 93jun commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

This updates the create-cronjob task entry point to use ansible.builtin.include_tasks instead of the removed bare include action.

Recent ansible-core versions reject include before task execution, so this keeps the role compatible with current Ansible releases.

Validation

  • git diff --check

@93jun

93jun commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Closing this narrower PR in favor of #75, which now includes both the include_tasks update and the cron package prerequisite in one branch. I re-tested the combined role path in a clean Ubuntu 22.04 container with ansible-core 2.16, and it completed successfully.

@93jun 93jun closed this Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant