Setting replication_strategy to None for MySQL Does Not Work

Bug #1367037 reported by Auston McReynolds
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Invalid
High
Auston McReynolds

Bug Description

setting replication_strategy to None in trove.conf under [mysql] does not work as intended (it does not throw an Exception as the code suggests it should @ https://github.com/openstack/trove/blob/f53ba944688b7a6a0df7a37529bb8be2da4b29d0/trove/instance/models.py#L698-L701). instead, it marches on and provisions the slave.

why this matters: some deployers (like me) want the ability to wire on or off replication support.

description: updated
description: updated
Changed in trove:
milestone: none → juno-rc1
importance: Undecided → High
status: New → Confirmed
Doug Shelley (0-doug)
Changed in trove:
assignee: nobody → Doug Shelley (0-doug)
Revision history for this message
Auston McReynolds (amcrn) wrote :

doug: let me know if you're still looking into this, if not, i'll take a look; thanks!

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

Auston - go ahead and take it - I don't think I will get to it this week.

Changed in trove:
assignee: Doug Shelley (0-doug) → Auston McReynolds (amcrn)
Revision history for this message
Auston McReynolds (amcrn) wrote :

interesting, because it's a StrOpt, if you have:
    replication_strategy = None

it converts it to the string "None", therefore the check "if not replication_support:" doesn't trigger.

the solution is to set it as follows:

    replication_strategy =

which is not obvious. i'd push a change to check for the string "None", but that sets a precedent that we should be doing it for other parameters; so for now, i'm marking this as invalid.

Changed in trove:
status: Confirmed → Invalid
milestone: juno-rc1 → none
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.