Updated the hacking version as per global-requirements.

Fixed some of new appearing hacking rules, as listed follows:
- E128 continuation line under-indented for visual indent
- E265 block comment should start with '# '
- E713 test for membership should be 'not in'
- H238  old style class declaration, use new style (inherit from `object`)

Rules which are not yet fixed have been incorporated to ignorelist
- H405 Multi line docstrings should start with a one line summary
       followed by an empty line
- H501 Do not use locals() or self.__dict__ for formatting strings

Change-Id: I885c6fa8ad0e6a98f7a8479f3441ed08ed2cbaa6

The Oslo libraries have moved all of their code out of the 'oslo'
namespace package into per-library packages. The namespace package was
retained during kilo for backwards compatibility, but will be removed by
the liberty-2 milestone. This change removes the use of the namespace
package, replacing it with the new package names.

The patches in the libraries will be put on hold until application
patches have landed, or L2, whichever comes first. At that point, new
versions of the libraries without namespace packages will be released as
a major version update.

Please merge this patch, or an equivalent, before L2 to avoid problems
with those library releases.

Blueprint: remove-namespace-packages
https://blueprints.launchpad.net/oslo-incubator/+spec/remove-namespace-packages

Change-Id: I2a8caa859830b3416bfe54e4261dd3415ac5a76a

The correct order of parameters to assertEqual is (expected,
actual).
It makes the assertion message more meaningful when there is
a failure.

Change-Id: Ifd35dbf6a6157c6ec7607e8ace6757b12f47657d
Closes-Bug: #1277104

1. Renames tes___eq__ to be test___eq__,
   because earlier method was not being called.
2. Corrects the test-code in test___eq__,
   for the failures it gave when it started running.
3. Removes some unused code from tests,
   there were few lines/methods which were not used by tests
   just removed/updated them.

Change-Id: I90aa8cb9bb9f5d15697f00df0ba1defa0ef7be59
Closes-Bug: #1441518

There is no test to test DatastoreVersionMembers. This change adds it.

Change-Id: I921d0134125b9a3c392b993e8bb91a5750e0d4f1

The V2 implentation of replication, including:
  - promote-to-replica-source
  - eject-replica-source

Implements: blueprint replication-v2

Change-Id: I9075365a8fae754e29d27d6e371a1d8c8980e26b
Authored-By: Morgan Jones <morgan@parelastic.com>
Co-Authored-By: Peter Stachowski <peter@tesora.com>

Python-troveclient 1.0.8 introduced a regression where the usage
of --bypass-url results in an AuthSystemNotFound exception.
Removing some redundant code, which was due to a previous merge
conflict, fixed the issue.

Closes-bug 1416120

Change-Id: I1c949f69ee08447ce88aeefa92c60f361ab54832

Replaced last remaining reference to TROVE_URL in python-troveclient.
Currently the user along with OS_URL , or TROVE_URL , is used as part
of a hash for a unique directory for the bash completion cache. Since
I'm not sure if there is a reason why the legacy URLs are used here,
I've replaced TROVE_URL with NOVA_URL instead of OS_AUTH_URL which
might be more appropriate for this purpose.

Change-Id: Id30aeae992aecba3c1fbe53c2ce421157410ca20
Closes-Bug: #1238282