<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/openstack/python-ceilometerclient.git/ceilometerclient/tests/v2/test_options.py, branch juno-eol</title>
<subtitle>opendev.org: openstack/python-ceilometerclient.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/'/>
<entry>
<title>Use HTTPClient from common Oslo code</title>
<updated>2014-07-31T15:59:58+00:00</updated>
<author>
<name>ekudryashova</name>
<email>ekudryashova@mirantis.com</email>
</author>
<published>2014-01-23T17:28:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=47934c777c50897b649793e0960eebdaad322c45'/>
<id>47934c777c50897b649793e0960eebdaad322c45</id>
<content type='text'>
In the process of unification of the clients code we should
reuse common functionality from Oslo.

bp common-client-library-2

Change-Id: I0e027c33ee42b6de032d33269caeea33e7837f40
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In the process of unification of the clients code we should
reuse common functionality from Oslo.

bp common-client-library-2

Change-Id: I0e027c33ee42b6de032d33269caeea33e7837f40
</pre>
</div>
</content>
</entry>
<entry>
<title>Refactor split_by_op and split_by_datatype</title>
<updated>2014-06-02T17:21:51+00:00</updated>
<author>
<name>Chris Dent</name>
<email>chdent@redhat.com</email>
</author>
<published>2014-05-30T10:40:11+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=2a2e2dd2732240a08d7f0e1288229a39ae380d1c'/>
<id>2a2e2dd2732240a08d7f0e1288229a39ae380d1c</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>Support the Event API</title>
<updated>2014-01-10T19:00:44+00:00</updated>
<author>
<name>John Herndon</name>
<email>john.herndon@hp.com</email>
</author>
<published>2013-10-30T16:44:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=c74231e74473cdfc8037731b6e9beb539d491814'/>
<id>c74231e74473cdfc8037731b6e9beb539d491814</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>align the order of parameters for urlencode()</title>
<updated>2013-10-14T16:51:58+00:00</updated>
<author>
<name>Kui Shi</name>
<email>skuicloud@gmail.com</email>
</author>
<published>2013-10-14T16:35:58+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=c58320a4eaa6743e59b31ec90f9f4f38cb9e3b3c'/>
<id>c58320a4eaa6743e59b31ec90f9f4f38cb9e3b3c</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>Enhance ceilometer statistics command with --period</title>
<updated>2013-07-23T12:51:17+00:00</updated>
<author>
<name>Guangyu Suo</name>
<email>guangyu@unitedstack.com</email>
</author>
<published>2013-07-19T02:34:17+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=3a780a6159e53a25e722ec083ab45d915279958a'/>
<id>3a780a6159e53a25e722ec083ab45d915279958a</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>Move tests to ceilometerclient.</title>
<updated>2013-07-01T05:40:55+00:00</updated>
<author>
<name>Monty Taylor</name>
<email>mordred@inaugust.com</email>
</author>
<published>2013-07-01T05:40:55+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=6f81aa1a99b4e073855270120d18643ee0a7133e'/>
<id>6f81aa1a99b4e073855270120d18643ee0a7133e</id>
<content type='text'>
tests.* implies an incorrect global package name. Additionally,
consuming code can benefit from being able to choose to consume
test code and fixtures.

Change-Id: I7ba2b3ba1c2a99410b54fc40b48dfe2fc53eb79a
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
tests.* implies an incorrect global package name. Additionally,
consuming code can benefit from being able to choose to consume
test code and fixtures.

Change-Id: I7ba2b3ba1c2a99410b54fc40b48dfe2fc53eb79a
</pre>
</div>
</content>
</entry>
</feed>
