Tech-Debt: Namespace the Docker volumes

[Éric Lemoine]

Currently the Docker named volumes created by Kolla don't have a namespace/prefix.

For example, on my AIO installation:

    [root@operator kolla]# docker volume ls
    DRIVER VOLUME NAME
    local heka
    local nova_compute
    local nova_libvirt
    local kolla_logs
    local rsyslog_socket
    local haproxy_socket
    local mariadb
    local rabbitmq
    local rsyslog
    local heka_socket

In https://review.openstack.org/#/c/275841/20/ansible/roles/common/tasks/start.yml@26 Steve suggested that we namespace our named volumes: kolla_heka, kolla_mariadb, etc.

[Steven Dake (sdake)]

Making critical since this would have repercussions when we were to namespace our named volumes down the road. Might as well take care of it now.

[Steven Dake (sdake)]

[07:14:35] <sdake> actually I'm not sure namespacing makes sense
[07:14:40] <sdake> since we expect dedicated gear
[07:14:47] <sdake> and the rest of our services are nto namespaced

[Steven Dake (sdake)]

Changed to medium priority.

[Éric Lemoine (elemoine)]

> actually I'm not sure namespacing makes sense since we expect dedicated gear and the rest of our services are nto namespaced

In that case I think we should be consistent and rename the volume "kolla_logs" to "logs". During the review process Sam Yaple suggested that I rename the logs volume from "logs" to "kolla_logs" (or something else) to avoid volume name conflicts. I'd say let's be consistent and use the kolla_ prefix for all the Docker volumes created by Kolla or for none. My preference is to not use the kolla_ prefix and effectively rename the volume "kolla_logs" to "logs".

[Éric Lemoine (elemoine)]

If people thing "logs" is too generic as a name another option is to rename "kolla_logs" to "service_logs".

[Steven Dake (sdake)]

I am pretty certain we don't need namespacing of the volumes since the nodes used in a Kolla deployment are dedicated except in development environments. As such, closing as wontfix.

[Éric Lemoine (elemoine)]

Why are we making an exception for the "kolla_logs" volume? I am +1 on renaming kolla_logs to logs.