<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/openstack/python-ceilometerclient.git/ceilometerclient/tests/unit/test_shell.py, branch 2.9.1</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>panko redirect</title>
<updated>2017-01-11T23:11:36+00:00</updated>
<author>
<name>gord chung</name>
<email>gord@live.ca</email>
</author>
<published>2017-01-11T22:52:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=72ea5ef82461d12644f0be6b80ac7dd07a4a2cec'/>
<id>72ea5ef82461d12644f0be6b80ac7dd07a4a2cec</id>
<content type='text'>
same as aodh but with panko_endpoint

Change-Id: Iae7d60e1cf139b79e74caf81ed7bdbd0bf2bc473
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
same as aodh but with panko_endpoint

Change-Id: Iae7d60e1cf139b79e74caf81ed7bdbd0bf2bc473
</pre>
</div>
</content>
</entry>
<entry>
<title>cleanup aodh mocks</title>
<updated>2017-01-11T23:01:47+00:00</updated>
<author>
<name>gord chung</name>
<email>gord@live.ca</email>
</author>
<published>2017-01-11T23:01:19+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=45f4f9584bb46e1d7f47d32ac9cfaab04a209d34'/>
<id>45f4f9584bb46e1d7f47d32ac9cfaab04a209d34</id>
<content type='text'>
don't pass in arg we don't use

Change-Id: I14b9cd3178f929aa878f5caff97a6ba4318a42a0
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
don't pass in arg we don't use

Change-Id: I14b9cd3178f929aa878f5caff97a6ba4318a42a0
</pre>
</div>
</content>
</entry>
<entry>
<title>move old oslo-incubator code out of openstack/common</title>
<updated>2016-11-05T00:18:45+00:00</updated>
<author>
<name>Steve Martinelli</name>
<email>s.martinelli@gmail.com</email>
</author>
<published>2016-10-29T16:05:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=c1141ed5be72ee169a7640b5d4957bfe02ad37ce'/>
<id>c1141ed5be72ee169a7640b5d4957bfe02ad37ce</id>
<content type='text'>
As part of the first community-wide goal, teams were asked
to remove the openstack/common package of their projects
if one existed. This was a byproduct of the old oslo-incubator
form of syncing common functionality.

The package, apiclient, was moved to a top level location
and a new i18n module was created. There is no oslo.apiclient
library or equivalent, the recommended solution is to move it
in tree and maintain it there.

Change-Id: Ia788313e0926dc872a87b090ef0a350898bfb079
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
As part of the first community-wide goal, teams were asked
to remove the openstack/common package of their projects
if one existed. This was a byproduct of the old oslo-incubator
form of syncing common functionality.

The package, apiclient, was moved to a top level location
and a new i18n module was created. There is no oslo.apiclient
library or equivalent, the recommended solution is to move it
in tree and maintain it there.

Change-Id: Ia788313e0926dc872a87b090ef0a350898bfb079
</pre>
</div>
</content>
</entry>
<entry>
<title>Replace assertEqual(None, ...) with assertIsNone(...)</title>
<updated>2016-09-19T16:59:53+00:00</updated>
<author>
<name>Hanxi Liu</name>
<email>hanxi.liu@easystack.cn</email>
</author>
<published>2016-09-19T15:11:12+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=1c53f7d09837dc670e00886e201152e05084ed14'/>
<id>1c53f7d09837dc670e00886e201152e05084ed14</id>
<content type='text'>
From OpenStack Style Guidelines [1], [H203] Use assertIs(Not)None to
check for None.

[1] http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises

Change-Id: Ib60e025af418678884d9bea91e1c318b3aadf4c9
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
From OpenStack Style Guidelines [1], [H203] Use assertIs(Not)None to
check for None.

[1] http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises

Change-Id: Ib60e025af418678884d9bea91e1c318b3aadf4c9
</pre>
</div>
</content>
</entry>
<entry>
<title>Switch to keystoneauth</title>
<updated>2016-06-01T12:24:03+00:00</updated>
<author>
<name>Clenimar Filemon</name>
<email>clenimar.filemon@gmail.com</email>
</author>
<published>2016-05-31T19:02:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=39ee8fbc087011602bcb37daa877e3dd2926a0a2'/>
<id>39ee8fbc087011602bcb37daa877e3dd2926a0a2</id>
<content type='text'>
keystoneclient.auth is deprecated in favour of keystoneauth. This patch
updates Ceilometerclient accordingly.

Change-Id: Id4b14cbccfabfeaa5322f7ae5702261d0ed9cca2
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
keystoneclient.auth is deprecated in favour of keystoneauth. This patch
updates Ceilometerclient accordingly.

Change-Id: Id4b14cbccfabfeaa5322f7ae5702261d0ed9cca2
</pre>
</div>
</content>
</entry>
<entry>
<title>remove default value of repeat-actions</title>
<updated>2016-03-10T13:43:09+00:00</updated>
<author>
<name>ZhiQiang Fan</name>
<email>aji.zqfan@gmail.com</email>
</author>
<published>2016-03-07T17:43:44+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=c8ff0e7f9672298859298960084721ee520fa060'/>
<id>c8ff0e7f9672298859298960084721ee520fa060</id>
<content type='text'>
I94468ff649dd4367848c74e94a3feb08494bb223 fixes problem that ceilometeclient
incorrectly reset repeat-actions when update any attribute.

I53f3af447a8f814388985f6e4ab57a8ffec18a2a reverts most of the above change,
which leading to bug 1539092. Because repeat-actions has default value False,
no matter we set it or not, it will replace this bit when we get alarm from API.

The unit test doesn't cover such case, so this change still succeeds in CI test.

We don't need to revert it again, because the redundant code for create and update
is not needed, repeat-actions has default value (False) in API side, so the proper
way is simply removing the default value in CLI side.

Change-Id: I39c28294db3e55141bb8a2024a6ddfdf8acf5e0d
Closes-Bug: #1539092
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
I94468ff649dd4367848c74e94a3feb08494bb223 fixes problem that ceilometeclient
incorrectly reset repeat-actions when update any attribute.

I53f3af447a8f814388985f6e4ab57a8ffec18a2a reverts most of the above change,
which leading to bug 1539092. Because repeat-actions has default value False,
no matter we set it or not, it will replace this bit when we get alarm from API.

The unit test doesn't cover such case, so this change still succeeds in CI test.

We don't need to revert it again, because the redundant code for create and update
is not needed, repeat-actions has default value (False) in API side, so the proper
way is simply removing the default value in CLI side.

Change-Id: I39c28294db3e55141bb8a2024a6ddfdf8acf5e0d
Closes-Bug: #1539092
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "Don't copy the auth_plugin for aodh"</title>
<updated>2016-01-12T11:01:03+00:00</updated>
<author>
<name>Jenkins</name>
<email>jenkins@review.openstack.org</email>
</author>
<published>2016-01-12T11:01:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=25517d0226e69bbabf9679c80445746a5cab846d'/>
<id>25517d0226e69bbabf9679c80445746a5cab846d</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Don't copy the auth_plugin for aodh</title>
<updated>2016-01-07T15:30:14+00:00</updated>
<author>
<name>Mehdi Abaakouk</name>
<email>sileht@redhat.com</email>
</author>
<published>2016-01-06T08:44:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=71cb6b3d0ee1f393c2bafee13aa2f013cc82f77d'/>
<id>71cb6b3d0ee1f393c2bafee13aa2f013cc82f77d</id>
<content type='text'>
We must not copy a keystone session object or auth_plugin object.

This change simplifies the aodh redirection code by:
* sharing the keystone session object between aodh and ceilometer client
* creating a new AuthPlugin() for the aodh client instead of copying the ceilometer one
  and then change its content.

Closes-bug: #1531452

Change-Id: I2b2195e32c5dd74136237f7166c9c0d325434611
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We must not copy a keystone session object or auth_plugin object.

This change simplifies the aodh redirection code by:
* sharing the keystone session object between aodh and ceilometer client
* creating a new AuthPlugin() for the aodh client instead of copying the ceilometer one
  and then change its content.

Closes-bug: #1531452

Change-Id: I2b2195e32c5dd74136237f7166c9c0d325434611
</pre>
</div>
</content>
</entry>
<entry>
<title>Use assertTrue/False instead of assertEqual(T/F)</title>
<updated>2016-01-07T07:08:00+00:00</updated>
<author>
<name>Swapnil Kulkarni (coolsvap)</name>
<email>me@coolsvap.net</email>
</author>
<published>2016-01-07T07:07:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=c4abadfee8de8f64f91e4579b0f58fc7b4943462'/>
<id>c4abadfee8de8f64f91e4579b0f58fc7b4943462</id>
<content type='text'>
The usage of assertEqual(True/False, ***) should be changed
to a meaningful format of assertTrue/False(***).

Change-Id: Iabf8d017b47025309f186afb5e30b8ad990c2448
Closes-Bug:#1512207
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The usage of assertEqual(True/False, ***) should be changed
to a meaningful format of assertTrue/False(***).

Change-Id: Iabf8d017b47025309f186afb5e30b8ad990c2448
Closes-Bug:#1512207
</pre>
</div>
</content>
</entry>
<entry>
<title>Replace assertEqual(None, *) with assertIsNone in tests</title>
<updated>2015-12-17T13:06:01+00:00</updated>
<author>
<name>Shuquan Huang</name>
<email>huang.shuquan@99cloud.net</email>
</author>
<published>2015-12-17T13:06:01+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/openstack/python-ceilometerclient.git/commit/?id=a63f15272c61d5b3ab54d00da1209e4d6bf6bcad'/>
<id>a63f15272c61d5b3ab54d00da1209e4d6bf6bcad</id>
<content type='text'>
Replace assertEqual(None, *) with assertIsNone in tests to have
more clear messages in case of failure.

Change-Id: I36db8bdcb67b8cc0a3bf1f063b4a7b42955b100b
Closes-bug: #1280522
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Replace assertEqual(None, *) with assertIsNone in tests to have
more clear messages in case of failure.

Change-Id: I36db8bdcb67b8cc0a3bf1f063b4a7b42955b100b
Closes-bug: #1280522
</pre>
</div>
</content>
</entry>
</feed>
