Explore using `retry` instead of 7-second pause for dockutil changes about ansible-collection-mac HOT 5 CLOSED

This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

from ansible-collection-mac.

This issue is no longer marked for closure.

from ansible-collection-mac.

I looked at #43 and see that it is closed so I'm adding my comments here. An alternative way to approach adding items via dockutil and avoid the 7 seconds wait between each change is to use the --no-restart option for all but the last item. I'm not sure when this option was added but it is present now.

If the task in the main playbook is modified to include extended loop information like this:

- name: Ensure required dock items exist.
  ansible.builtin.include_tasks: dock-add.yml
  loop: "{{ dockitems_persist }}"
  loop_control:
    extended: true
    extended_allitems: false
  tags: ['dock']

The task in the add playbook can be modified to take advantage of the ansible_loop.last variable like this:

- name: Ensure Dock item {{ item.name | default(item) }} exists.
  ansible.builtin.command: "dockutil --add '{{ item.path }}' --label '{{ item.name }}' {% if ansible_loop.last is sameas true %}--no-restart{% endif %}"
  when: dockitem_exists.rc >0 or
        dockitem_exists.rc == 0 and current_section == 'recent-apps'
  tags: ['dock']

I've not used ansible in a while so my syntax above could be incorrect. Please feel free to correct / improve the suggestion.

from ansible-collection-mac.

@karthicraghupathi - Oh... something like that could work, and would make the setup of the Dock a bit more robust (I hope). If someone can work on a PR implementing that change it could be good to go. I haven't had a chance to dig into that or any other little issues on this project in a couple months.

from ansible-collection-mac.

@geerlingguy Thank you for taking a look. I will look to submit a PR soon. Things are a bit hectic at work atm...

from ansible-collection-mac.