CRS-1632 Add cadvisor

[startnow65]

• Fix Deprecation warnings
• Add cadvisor

[antonu17]

A common convention for role variables is to have them prefixed with the role name, in order to avoid possible variable conflicts with the other roles used in the same playbook.

Here we have prefixes:

• dockerhost_
• docker_
• install_
• cadvisor_
• prometheus_
• disable_

It can lead to conflicts. I guess renaming existing vars is not a good idea, but maybe stick to a constant prefix for the new variables?

Everything else is 👍

[startnow65]

A common convention for role variables is to have them prefixed with the role name, in order to avoid possible variable conflicts with the other roles used in the same playbook.

Here we have prefixes:

• dockerhost_
• docker_
• install_
• cadvisor_
• prometheus_
• disable_

It can lead to conflicts. I guess renaming existing vars is not a good idea, but maybe stick to a constant prefix for the new variables?

Everything else is 👍

Thanks, updated