cinder_volumes_container bind mount for /var/log not configured

Bug #1756005 reported by Janis Marrek
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Jesse Pretorius

Bug Description

common-tasks/os-lxc-container-setup.yml only gets executed when the lvm-backend for cinder is used because of

    "cinder_backend_lvm_inuse | bool"

in common-playbooks/cinder.yml.

As a result the bind mount for /var/log is missing in the cinder_volmues_container, when default_bind_mount_logs is set to true.

Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (master)

Fix proposed to branch: master
Review: https://review.openstack.org/565951

Changed in openstack-ansible:
assignee: nobody → Kevin Carter (kevin-carter)
status: Confirmed → In Progress
Changed in openstack-ansible:
assignee: Kevin Carter (kevin-carter) → Chenjun Shen (cshen)
Changed in openstack-ansible:
assignee: Chenjun Shen (cshen) → Jesse Pretorius (jesse-pretorius)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (master)

Reviewed: https://review.openstack.org/581248
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=92427a6531aca062d3e4cb536b0db55992aa0ef6
Submitter: Zuul
Branch: master

commit 92427a6531aca062d3e4cb536b0db55992aa0ef6
Author: Chenjun Shen <email address hidden>
Date: Tue Jul 10 10:15:10 2018 +0200

    Fix "configure container (other services)" when statement

    Configure container (other services) task should be executed when
    cinder_volume uses other than lvm as backend (for example. rbd). But the
    existing code completely skips both "Configure container
    (cinder-volume)" and "Configure container (other services)" tasks. So
    the cinder_volume container is not properly initialized when using rbd as
    backend.

    Change-Id: Ieb195e8a0cd3435fa3e872b609717aadc2ff41e8
    Closes-Bug: #1780789
    Closes-Bug: #1756005

Changed in openstack-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/622507

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (stable/rocky)

Reviewed: https://review.openstack.org/622507
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=a83274abfd7c1130a95481ece32cb44ce04745f1
Submitter: Zuul
Branch: stable/rocky

commit a83274abfd7c1130a95481ece32cb44ce04745f1
Author: Chenjun Shen <email address hidden>
Date: Tue Jul 10 10:15:10 2018 +0200

    Fix "configure container (other services)" when statement

    Configure container (other services) task should be executed when
    cinder_volume uses other than lvm as backend (for example. rbd). But the
    existing code completely skips both "Configure container
    (cinder-volume)" and "Configure container (other services)" tasks. So
    the cinder_volume container is not properly initialized when using rbd as
    backend.

    Change-Id: Ieb195e8a0cd3435fa3e872b609717aadc2ff41e8
    Closes-Bug: #1780789
    Closes-Bug: #1756005
    (cherry picked from commit 92427a6531aca062d3e4cb536b0db55992aa0ef6)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible 18.1.1

This issue was fixed in the openstack/openstack-ansible 18.1.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible 19.0.0.0b1

This issue was fixed in the openstack/openstack-ansible 19.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on openstack-ansible (master)

Change abandoned by Kevin Carter (cloudnull) (<email address hidden>) on branch: master
Review: https://review.opendev.org/565951

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.