PXC Cluster Grow handles nic parameter inconsistently

Bug #1634629 reported by Morgan Jones
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
High
Duk Loi

Bug Description

The grow API for galera based clusters doesn't handle the nic parameters in a manner consistent with the create operation.

The Create API takes a nic parameter for each instance specified. The grow operation doesn't handle the nic parameter, if specified, but instead pics a network id from a random existing instance and sets it to every instance in the grow set.

This should be handled in a consistent manner.

Also, arguably, this should be handled consistently across all clusters.

Revision history for this message
Doug Shelley (0-doug) wrote :

Also, the code that does what is described above only appears to work for nova-network. When run in a cloud with Neutron, it fails with this error message (which is eaten silently due to [1]):
       Listing virtual interfaces is not supported by this cloud.

[1] https://bugs.launchpad.net/trove/+bug/1634627

Duk Loi (duktesora)
Changed in trove:
assignee: nobody → Duk Loi (duktesora)
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to trove (master)

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

Changed in trove:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/389758
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=741b86b4cd78073996a3f23c4b5dfcdeda24be15
Submitter: Jenkins
Branch: master

commit 741b86b4cd78073996a3f23c4b5dfcdeda24be15
Author: Duk Loi <email address hidden>
Date: Tue Oct 18 21:32:23 2016 -0400

    Remove nic handling in galera cluster grow

    Since the nic is required to be specified in the cluster grow,
    remove the unnecessary nic handling in the Galera cluster grow.

    This would make it consistent with the way all the other clusters
    are handling the nic parameter.

    Change-Id: I136141b3db78b7a0a1804bb5e3d4ca5cf4bc3bb3
    Closes-Bug: #1634629

Changed in trove:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/trove 7.0.0.0b2

This issue was fixed in the openstack/trove 7.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.