| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
| |
- sync code up to Change-Id: Ie6064e73abe4b0729498a0343d50e1be35684b75
- includes fix to resolve alarm-evaluator failure
- does not include openstack/common/strutils sync as it requires
test update
Co-Authored-By: Vaibhav Kale <vaibhav_kale@persistent.co.in>
Closes-Bug: #1357343
Change-Id: Ieec08520cb39c5bf2e795dfeb3e36e52c6fd2a82
|
| |
|
|
|
|
|
| |
oslo.utils is released and corresponding code in oslo-incubator has
been removed.
Change-Id: Ic3a13380809a8b00c979e5ac405b4084f486a31d
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: I0aee668d9a19da76bb01646c87b3cd17418822e5
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| | |
version discovery and sessions aren't supported by all external
auth systems. this patch adds a fallback when discovery does not
work.
Closes-Bug: #1333961
Change-Id: Ib253c736e37f8dbc50881f9fc691bfdb4ab87d71
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When deleting an alarm, we use "ceilometer alarm-delete -a <ALARM_ID>",
unlike other deleting commands of openstack, the parameter-a/--alarm_id
is redundant. The similar situations exist in showing alarm, geting alarm
state, showing resource and so on.
It is more easy to use for reducing these parameters.
New behaviour:
$ ceilometer help alarm-show
usage: ceilometer alarm-show [<ALARM_ID>]
Show an alarm.
Positional arguments:
<ALARM_ID> ID of the alarm to show.
$ ceilometer alarm-show
alarm_id should not be empty
$ ceilometer alarm-show abcde
Not Found (HTTP 404)
$ ceilometer alarm-show -a abcde
-a is obsolete! See help for more details.
Not Found (HTTP 404)
$ ceilometer alarm-show --alarm_id abcde
--alarm_id is obsolete! See help for more details.
Not Found (HTTP 404)
Co-Authored-By: Nejc Saje <nsaje@redhat.com>
Change-Id: I1fbc85aa253929bfbb5e73ed834a725b9cf828b4
Closes-bug: #1268557
|
| | |
| |
| |
| |
| |
| |
| | |
Misspelling of "authenticated" found in docstring of
method get_client in client.py
Change-Id: I0b661a6b6008b416341ccbc691739094420723b1
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
AuthPlugin wrongly assumed that self.opts['token'] is always a function.
Change-Id: I968c62aec0d16d736693e403d03a495baf8ea15a
Closes-Bug: #1369461
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Remove intersphinx from the docs build as it triggers network calls that
occasionally fail, and we don't really use intersphinx (links other
sphinx documents out on the internet)
This also removes the requirement for internet access during docs build.
This can cause docs jobs to fail if the project errors out on
warnings.
Change-Id: I71e941e2a639641a662a163c682eb86d51de42fb
Related-Bug: #1368910
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Current behavior is to retrieve alarm by id and conduct operations on the
object. If the tenant doesn't own the alarm or isn't admin, the user will
receive the message: 'NoneType' object has no attribute 'to_dict'
Above message doesn't provide any useful diagnostic information and indicates a
programming error since an unexpected None-type is encountered and not handled.
This change verifies the alarm is found before using the object. If alarm not
found it prints the same message for a not found Alarm as other PUT operations
like alarm-state-set: Alarm not found: <alarm_id>.
This message is more useful for diagnosis and gets rid of the uncaught
None-type error.
Change-Id: I66abcd4498b24ac7cadcf29fe3ced3fcda08458c
Closes-Bug: #1348387
|
| |\ \ \ |
|
| | | |/
| |/|
| | |
| | | |
Change-Id: Ifdb40d061dd7e8e822a84f31d6f6d22e65c0ccfd
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
A dict embedded in a list is the expected input. In
47934c777c50897b649793e0960eebdaad322c45 the list was lost, this
puts it back.
Note that the _exact_ date of this failure was identified by
existing Rally jobs which started failing in early August. That made
searching the commit logs straightforward.
Change-Id: I37e794de7a23b8ce8e67a086732af5a19f182409
Closes-Bug: #1367867
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Documentation is modeled after python-novaclient docs.
(http://docs.openstack.org/developer/python-novaclient/)
Built preview for reviewers:
https://nsaje.fedorapeople.org/python-ceilometerclient/
Change-Id: I61e9d6593e4a1aea5cf018081416d4be3a9372ac
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fixes the bug that broke the alarms CLI if the alarm didn't have
a time constraints field.
Reduces code duplication of alarm printing code, so now all the alarm
printing code is actually tested.
Renames some auxiliary methods for more clarity.
Change-Id: Ib691b4a5a6cf5ae133cd0a5576f90e4d0d189a92
Closes-bug: #1316390
|
| | |/
|/|
| |
| | |
Change-Id: Ie52abd0eeab5042606855c4ce668f3b3f11a2d2c
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There was a call from install_venv.py file to post_process method
of InstallVenv class, but this method is not defined in this class
as result it was raising an error. Even when this error doesn't
affect dependencies installation, it is not allowing to display user
information.
Change-Id: I39c995adfbb411f21ad7ea222b2392da6946e25b
Closes-Bug: #1275025
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
assertTrue(*) means to check whether a boolean value/expression
is true or not. For checking an object is not None, we should use
assertIsNotNone(*), which gives more explicit expression.
Change-Id: Ia4d0dcec81e97fe4774bff4e41a017f00b899ef3
Closes-Bug: #1297156
|
| |\ \ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | | |
Change-Id: I38fd12afe0ff6b62b21d7374642a65db0deafdc1
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Endpoint should be present in auth_plugin options to allow
connect to ceilometer through specified endpoint and token
Closes-Bug: #1354507
Change-Id: I0aaafc6f599a67b242ade25efb3e667738bd71fd
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| | |
In the process of unification of the clients code we should
reuse common functionality from Oslo.
bp common-client-library-2
Change-Id: I0e027c33ee42b6de032d33269caeea33e7837f40
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
The current developer docs template used by ceilometer client is
not the latest template used by the rest of the openstack projects.
This patch will update the docs to provide a more consistent look
and feel when using project docs.
Change-Id: Ic25324da75100c0efe3e814de13807740592d4a1
|
| | |
| |
| |
| |
| |
| | |
Generated docs should be ignored, so adding doc/build to gitignore
Change-Id: I4fc03244df37f63351ec1e54b1bcd5b9b3790f5c
|
| | |
| |
| |
| |
| |
| |
| | |
This will allow developers to more easily run the docs job that
generates the developerdocs for python-ceilometerclient
Change-Id: I27bb80e476d4bef85d3a543d7b4738ec6c29b6db
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Currently, empty entity field is accepted but will fail when call cli,
it is because the empty field leads to incorrect rest api uri, which will
get 404 error.
This patch checks some required entity fields, which will present in uri path,
and raises CommandError if they are empty. Note the other required fields
will not cause the same problem, so they are not checked.
Change-Id: I59411b760ff7457064b386911e868518b3057e3a
Closes-Bug: #1313679
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Ceilometer CLI exposes X-Auth-Token in debug mode. This patch
replaces X-Auth-Token's value with '{SHA1}<sha1oftoken>'. Some
credentials are exposed by keystoneclient as ceilometerclient
uses keystoneclient to authenticate, it will be fixed in bug:
100414.
Change-Id: Ia6364314e4b4d26301f974582c0c2ba34b054c86
Partial-Bug: #1327019
|
| |\ \ \ \ |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This change makes the following improvements to the output of a
ceilometer command with the --debug option set:
* Silences spurious logging from iso8601 and urllib3.connectionpool
* Removes the line number from the logging format, since the intent
of --debug is to show the user what network requests are occuring.
Change-Id: I1b286adf63b1ca05ec4fcfb27fbc939aa662cc7e
Closes-bug: #1324470
|
| | |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Overwrite the method add_arguments in the class
OpenStackHelpFormatter to calculate a suitable columnt width
(max_help_position) for the positional arguments.
example output before this patch:
alarm-threshold-create
Create a new alarm based on computed statistics.
alarm-threshold-update
Update an existing alarm based on computed statistics.
alarm-update Update an existing alarm (Deprecated).
example output after this patch:
alarm-threshold-create Create a new alarm based on computed statistics.
alarm-threshold-update Update an existing alarm based on computed
statistics.
alarm-update Update an existing alarm (Deprecated).
Change-Id: I945e46bb9e0309b960e3322261d9d6ff9fa35b52
Closes-Bug: #1326471
|
| | |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
Replace assertTrue(a in b) with assertIn(a, b)
Replace assertFalse(a in b) with assertNotIn(a, b)
Replace assertTrue(a is None) with assertIsNone(a)
Change-Id: I981c81908a823d9dbecbb2e139a86fbf589ce55e
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Keystone is now deprecating the V2 API in favor of the V3 API.
The ceilometer client is now using the keystone session to
delegate the discovery of the version that is going to be used,
this eliminates the need of updating the client every time there
is a new API version.
DocImpact
blueprint support-keystone-v3-api
Change-Id: I6ebacce7adf70f24bdede9b920853ab9851071cf
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Currently, alarm-threshold-update command can not update its query
field, because the dest value is removed accidently so the query is
not put under threshold_rule dict. This patch simply adds it back.
Change-Id: I2eeaf8cbb26f506e097dafdcb6d0eae4ad7fec3e
Closes-Bug: #1313676
|
| |\ \ \ \
| |_|/ /
|/| | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Resource classes of managers should have methods
like delete() and get(). Basicly base.Resource
class has method get(), which uses id attribute,
but classes Alarm, Event, etc. have no this attribute.
Added intercept getting of id in the resource classes.
Also added catch of HTTPNotFound for alarm get()
because it affects correct functionality of
scenario tests.
Closes-Bug: #1312146
Change-Id: Ie598b73c64dece2185955467936dc6544a8a3dc7
|
| | | | |
| | | |
| | | |
| | | | |
Change-Id: Ic2fd2ffb3d5c464d51933e97c6715f8608770487
|
| |\ \ \ \ |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Currently, the "--resource-metadata" in ceilometer sample-create command
is not easy to use. For compatibility, this patch just provides an
explicit help string for the parameter.
Change-Id: I6d68d5b653c3b33808cba6f9a781301fba9b44e3
Closes-Bug: #1324759
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In the Ceilometer storage backends implementation we are specially
sorting the alarm history points in the following way:
the first one by the timestamp should be the last one in the list.
The Python client itself provides the information in this way, but
the table printed had the opposite one, so this was fixed.
Change-Id: If39bb5868e360f37ed76c12b55fc71eade331f47
|
