database_list shows Percona system databases

Bug #1297000 reported by Doug Shelley
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Low
Sushil Kumar

Bug Description

trove database-list <instance-id>

+--------------------+
| name |
+--------------------+
| database9 |
| performance_schema |
| test |
+--------------------+

"performance_schema" and "test" are, I believe, internal schemas to Percona and should be excluded from this list.

Doug Shelley (0-doug)
Changed in trove:
assignee: nobody → Doug Shelley (0-doug)
Revision history for this message
Steve Leon (steve-leon) wrote :

Hey Doug,

this bug might be invalid.
You can hide DBs by tweaking the config. Just add the DBs that you want to ignore.
https://github.com/openstack/trove/blob/master/etc/trove/trove-guestagent.conf.sample#L56

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

Steve,

Thanks for the feedback - I guess I was thinking we would want to "ship" with these in the configuration?

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

And one other thing I noticed - the guest agent has some database excludes for mysql hardcoded in the query the gets the db list. (Around line 422 in guestagent/datastore/mysql/service.py)

Revision history for this message
Steve Leon (steve-leon) wrote :

Interesting. Now that you mentioned, this could be indeed a bug where list_database does not make use of ignore_dbs conf, but uses hard coded strings in the query.

The ignore_dbs conf is used in ValidatedMySQLDatabase to make sure that any action that create/delete/update any databases is not applied in any of these databases defined here.

Could you verify if adding 'performance_schema' and 'test' in the ingore_dbs conf prevents these DBs to appear when being listed?. If it doesnt, then you have a bug and should be fixed by using the conf.

Revision history for this message
Nikhil Manchanda (slicknik) wrote :

Marking as incomplete until we get further info. Please reactivate based on findings if the bug still exists.

Thanks!

Changed in trove:
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
Sushil Kumar (sushil-kumar2) wrote :

updated a patchset https://review.openstack.org/92176 to add performance_schema to ignore_dbs list

Changed in trove:
status: Incomplete → Opinion
status: Opinion → In Progress
assignee: Doug Shelley (0-doug) → Sushil Kumar (sushil-kumar2)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on trove (master)

Change abandoned by Nikhil Manchanda (<email address hidden>) on branch: master
Review: https://review.openstack.org/92176
Reason: Abandoning patches that have a negative review and are older than 4 weeks as part of a house-keeping sweep. Please feel free to restore the patch if it is still actively being worked on.

Changed in trove:
milestone: none → kilo-rc1
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in trove:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in trove:
milestone: kilo-rc1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.