Bug 1845014 - Grafana dashboard shows the listener IP addresses of controller nodes in the "Nodes" list.
Summary: Grafana dashboard shows the listener IP addresses of controller nodes in the ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Service Telemetry Framework
Classification: Red Hat
Component: Documentation
Version: 1.2
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: z2
: ---
Assignee: Paul Leimer
QA Contact: Leonid Natapov
Joanne O'Flynn
URL:
Whiteboard:
Depends On: 1912895 1943801
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-06-08 09:44 UTC by Leonid Natapov
Modified: 2021-06-24 14:48 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-06-24 14:48:19 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 742685 0 None MERGED Memcached collectd plugin uses host URI instead of IP address. 2021-01-05 13:46:34 UTC

Internal Links: 1943801

Description Leonid Natapov 2020-06-08 09:44:59 UTC
In addition to the overlcoud nodes names,Grafana dashboard also shows listener IP addresses of controller nodes in the "Nodes" list. In the "Nodes" list shoule appear only overcloud nodes names.

Comment 3 Paul Leimer 2020-08-05 19:34:16 UTC
https://review.opendev.org/#/c/742685/

Comment 9 Leonid Natapov 2021-01-05 14:24:55 UTC
memcached is still reporting it by IP address. Moving back to Assign

Comment 10 Paul Leimer 2021-04-28 17:32:49 UTC
This can be fixed by adding the following to the stf-connectors.yml:

```
    ExtraConfig:
      collectd::plugin::memcached::instances:
        local:
          host: "%{hiera('fqdn_canonical')}
          port: 11211
```

Comment 12 Paul Leimer 2021-05-05 12:52:22 UTC
Hi Joanne,

Originally I figured this would go in the dashboard section, but I think it would be better to be upfront about the change. The section you pointed out would be better IMO.

We can add a bullet point that goes as follows:

- In the ExtraConfig, add the following to tell the collectd memcached to report the hostname by fqdn rather than ip address in accordance with the rest of the collectd plugins:

>       collectd::plugin::memcached::instances:
>         local:
>           host: "%{hiera('fqdn_canonical')}
>           port: 11211

And then below, where the whole stf-connectors.yaml is laid out, also include the changes:

> parameter_defaults: 
>     EventPipelinePublishers: []
>     PipelinePublishers: []
>     CeilometerQdrPublishEvents: true
>     CeilometerQdrPublishMetrics: true
>     MetricsQdrConnectors:
>     - host: default-interconnect-5671-service-telemetry.apps.infra.watch
>       port: 443
>       role: edge
>       sslProfile: sslProfile
>       verifyHostname: false
>     ExtraConfig:
>       ceilometer::agent::polling::polling_interval: 5
>       collectd::plugin::memcached::instances:
>         local:
>           host: "%{hiera('fqdn_canonical')}
>           port: 11211

Comment 15 Paul Leimer 2021-06-10 19:11:05 UTC
This issue will be solved by updates to our enable-stf.yaml and will no longer need a documentation update: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795608/4/environments/enable-stf.yaml

Comment 16 Leif Madsen 2021-06-10 19:20:24 UTC
(In reply to Paul Leimer from comment #15)
> This issue will be solved by updates to our enable-stf.yaml and will no
> longer need a documentation update:
> https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795608/4/
> environments/enable-stf.yaml

Actually I think we'll need to include this in the docs in the short term. I think the way this will go will be:

1) change as you've described here

2) migrate to documenting the enable-stf.yaml and changing the installation procedure slightly, with a copy/pasta of the new enable-stf.yaml

3) once changes are available in the downstream product, refer to the in-tree enable-stf.yaml and remove the documented copy/pasta method


Note You need to log in before you can comment on or make changes to this bug.