moved FIXMEs to an external list

icinga2/plugins.sls

@@ -18,8 +18,6 @@
       - git: /usr/local/lib/nagios
       - pkg: monitoring_packages
 
-# TODO: make all this venv stuff a cmd.script
-
 venv-ensurepip:
   cmd.run:
     - name: /usr/local/lib/nagios/venv/bin/python3 -m ensurepip

restic/init.sls

@@ -15,7 +15,6 @@ restic-{{ name }}:
     - user: restic-{{ name }}
     - group: restic-{{ name }}
     - mode: 400
-    # TODO: we don't ensure a root rsa key exists on the minion
     - contents: {{ salt.mine.get(minion_id, 'root_id_rsa_pub').get(minion_id, "") | yaml_encode }}
 {% endfor %}
 

salt/init.sls

@@ -106,7 +106,6 @@ salt-{{ daemon }}:
       - file: /etc/salt/{{ daemon }}
 {%   endfor %}
 
-# FIXME: D.R.Y.
 salt-common:
   pkg.installed:
     - version: '{{ salt.pillar.get("salt:version") }}'

samba/files/smb.conf.jinja

@@ -1,6 +1,5 @@
 #
 # Managed by Salt
-# TODO: this could use file.serialize
 # run testparm -s after making changes to validate them
 
 [global]

users/init.sls

@@ -3,7 +3,6 @@
   {% set home = '' if user == 'root' else '/home' %}
 
   {% if user != 'root' %}
-  # FIXME - both config and auth keys need this and this logic is dumb
 {{ home }}/{{ user }}/.ssh:
   file.directory:
     - user: {{ user }}