fix prometheus role comments

Author: sjpb

ansible/roles/prometheus/README.md

@@ -2,12 +2,9 @@
 Creates a systemd service `prometheus` which uses the `podman` user to run a containerised [Prometheus](https://github.com/prometheus/prometheus) monitoring system.
 
 Note this contains two task books:
-    - `install.yml`: This is safe to run during a Packer build. It pulls the container and creates the systemd unit file.
+    - `install.yml`: This is safe to run during a Packer build. It pulls the container image and creates the systemd unit file.
     - `runtime.yml`: This cannot be run during a Packer build. It templates out config and restarts/starts the service as required.
 
-## TODO
- - Check upgrading clusters from cloudalchemy.prometheus role works OK. The state dir will be owned by `prometheus` user, not `podman`, but this role should fix that.
-
 ## Role Variables
 
 See `defaults/main.yml`. All variables can be updated by running `runtime.yml`, except the below which require `install.yml` to be run to update:

ansible/roles/prometheus/tasks/install.yml

@@ -17,7 +17,7 @@
     podman_service_volumes:
       - "{{ prometheus_config_dir }}:/etc/prometheus/:U,ro"
       - "{{ prometheus_db_dir }}:/prometheus:U"
-    podman_service_command: | # TODO: make retention time/size runtime configurable (can't be specified in config yml)
+    podman_service_command: |
       --config.file=/etc/prometheus/prometheus.yml \
       --storage.tsdb.path=/prometheus \
       --web.console.libraries=/usr/share/prometheus/console_libraries \