refactor: move naive assertion to common role

[zhan9san]

If this proposal is accepted, I'll update all other roles

[github-actions]

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main:
https://prometheus-community.github.io/ansible/branch/main

[gardar]

Thanks! Please go ahead with applying the change across all the roles

[zhan9san]

Done. It's ready for review

[gardar]

Don't forget to remove the naive assertion task here

[zhan9san]

Good point. It's removed.

[gardar]

I wonder if this assert should be added to the cadvisor role too.

[zhan9san]

I wonder if this assert should be added to the cadvisor role too.

I should list cadvisor in the description.

It is not added intentionally. Because it is not in standard format.

$ docker run --rm -it gcr.io/cadvisor/cadvisor --help | grep -E 'listen|port'
    	Only report docker containers in addition to root stats
  -listen_ip string
    	IP to listen on, defaults to all IPs
  -port int
    	port to listen (default 8080)

node_exporter and others is in this format, --web.listen-address=:9100.

[gardar]

LGTM