instack-undercloud doesn't use default Swift ring partition power

Bug #1631926 reported by Christian Schwede
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Christian Schwede

Bug Description

The built rings actually use a partition power of 18, which is the default in puppet-swift if nothing else is defined. However, a partition power of 10 is defined in elements/puppet-stack-config/puppet-stack-config.yaml.template in instack-undercloud and puppet/services/swift-ringbuilder.yaml in tripleo-heat-templates, but obviously not used.

This might create some serious problems in the future, because changing the partition power is not possible later on, and it will slow down replication a lot and create many extra inode entries that are not required at all.

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

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

Changed in tripleo:
assignee: nobody → Christian Schwede (cschwede)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

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

description: updated
Steven Hardy (shardy)
Changed in tripleo:
importance: Undecided → High
milestone: none → newton-rc3
milestone: newton-rc3 → ocata-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to instack-undercloud (master)

Reviewed: https://review.openstack.org/384436
Committed: https://git.openstack.org/cgit/openstack/instack-undercloud/commit/?id=d80a144b14ad5568a8820911a1e4349f23bc9fa9
Submitter: Jenkins
Branch: master

commit d80a144b14ad5568a8820911a1e4349f23bc9fa9
Author: Christian Schwede <email address hidden>
Date: Mon Oct 10 13:29:55 2016 +0200

    Fix default Swift ring partition power

    Looks like tripleo::ringbuilder::part_power is not used at all; actually
    the partition power on the underclod is 18, which is the default in
    puppet-swift if nothing else is defined.

    Closes-Bug: 1631926
    Change-Id: Ic32c44810aba3ccbb9429cf210f27843d8b6b2c4

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

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/386239

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

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

commit 1c8ef6c5e8473ccde3755710540225dbd74edb5b
Author: Christian Schwede <email address hidden>
Date: Mon Oct 10 13:42:22 2016 +0200

    Fix default Swift ring partition power

    Looks like swift::ringbuilder::part_power is not used at all; actually
    the partition power on the overcloud is 18, which is the default in
    puppet-swift if nothing else is defined.

    Closes-Bug: 1631926
    Depends-On: I78049105adf52226d47cc6764b1ba6c2c06e91e5
    Change-Id: I65335c8d31ed1130e71f1e193eb519b9f7f2438e

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to instack-undercloud (stable/newton)

Reviewed: https://review.openstack.org/386239
Committed: https://git.openstack.org/cgit/openstack/instack-undercloud/commit/?id=c11da8e0d013b37fd544306abc6380f1dd753e1e
Submitter: Jenkins
Branch: stable/newton

commit c11da8e0d013b37fd544306abc6380f1dd753e1e
Author: Christian Schwede <email address hidden>
Date: Mon Oct 10 13:29:55 2016 +0200

    Fix default Swift ring partition power

    Looks like tripleo::ringbuilder::part_power is not used at all; actually
    the partition power on the underclod is 18, which is the default in
    puppet-swift if nothing else is defined.

    Cherry-picked from commit d80a144b14ad5568a8820911a1e4349f23bc9fa9

    Closes-Bug: 1631926
    Change-Id: Ic32c44810aba3ccbb9429cf210f27843d8b6b2c4

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/instack-undercloud 5.0.0.0rc3

This issue was fixed in the openstack/instack-undercloud 5.0.0.0rc3 release candidate.

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

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/389743

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/389748

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

Reviewed: https://review.openstack.org/389748
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=99ce66c0c8eeef137d56d81dfab64f2f33ce5680
Submitter: Jenkins
Branch: stable/newton

commit 99ce66c0c8eeef137d56d81dfab64f2f33ce5680
Author: Christian Schwede <email address hidden>
Date: Mon Oct 10 16:19:26 2016 +0200

    Add part_power and min_part_hours for Swift

    Change-Id: I78049105adf52226d47cc6764b1ba6c2c06e91e5
    Related-Bug: 1631926
    (cherry picked from commit 1174c61cc55dbb3a4c003b52dfe7e4235e0916c1)

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

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

commit 4da6256caaf2f2dd4b177262dbbea474b05bbba3
Author: Christian Schwede <email address hidden>
Date: Mon Oct 10 13:42:22 2016 +0200

    Fix default Swift ring partition power

    Looks like swift::ringbuilder::part_power is not used at all; actually
    the partition power on the overcloud is 18, which is the default in
    puppet-swift if nothing else is defined.

    Closes-Bug: 1631926
    Depends-On: I78049105adf52226d47cc6764b1ba6c2c06e91e5
    Change-Id: I65335c8d31ed1130e71f1e193eb519b9f7f2438e
    (cherry picked from commit 1c8ef6c5e8473ccde3755710540225dbd74edb5b)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/instack-undercloud 5.0.0

This issue was fixed in the openstack/instack-undercloud 5.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 5.1.0

This issue was fixed in the openstack/tripleo-heat-templates 5.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 6.0.0.0b1

This issue was fixed in the openstack/tripleo-heat-templates 6.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 5.1.0

This issue was fixed in the openstack/tripleo-heat-templates 5.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/instack-undercloud 6.0.0.0b2

This issue was fixed in the openstack/instack-undercloud 6.0.0.0b2 development milestone.

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.