Remove 'mount_point' from guestagent RPC calls and CONF

Registered by Illia Khudoshyn

The 'mount_point' parameter in guestagent holds path where volume should be mounted to.
trove.guestagent.datastore.mysql.manager.Manager#prepare method receives 'mount_point' as a parameter.
But in come occasions its value is taken from CONF rather than from the parameter, see
https://github.com/openstack/trove/blob/master/trove/guestagent/datastore/mysql/manager.py#L99,
https://github.com/openstack/trove/blob/master/trove/guestagent/datastore/mysql/manager.py#L105
vs
https://github.com/openstack/trove/blob/master/trove/guestagent/datastore/mysql/manager.py#L107.
Moreover, in some occasions the CONF is the only source where the value of the 'mount_point' parameter
can be obtained, see
https://github.com/openstack/trove/blob/master/trove/guestagent/backup/backupagent.py#L64.

It is suggested to remove 'mount_point' from both RPC API calls and Trove configuration files.
When needed, this value should be taken from existing datastorage configuration,
e.g. my.cnf for MyySQL.

Blueprint information

Status:
Complete
Approver:
Michael Basnight
Priority:
Undefined
Drafter:
None
Direction:
Needs approval
Assignee:
Illia Khudoshyn
Definition:
Obsolete
Series goal:
None
Implementation:
Started
Milestone target:
milestone icon next
Started by
Thierry Carrez
Completed by
Nikhil Manchanda

Related branches

Sprints

Whiteboard

The only thing that scares me is that we arent versioning the client messages yet. But this is for a create call, so in theory the new instances should have a new guest anyway so its likely not a problem.

Gerrit topic: https://review.openstack.org/#q,topic:bug/1248212,n,z

Addressed by: https://review.openstack.org/57189
    Remove 'mount_point' from guestagent RPC API

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.