docker can't share nailgun dir ro to some and rw to others

Bug #1336771 reported by Matthew Mosesohn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Matthew Mosesohn
5.0.x
Fix Released
High
Aleksey Kasatkin

Bug Description

We expose /var/www/nailgun to most containers as read only, then read write to mcollective container. Mcollective ends up with a read only container, so it can't save diagnostic snapshots.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

summary: - docker can't share nailgun dir ro to some and rw to otehrs
+ docker can't share nailgun dir ro to some and rw to others
Changed in fuel:
milestone: none → 5.1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/104173
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=73db664c16321e2beef1afa679625ac520c1d9b2
Submitter: Jenkins
Branch: master

commit 73db664c16321e2beef1afa679625ac520c1d9b2
Author: Matthew Mosesohn <email address hidden>
Date: Wed Jul 2 16:29:55 2014 +0400

    Allow mcollective rw access to dump directory

    There is a bug in docker where ro/rw export of the
    same directory results in the rw export to be
    in fact read only.

    Change-Id: I83e56177df323fd1c465291415cd2a77d3153281
    Closes-Bug: 1336771

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/104253

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/104253
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=cd693ac54ef54d12d5687538afd85377332e4c1e
Submitter: Jenkins
Branch: master

commit cd693ac54ef54d12d5687538afd85377332e4c1e
Author: Matthew Mosesohn <email address hidden>
Date: Wed Jul 2 20:09:02 2014 +0400

    Fix broken repo config for dockerctl

    Previous commit 73db664c16321e2beef1afa679625ac520c1d9b2
    incorrectly addresses the problem by populating requirements
    list twice with the same key, which was removing repo
    volume from being exposed to most containers.

    Change-Id: I4926e35827c264850326a59f085259c044a4b838
    Related-Bug: #1336771

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/105091

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (stable/5.0)

Related fix proposed to branch: stable/5.0
Review: https://review.openstack.org/105092

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/105148

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (stable/5.0)

Change abandoned by Aleksey Kasatkin (<email address hidden>) on branch: stable/5.0
Review: https://review.openstack.org/105091
Reason: Squashed into https://review.openstack.org/#/c/105148/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Aleksey Kasatkin (<email address hidden>) on branch: stable/5.0
Review: https://review.openstack.org/105092
Reason: Squashed into https://review.openstack.org/#/c/105148/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.0)

Reviewed: https://review.openstack.org/105148
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=d4cb36208efaf51a7c0ca012fa63d596d4ee2e29
Submitter: Jenkins
Branch: stable/5.0

commit d4cb36208efaf51a7c0ca012fa63d596d4ee2e29
Author: Aleksey Kasatkin <email address hidden>
Date: Mon Jul 7 15:26:04 2014 +0300

    Backport to 5.0.1 of bug-fixes for 1336771 1332496

        Fix broken repo config for dockerctl

        Orig.Change-Id: I4926e35827c264850326a59f085259c044a4b838

        Allow mcollective rw access to dump directory

        Orig.Change-Id: I83e56177df323fd1c465291415cd2a77d3153281

        Combine dump and repo volumes in dockerctl

        Orig.Change-Id: I6548a468368ad59fe989fbccfddda50661a17b9b

        Closes-Bug: #1336771
        Closes-Bug: #1332496

    Change-Id: I690305a58535d46101699cb30157b9b177b0ff34

Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :

verified on fuel-5.0-upgrade-106-2014-07-08_00-31-14.tar

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/5.1.x
tags: added: module-shotgun
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.