Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Include project_name and user_name fields to metric messages #110

Closed
yadneshk opened this issue May 2, 2023 · 0 comments · Fixed by #111
Closed

Include project_name and user_name fields to metric messages #110

yadneshk opened this issue May 2, 2023 · 0 comments · Fixed by #111

Comments

@yadneshk
Copy link
Contributor

yadneshk commented May 2, 2023

As of now ceilometer sample for each resource has UUID of the project it belongs to and the user that created it. Identifying resources from the dashboard with such details isn't ideal, cloud administrators will have to manually list these resources in keystone to know their names.

Including these fields for all resources on dashboard would help administrators to understand it's ownership.
yadneshk added a commit to yadneshk/sg-core that referenced this issue May 2, 2023
@yadneshk
Make room for project/user names in metric messages
debcdeb 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
@yadneshk yadneshk mentioned this issue May 2, 2023
Merged
yadneshk added a commit to yadneshk/sg-core that referenced this issue May 2, 2023
@yadneshk
Make room for project/user names in metric messages
4b38519 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
yadneshk added a commit to yadneshk/sg-core that referenced this issue May 2, 2023
@yadneshk
Make room for project/user names in metric messages
d1ccbeb 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
yadneshk added a commit to yadneshk/sg-core that referenced this issue May 2, 2023
@yadneshk
Make room for project/user names in metric messages
eb1251f 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
yadneshk added a commit to yadneshk/sg-core that referenced this issue May 2, 2023
@yadneshk
Make room for project/user names in metric messages
4c79806 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
yadneshk added a commit to yadneshk/sg-core that referenced this issue Jul 18, 2023
@yadneshk
Make room for project/user names in metric messages
a4ddd8d 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
yadneshk added a commit to yadneshk/sg-core that referenced this issue Aug 29, 2023
@yadneshk
Make room for project/user names in metric messages
d922bad 
With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: infrawatch#110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby
leifmadsen added a commit that referenced this issue Sep 21, 2023
@yadneshk @leifmadsen
Add new fields to sg-core metric messages (#111)
0a5d5bc 
* Make room for project/user names in metric messages

With changes [1] Ceilometer is now able to include "project_name"
and "user_name" fields into it's metrics. These fields are only
added when ceilometer has `tenant_name_discovery` config enabled.

This change amends these fields to the outgoing metric messages.

Fixes: #110

[1] https://review.opendev.org/q/topic:enhanced_metrics-stable/wallaby

* Add resource name field to outgoing sg-core metrics

Few Ceilometer meters like `cpu` & `network` have 2 name fields
in their metrics, `name` and `display_name`. Use delimiter ":"
to merge the values of both fields.

The `name` field provides extra information about the actual resource
related to the instance like `cirros:tap3a84e3fc-d9` in `network` meter
and `cirros:instance-00000001` in `cpu` meter.

---------

Co-authored-by: Leif Madsen <lmadsen@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add new fields to sg-core metric messages yadneshk/sg-core
1 participant
@yadneshk