Currently, OpenStack Proposal Bot tries to update requirements with
global requirements, while the upgraded hacking has introduced some
new rules which are not fully handled by current code.

This patch fixes some simple rules which are quite straight-forward.

Change-Id: If8334f69fb1ad34fbbd6ad898e0e92eb3f81e95a

split_by_op was returning data against which further error checking was
then required. This change moves that error handling inside the method
and simplifies the regular expression so that it splits (greedily) on
the first operator it finds. If the split is not possible, it is
a ValueError. If the field or value are empty, that is a ValueError.

Both split_by_op and split_by_datatype were doing a findall() where
a match() and split() do the right job and more efficiently.

Regular expression compilation has been moved to the module level
to insure they need only be compiled once. Operator keys must be
sorted by length to ensure the point at which the split happens is
most greedy. Using a split keeps the regex short and removes any
statements about the left and right hand sides of the operator.

Tests added to cover the method more completely, including testing
for corner cases such as single character field or values or
operators showing up in unexpected locations. 'string' variable
renamed to 'query' and 'query_value' to avoid confusion. Named parameters
on string substitution for clarity.

Note that the tests which do self.assertRaises could more explicitly
check the exception with self.assertRaisesRegexp but that would
break compatibility with Python 2.6.

Change-Id: Icd815ff65aba9eae3f76afee3bb33e85d85bea72
Closes-Bug: #1314544

  * E122 continuation line missing indentation or outdented
  * E126 continuation line over-indented for hanging indent
  * E128 continuation line under-indented for visual indent
  * E265 block comment should start with '# '

Tested with pep8 version 1.5.6 (2014-04-14).

Change-Id: Iaea72c74d3678061b0ac837eb8fe8c85a883bb4a

We already support 'type' attribute for query, but the docstring for
build_url and cli_to_array don't refer to it.

And the urlencode with doseq=True will force filter parameters grouped
by name, so the sample in build_url docstring should be corrected too.

Change-Id: I2b526d9e23175b5378ea36e0a532052269c7f0c4

Everything is already in six>=1.4.0 and we already depend on such a
version. There's no reason to keep this file. So remove py3kcompat
from python-ceilometerclient.

Change-Id: Ide6876e7a60377cb3843816d1d9ab2c589bc8f1b
Closes-Bug: #1280033

Joe Gordon has introduced a module named misspellings which can check
typo automatically.

$ pip install misspellings
$ git ls-files | grep -v locale | misspellings -f -

Change-Id: I9eca2e097b03468651814fa563e4380fd573205e
ref: http://lists.openstack.org/pipermail/openstack-dev/2013-December/020975.html

This patch adds support for the currently available
Event API, including events, event types, and traits.

Add an optional data type to the query, ex:
   ceilometer list-events -q 'hostname=string::localhost"

bp extend-client-operations

Change-Id: Icea9bd67f8ee4ff2bf9da9ff6894218689580eb3

In Python 3.3, hash randomization is enabled by default. It causes the
iteration order of dicts and sets to be unpredictable and differ
across Python runs.

In the test case, the fixed expecting string will not match the test
result, it is relying on the dict order.

This change transforms the input dict to a sequence of two-element list,
with fixed order, and update the related expecitng string / fixture
in test cases.

Partial Implement: blueprint py33-support

Change-Id: I6dccde9e584be8335a6375f5fbad5c5cbd7b9b6d

Use the openstack common urlutils.

Partial Implement: blueprint py33-support

Change-Id: I45f21d7316fe87952e841004d4c25999138a842b

Currently, if don't specify query parameter in ceilometer statistics
command, ceilometer statistics with --period will not work. This change
is trying to fix this issue.

Change-Id: I8723bad11d5c452c2834e33df9bb01ebdc6ce9ce
Fixs: bug #1202658