| Commit message (Collapse) | Author | Age | Files | Lines |
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| | |
The comment describes the handling of HTTPNotFound exception about
alarm, but it is ambiguous, this change make it more clear.
Closes-Bug:#1523866
Change-Id: Id7be71dcda6693875222435e10cba929e7c81e56
|
| |/
|
|
|
|
|
|
|
|
|
| |
strict the input parameter of enabled in the command :
ceilometer alarm-threshold-update <alarm-id> --enabled {True|False}
This commit will improve the enabled value validation, if the input
value is invalid such as "abde" "enabled", error info will be returned.
Closes-Bug: #1531042
Change-Id: Ic553bc9ffa05a84575fc5626416c79fc84687bbe
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: I808a2a32860a6fe6b9b1797dd59a0ebed08d7b0a
|
| |\ \
| |/ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
We must support session object that comes from keystoneauth1 and
keystoneclient.
This change adds code path for keystoneauth1 when it misses.
Change-Id: I1164a0a0745605b6bc4fd1fc8fb2813c7e40fb9b
|
| |/
|
|
|
|
| |
Closes-Bug: #1509430
Change-Id: I4f3b60776230dec5ac0ee31369fbe95cb0a8cd71
|
| |
|
|
|
|
|
|
|
|
| |
Help message for '--aggregation-method' in
alarm-gnocchi-resources-threshold-create command is not precise.
The problem is that 'avg' is not a valid aggregation method in
gnocchi. This fix updates the help message.
Change-Id: I513b6fb9db9f96bfdd10391b6acd5c95e278f601
Closes-Bug: #1506549
|
| |
|
|
|
|
|
|
|
|
| |
This adds an optional param to resource-list
CLI to enable meter-links. By default, meter
links are not generated, this patch adds an
optional param to generate links.
Change-Id: I308b94c8a57833cf30b4815ecec2ff9a50e91af6
Closes-Bug: #1508709
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Provide sample id can make it possible for end user to query particular
sample by its id, this is needed because we don't show everything in
the list.
Change-Id: I06bfc2de1a4e9a24892b2e36b6d2137e1b38cb27
Closes-Bug: #1506745
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
patch I4804c45c2ba69982163503ba5d9d4eba86f904fe has some typos in
docstring:
* :param *args: should be :param args:, there is no need to use the
leading star, meanwhile the star has special meaning in doc string
which need to appear by pairs.
* :param for this ceiloclient, actually this phase is part of last
param's help string
Change-Id: Ib7b964ffe064b805130953f6a6abca354d9eef79
|
| |/
|
|
|
|
|
|
|
|
|
|
| |
ceilometer client is a rare client that doesn't allow to use
a keystoneclient/keystoneauth session object.
This change fixes that and will allow to remove the
deprecated apiclient code in the future.
Depends-On: I43223a8b38ff1f4ef30c1d2345a28de6ade51863
Depends-On: I7cbe0afa12c9d11ce2eaed1f4d29f41507e273f8
Change-Id: I4804c45c2ba69982163503ba5d9d4eba86f904fe
|
| |
|
|
|
|
|
|
| |
This change displays the 'statistic' of an threshold rule.
When the detail of rule is asked.
Change-Id: I59d5a9603b7666da87d734a95c0ada758ae24ad3
Closes-bug: #1498901
|
| |
|
|
|
|
|
|
|
|
|
|
| |
settable => configurable
and moves the word "only"
This is a suggestion from CLI reference review.
https://review.openstack.org/#/c/221996/1
The CLI reference generation tool generates
from command help message.
Change-Id: I3a746351e922f33aba10e28adb54be03b784ccf7
|
| |
|
|
|
|
|
|
| |
limit is now a mandatory condition when querying. this patch
allows users to control this value.
Change-Id: Ied5f80dc8cfe8b5f85e60fb8ca1d4a8f26663456
Closes-Bug: #1493511
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
meter links are not accessible via resource_list and add significant
overhead to query. we shouldn't generate these anymore.
Change-Id: Ib20f410a70171aa09e4ebbbb748a5ce495fc6a43
Closes-Bug: #1493400
|
| | |
| |
| |
| | |
Change-Id: I53f3af447a8f814388985f6e4ab57a8ffec18a2a
|
| |/
|
|
| |
Change-Id: Ic29d7d5fcf95ff29112bd42fed747c54181e3640
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This change includes:
* Allow users of ceilometerclient specifying aodh_endpoint explicitly
* If aodh_endpoint didn't provided and users just provide ceilometer
endpoint and token, no auth_url, we assume aodh is unavailable and
avoid trying to get aodh endpoint.
Change-Id: Idd0ce60058d23507a99b89926e461d19db267ca2
Closes-Bug: #1488290
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Add --direct flag for ceilometer sample-create and sample-create-list
commands to support this feature
* Add parameter for ceilometerclient.v2.samples.create() and
ceilometerclient.v2 samples.create_list() to support this feature
Partially implements: blueprint api-no-pipeline
Change-Id: Ia3d787bbeaf360ddc198cb4ec183d83a5e76203a
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Currently, alarm module of ceilometer has been decided to be splitted out
of ceilometer tree. To keep compatibility for others upstream users
(such as Heat), we need to redirect alarm related request to aodh endpoint.
Change-Id: Id13b56eef55f807d5ad2026f225c93fd5585521e
|
| |\ \ |
|
| | |/
| |
| |
| | |
Change-Id: I17ca56142870313681c8ae9410db4173c2946f65
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Info about alarm's severity is missing from
the detail column for creation and deletion
type of alarm-history call.
This fix adds that.
Change-Id: I669126596ef7391530e7822c2807ffca7c6a1de1
Closes-Bug: 1468295
|
| |/
|
|
|
|
|
|
|
| |
Its well known fact that python default values initialized only once
at the stage if creation function. So if we call function with parameter
foo=[] as default value and change foo.append("bar") at the next call of
the function default value would be ["bar"]. It will save.
Change-Id: Iba9e55bc2d4dfd811ce7d068b6a9cceae7c000d6
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
When creating sample with wrong resource metadata format, Ceilometer CLI
will return a ValueError about json decoding, this change provide a more
explicit error info.
Change-Id: I18e757ddb319fbf0ffad13c07528355785ea5e9a
Closes-Bug: 1465193
|
| |/
|
|
|
|
|
|
|
|
| |
Strips leading and trailing whitespaces
from CLI query field and value parameter
before building url.
Closes-Bug: 1464694
Change-Id: Ibe99747e22afe05eee187d59aaf0603f58e91373
|
| |
|
|
|
|
|
|
|
|
| |
Added method for creating array of samples to samples manager.
Added shell command for creating array of samples.
Added tests for this changes.
It is made for Rally tests that use big data arrays. So now only one
function may be called instead of numerous calls simple create function.
Change-Id: I01ba4f0c0db40c95ef72dadaa34b3fafc034e417
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Whenever an entity wasn't found, the ceilometerclient would issue a
NotFound exception. Which is fine, except that it's not the exception
that was being expected in the client at all. This exception was defined
in the apiclient in the openstack/common directory. So, instead this
exception is now being caught in the manager, and now the appropriate
exception is being raised.
There were some functions where, if the entity was not to be found, a
404 exception would be raised, which was never caught by the client
since it was expecting the function to return None, instead of raising.
This CR fixes that.
Closes-Bug: #1451833
Change-Id: I47b34af2df9c85f86ba1f7f4d7b2951f8d27f96c
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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: I1f539cf9f5d6f48e10cac381c13eeb5fa314abd4
|
| |/
|
|
| |
Change-Id: Ie4c55f33d18294924701032031cf78539ee3b23b
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Ceilometer alarm-threshold-update should support updating
project-id and user-id, but the update function was silently
ignoring changes of these options.
Change-Id: I33231bff44b90f3c58e5ef1646349b6dc63a3ce7
Closes-Bug: 1395932
|
| | |
| |
| |
| |
| |
| | |
add ability to show raw details of event.
Change-Id: I7068eed3647b7068d3286d3d205ed4c9f4372122
|
| |\ \
| |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Currently, if we update an alarm with wrong time constraint which
doesn't get name defined, then the shell only prints a very simple
string 'name'.
This is because our code assume name field has always been specified,
however it is not true, then KeyError exception will be raised but not
handled well, finally user only gets an implicit message.
This patch uses dict.get() for name field, and sends request (may be broken)
to ceilometer api, then extracts error message from response.
Change-Id: I086c4ec790acc22767ba7f5e43dbcf73f3af5dff
Closes-Bug: #1439207
|
| |/
|
|
|
|
| |
Implements blueprint capability-cli
Change-Id: I23fe9c0bc9c27e04e3e0133fc84da7cf1f87321f
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since https://review.openstack.org/#/c/163462/ and
https://review.openstack.org/#/c/162649/4 in gnocchi, all kind of aggregations
have been moved in the same URI namespace (/v1/aggregation/...)
and we can use complex query to make aggregation across multiples metrics.
This change sync the ceilometerclient alarm shell to match these
changes.
Change-Id: I5717a4604af5505dfb8c56463e2bdb86c59e41ce
|
| |/
|
|
|
|
|
|
|
|
| |
The change of I9ba984f804aecee5b63e5a86e699733040dee245 enables
us to specify os-user-id in CLI, but it shadows the user-id when
creating sample and alarm, this patch fixes it as what we've done
for project-id.
Change-Id: Ic3ce5dc8e1aea49b3e41fe495be4921fbf85ea5a
ref-change: I0ce2416dccd61eb50584799e6df0b8c45d44cdda
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
After we port to oslo-incubator.apiclient, the project_id can no
longer be specified for alarm-{,threhsold,combination}-{create,update}
and sample-create, this is because client.AuthPlugin registers a CLI
argument named --os-project-id which will shadow the project-id
argument.
Since os-project-id is used for Keystone V3 API, we should not
remove it from AuthPlugin, so this patch moves the dest of project_id
to {alarm,sample}_project_id, and adds a decorator to restore shadowed
project_id field when we call v2 client.
Change-Id: I0ce2416dccd61eb50584799e6df0b8c45d44cdda
Closes-Bug: #1393489
|