Lost ability to deploy Glance backed by NFS

Bug #1635606 reported by Jiří Stránský
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Jiří Stránský

Bug Description

We lost ability to store Glance images in NFS mounts as we moved to NG HA architecture, because the mounts were managed by Pacemaker.

Revision history for this message
Jiří Stránský (jistr) wrote :
Revision history for this message
Jiří Stránský (jistr) wrote :

With the fixes the glance image storage directory is a NFS mount, and pingtest succeeds fine, the images are getting created inside NFS. The NFS mount also gets restored after reboot.

[root@overcloud-controller-0 ~]# mount | grep glance
172.16.10.251:/export/glance on /var/lib/glance/images type nfs4 (rw,relatime,context=system_u:object_r:glance_var_lib_t:s0,vers=4.0,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=172.16.10.11,local_lock=none,addr=172.16.10.251)

[root@overcloud-controller-0 ~]# grep glance /etc/fstab
172.16.10.251:/export/glance /var/lib/glance/images nfs4 bg,intr,context=system_u:object_r:glance_var_lib_t:s0 0 0

Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to puppet-tripleo (master)

Reviewed: https://review.openstack.org/389284
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=ad4e331917a2da9f2fd8215369736896348244fd
Submitter: Jenkins
Branch: master

commit ad4e331917a2da9f2fd8215369736896348244fd
Author: Jiri Stransky <email address hidden>
Date: Thu Oct 20 19:26:16 2016 +0200

    NFS mounting for Glance file backend

    Previously we did this with Pacemaker, but with move to NG HA
    architecture we lost the ability to use NFS mounts as image storage for
    Glance. This reimplements the mounting without utilizing Pacemaker. The
    mount is by default also written to /etc/fstab so that it persists over
    reboot, but this behavior can be disabled.

    This could also go to puppet-glance eventually, but not yet -- we need
    this backported to Newton because it's a TripleO regression. I don't
    think puppet-glance would allow backporting this to Newton, because from
    their point of view it would be a RFE rather than a regression.

    Change-Id: I45ad34c36587a8d695069368cf791f1efb68256c
    Related-Bug: #1635606

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/389285
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=b6a4bdc3e4db97785b930065260c713f6e70a4da
Submitter: Jenkins
Branch: master

commit b6a4bdc3e4db97785b930065260c713f6e70a4da
Author: Jiri Stransky <email address hidden>
Date: Thu Oct 20 19:27:42 2016 +0200

    Re-add NFS backend for Glance

    We lost ability to store Glance images in NFS mounts as we moved to NG
    HA architecture. This patch re-adds that ability, but the parameter
    interface changes because the semantics change as well. (Pacemaker
    allowed for different mounts than just NFS so the parameters were more
    generic, although we only ever tested and documented NFS usage.)

    Change-Id: Ic5197e09846bbf75d780dcc74da1717dcf8301d0
    Related-Bug: #1635606

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (stable/newton)

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/392128

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to puppet-tripleo (stable/newton)

Related fix proposed to branch: stable/newton
Review: https://review.openstack.org/392131

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/newton)

Reviewed: https://review.openstack.org/392128
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=41f04626f8a7007dafaf79adecaee9ab48e168e3
Submitter: Jenkins
Branch: stable/newton

commit 41f04626f8a7007dafaf79adecaee9ab48e168e3
Author: Jiri Stransky <email address hidden>
Date: Thu Oct 20 19:27:42 2016 +0200

    Re-add NFS backend for Glance

    We lost ability to store Glance images in NFS mounts as we moved to NG
    HA architecture. This patch re-adds that ability, but the parameter
    interface changes because the semantics change as well. (Pacemaker
    allowed for different mounts than just NFS so the parameters were more
    generic, although we only ever tested and documented NFS usage.)

    Change-Id: Ic5197e09846bbf75d780dcc74da1717dcf8301d0
    Related-Bug: #1635606
    (cherry picked from commit b6a4bdc3e4db97785b930065260c713f6e70a4da)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to puppet-tripleo (stable/newton)

Reviewed: https://review.openstack.org/392131
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=40636997c109bd22568ef9a585e515d5e56b87a8
Submitter: Jenkins
Branch: stable/newton

commit 40636997c109bd22568ef9a585e515d5e56b87a8
Author: Jiri Stransky <email address hidden>
Date: Thu Oct 20 19:26:16 2016 +0200

    NFS mounting for Glance file backend

    Previously we did this with Pacemaker, but with move to NG HA
    architecture we lost the ability to use NFS mounts as image storage for
    Glance. This reimplements the mounting without utilizing Pacemaker. The
    mount is by default also written to /etc/fstab so that it persists over
    reboot, but this behavior can be disabled.

    This could also go to puppet-glance eventually, but not yet -- we need
    this backported to Newton because it's a TripleO regression. I don't
    think puppet-glance would allow backporting this to Newton, because from
    their point of view it would be a RFE rather than a regression.

    Change-Id: I45ad34c36587a8d695069368cf791f1efb68256c
    Related-Bug: #1635606
    (cherry picked from commit ad4e331917a2da9f2fd8215369736896348244fd)

Changed in tripleo:
status: In Progress → Fix Committed
Changed in tripleo:
status: Fix Committed → Fix Released
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.